Skip to content

chore(skills/update-models): handle doc-repo und improve retirement handling#1949

Merged
KavithaSiva merged 7 commits into
mainfrom
davidkna-sap_update-models-skill-table
Jun 22, 2026
Merged

chore(skills/update-models): handle doc-repo und improve retirement handling#1949
KavithaSiva merged 7 commits into
mainfrom
davidkna-sap_update-models-skill-table

Conversation

@davidkna-sap

@davidkna-sap davidkna-sap commented Jun 22, 2026

Copy link
Copy Markdown
Member

Context

Closes https://github.com/SAP/ai-sdk-js-backlog/issues/595

Also link .agents to .claude for better non-CC skill handling.

What this PR does and why it is needed

@davidkna-sap davidkna-sap requested a review from KavithaSiva June 22, 2026 09:19
@hyperspace-insights

Copy link
Copy Markdown
Contributor

Summary

The following content is AI-generated and provides a summary of the pull request:

⚠️ Warnings:

  • Could not get issue SAP/ai-sdk-js-backlog#595. Status: 404 - UnknownObjectException

Improve update-models Skill: Add Doc-Repo Sync and Enhanced Retirement Handling

Chore

🔧 Enhanced the update-models Claude skill to also sync the deprecated models table in the SAP/ai-sdk docs repo, and improved the retirement model tracking logic to distinguish between retired and deprecated models in release notes and type generation.

Changes

  • .claude/skills/update-models/SKILL.md: Extended the skill workflow to:

    • Clone SAP/ai-sdk docs repo and sync the docs-js/overview.mdx deprecated models table (remove retired entries, update replacements, append newly deprecated models).
    • Open a separate draft PR to SAP/ai-sdk for doc changes alongside the existing ai-sdk-js PR.
    • Expanded retired model checks to include models with retired: "yes" flag, not just deprecated ones.
    • Renumbered and clarified all steps.
  • scripts/extract-model-table.js: Updated the browser-side extraction script to return { active: [...], retired: [...] } instead of a flat array. Added findRetiredTable() and extractRetiredRows() functions to also scrape the "Recently Retired Models" table from SAP Notes.

  • scripts/sap-models.json: Reorganized and extended the model data:

    • Added retired: "yes" entries for many historically retired models (e.g., gpt-35-turbo, gemini-1.5-flash, anthropic--claude-4-opus, legacy Vertex AI and Bedrock models).
    • Corrected duplicate/misaligned entries for amazon--nova-* and anthropic--claude-* models.
    • Removed entries that had been moved to retired status.
  • scripts/sync-model-types.ts:

    • Added retired? field to the ModelRow interface.
    • Updated isRetired() to check the new retired field.
    • Updated buildReleaseNote() to group removed models into "retired" vs "deprecated" with distinct labels.
    • Propagated isRetiredFromTable through the model map and release note builder.
    • Added a summary log of retired models after sync.
  • packages/core/src/model-types.ts: Removed anthropic--claude-4-opus from AwsBedrockChatModel as it has been retired (replaced by anthropic--claude-4.7-opus).

  • .changeset/model-types-sync-2026-06-17.md: Updated changeset to include removal of anthropic--claude-4-opus and o3-mini.

  • .agents: Added .claude/ reference for agents configuration.


  • 🔄 Regenerate and Update Summary
  • ✏️ Insert as PR Description (deletes this comment)
  • 🗑️ Delete comment
PR Bot Information

Version: 1.26.4

  • Correlation ID: da7bc552-93b1-484e-a934-9a676dadc908
  • Event Trigger: pull_request.opened
  • LLM: anthropic--claude-4.6-sonnet
  • Output Template: Default Template
  • Summary Prompt: Default Prompt
  • File Content Strategy: Full file content

@KavithaSiva KavithaSiva left a comment

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.

Good idea, to classify the model results into active, retired. Could you also please adapt the merge condition to avoid duplicates in the json file?

Comment thread scripts/sap-models.json Outdated
Comment thread scripts/sap-models.json Outdated
@davidkna-sap davidkna-sap force-pushed the davidkna-sap_update-models-skill-table branch from f211ab9 to 1529942 Compare June 22, 2026 12:14
@davidkna-sap davidkna-sap requested a review from KavithaSiva June 22, 2026 12:16
Comment thread scripts/extract-model-table.js Outdated
Comment thread scripts/extract-model-table.js Outdated

@KavithaSiva KavithaSiva left a comment

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.

LGTM, Thanks for taking care.

@KavithaSiva KavithaSiva merged commit 4f0bbf0 into main Jun 22, 2026
14 checks passed
@KavithaSiva KavithaSiva deleted the davidkna-sap_update-models-skill-table branch June 22, 2026 13:47
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