Keep command metadata current across ZCode model releases - #45
Closed
bwestlund17 wants to merge 1 commit into
Closed
Keep command metadata current across ZCode model releases#45bwestlund17 wants to merge 1 commit into
bwestlund17 wants to merge 1 commit into
Conversation
Pending ACP sessions now select the first enabled model from ZCode configuration instead of pinning GLM-5.2, and slash-command hints describe stable argument classes rather than enumerating transient model and reasoning catalogs. Constraint: ZCode updates its model catalog independently of the ACP bridge package. Rejected: Add GLM-5.3 to another static hint | the next model release would recreate the same drift. Confidence: high Scope-risk: narrow Directive: Derive selectable models from ZCode config; do not hardcode release-specific model ids in command metadata. Tested: build; typecheck; lint; 463 Vitest tests; source-built package installed globally; isolated Paseo returned 78 GLM-5.3 commands. Not-tested: npm publication requires the package maintainer.
Owner
|
Thanks! Reviewed in a clean worktree — typecheck + 463 tests pass. Heads-up before you rebase: #47 just merged (squash 2e364dd), and it overlaps with this PR in two places:
Your slash-hint de-hardcoding ( |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
/modeland/thoughthints with stable argument descriptionsProblem
The bridge still selected
GLM-5.2for pending sessions and advertisedGLM-5.2|GLM-5-Turboin/modelmetadata. ZCode 0.16.1 now puts GLM-5.3 first, so the ACP surface drifted as soon as the model catalog changed.Verification
npm run buildnpm run typechecknpm run lintnpm test(463 tests)