Skip to content

fix: preserve registered integration skills during extension update - #5005

Open
chelsealong wants to merge 1 commit into
github:mainfrom
chelsealong:fix/codex-spec-kit-5002
Open

chelsealong wants to merge 1 commit into
github:mainfrom
chelsealong:fix/codex-spec-kit-5002

Conversation

@chelsealong

Copy link
Copy Markdown
Contributor

Description

Fixes #5002. Updating an extension now refreshes commands and native skills for integrations recorded in its previous registry entry, including a non-default Codex integration. An integration with no prior registration remains untouched.

Testing

  • Regression test: failed before the fix because the Codex SKILL.md was deleted; both positive and negative cases pass after the fix.
  • .venv/bin/python -m pytest tests/specify_cli/extensions/test_command_update.py tests/specify_cli/extensions/test_command_update_transaction.py tests/specify_cli/extensions/test_command_update_artifacts.py -q — 61 passed.
  • .venv/bin/specify --help and git diff --check — passed.
  • Ruff reports 20 findings in the touched files; the unchanged files report the same 20 findings.

AI Disclosure

  • I used AI assistance.

AI disclosure: Codex (GPT-6, autonomous mode) investigated the issue and generated the code and regression test. No human review is claimed.

@chelsealong
chelsealong requested a review from mnriem as a code owner October 17, 2026 10:36
@mnriem mnriem added the triage-must-have Verdict: high-value, important work for Spec Kit — do first label Oct 18, 2026

This branch has not been deployed

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

Labels

triage-must-have Verdict: high-value, important work for Spec Kit — do first

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug]: extension update removes extension skills of non-default integrations

2 participants