Skip to content

Remove Arch-Function, prompt targets, and function calling - #1009

Open
Spherrrical wants to merge 1 commit into
mainfrom
cursor/f47d03f5
Open

Remove Arch-Function, prompt targets, and function calling#1009
Spherrrical wants to merge 1 commit into
mainfrom
cursor/f47d03f5

Conversation

@Spherrrical

Copy link
Copy Markdown
Collaborator

Summary

  • Remove the Arch-Function runtime path (prompt_gateway/function_calling → hosted Arch-Function), including the brightstaff handler and CLI auto-inject of the arch-function provider.
  • Drop prompt_targets from config/schema/common and simplify prompt_gateway to a passthrough filter.
  • Delete related demos, docs, skills, and tests; keep the Envoy plano cluster for Plano-Orchestrator.

Test plan

  • cargo test -p common --lib
  • cargo test -p brightstaff --lib
  • cargo check --target=wasm32-wasip1 -p prompt_gateway -p llm_gateway
  • Confirm generated configs no longer include arch-function
  • Confirm POST /function_calling returns 404
  • Smoke planoai up with a model-listener config (e.g. llm_gateway demo)

@Spherrrical
Spherrrical marked this pull request as ready for review August 11, 2026 18:48

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's keep archfc.katanemo.dev for now.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same, let's keep archfc.katanemo.dev for now.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Spherrrical For these headers, can we really remove them? We just want to kill Arch-Function, but not full function calling logic. The headers like X_ARCH_STATE_HEADER are general and applicable to any other models that support function calling. Double check here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants