Skip to content

feat: add versioned capability ownership and narrow deployment - #90

Merged
roodboi merged 1 commit into
mainfrom
codex/capability-ownership
Oct 7, 2026
Merged

roodboi merged 1 commit into
mainfrom
codex/capability-ownership

Conversation

@roodboi

@roodboi roodboi commented Oct 7, 2026 •

Copy link
Copy Markdown
Contributor

A skill installer can replace a managed symlink with a standalone directory, leaving canonical content stale. Another installer can follow the link and change canonical content directly. Add fclt capability to detect both cases and deploy one explicitly owned copy from a pinned source.

The schema-versioned registry records ownership, origin, revision, digest, and host/platform selection. Ordered overlay files keep machine/private choices separate. digest, inventory, plan, and hash-checked apply preserve divergent files, report missing credential references without resolving values, and leave native/external plugin files independently owned. Updates retain the prior target under the separate runtime state root; repeated apply is a no-op. Existing broad sync remains deprecated and the read-only text-file deployment contract is unchanged.

Native records describe intent, not verified installation. Disabled entries report retained files and do not claim provider disablement. Runtime caches cause a per-entry conflict instead of aborting inventory or deleting content. Acquisition/import/update and recovery procedures are documented in docs/capability-registry.md and bundled plugin guidance.

Validation:

  • 29 ownership regressions; focused capability/builtin/remote set: 90 passed.
  • Lint, type checking, build, compiled verifier, package dry run, diff checks, and public-surface privacy review passed.
  • Compiled binary smoke in a temporary home: digest, plan, apply, inventory, and repeat no-op passed.
  • Full local run was performed. A follow-up CI-mode run had 1085 passing tests and one existing Codex cold-start isolation fixture failure, also reproduced on unchanged PR fix: publish npm releases through trusted OIDC identity #88. An audit timing failure passed a focused longer-timeout rerun. The generated builtin mismatch from editing plugin docs was fixed by regenerating embedded assets; focused tests pass.
  • The commit hook was bypassed only after these explicit checks because of that reproduced unrelated local fixture failure. Remote exact-commit CI remains a merge gate.

Apply coordinates fclt writers and revalidates content before publication; unrelated installers should be stopped during apply. This PR does not deploy real tool homes or prove provider discovery/runtime behavior.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ✅ Completed 2026-10-07T14:47:31.645352Z 06d42b0 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@roodboi
roodboi merged commit 125dbe4 into main Oct 7, 2026
4 checks passed
@roodboi
roodboi deleted the codex/capability-ownership branch October 7, 2026 14:47
github-actions Bot pushed a commit that referenced this pull request Oct 7, 2026
## [2.32.0](v2.31.4...v2.32.0) (2026-10-07)

### Features

* add versioned capability ownership and narrow deployment ([#90](#90)) ([125dbe4](125dbe4))
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown

🎉 This PR is included in version 2.32.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant