Skip to content

manifest: recognize the available_models config key - #311

Open
jmm-ant wants to merge 2 commits into
anthropics:mainfrom
jmm-ant:jmm/available-models-key
Open

manifest: recognize the available_models config key#311
jmm-ant wants to merge 2 commits into
anthropics:mainfrom
jmm-ant:jmm/available-models-key

Conversation

@jmm-ant

@jmm-ant jmm-ant commented Aug 7, 2026

Copy link
Copy Markdown

What

Adds available_models to the install plugin's KEYS validator in build-manifest.mjs, and documents it in commands/manifest.md (org-wide manifest layer) and commands/bootstrap.md (per-user layer).

Why

The add-in now reads a single available_models override for the model picker (anthropics/office-agent#2354, merged 2026-08-05), replacing the retired disabled_models/additional_models pair that #294 targeted before it closed. Without this entry the build script throws unknown key and admins cannot deliver the key at all.

Format

Comma-separated model ids, or JSON — an array of ids and/or {id, label} objects (lone object = single-entry shorthand). Mirrors the add-in's parseModelIdEntries: validator warns where the runtime would silently drop an entry, and hard-fails only on invalid JSON.

Tested

Ran build-manifest.mjs directly: comma and JSON forms build and URL-encode into the manifest; a {label}-only entry warns missing string "id"; malformed JSON fails with the parse error.

Adds available_models to the KEYS validator (comma-separated ids or JSON
array of ids / {id,label} objects, mirroring the add-in's parser) and
documents it in manifest.md and bootstrap.md. Replaces the retired
disabled_models/additional_models pair from anthropics#294: available_models is a
single override list, so admins state the full picker rather than diffs.

No-Verification-Needed: exercised build-manifest.mjs directly against valid and invalid inputs
mihilmy
mihilmy previously approved these changes Aug 7, 2026
No-Verification-Needed: version bump only
@jmm-ant
jmm-ant enabled auto-merge (squash) August 7, 2026 20:26
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