[codex] export approved managed skills immediately#222
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 00e1f5593f
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
Dashboard approval of a managed skill previously only flipped its lifecycle state; the overlay/prompt-index export to agent hosts ran only during `tracedecay install` / `update-plugin`, so approved skills never reached any agent until the next install. - Add `AgentIntegration::export_managed_skills` hook that refreshes the managed-skill overlay (Cursor, Codex) or prompt index (Claude Code, OpenCode, Kimi, Kiro, Copilot, Vibe) for already-detected installs without touching other config. - Add `export_managed_skills_to_agents` sweep that collects per-agent success/failure instead of aborting, and run it from the dashboard approve/disable/archive/restore handlers; results are reported on the response payload as `skill_exports` and export failures never fail the lifecycle action itself. - Surface per-agent export results in the ManagedSkillsSection UI. - Isolate HOME/USERPROFILE in dashboard API test fixtures so lifecycle endpoints never write into the developer's real agent configs.
ScriptedAlchemy
force-pushed
the
codex/export-approved-skills
branch
from
d6fd5b5 to
40a11e9
Compare
1 participant
Summary
skill_exportsreports in the dashboard API response.Validation
git diff --check origin/master...HEADCARGO_TARGET_DIR=/tmp/tracedecay-target/r1-export-approved-skills cargo check --workspaceCARGO_TARGET_DIR=/tmp/tracedecay-target/r1-export-approved-skills cargo test --test agent_suite skill_targets_test -- --nocaptureCARGO_TARGET_DIR=/tmp/tracedecay-target/r1-export-approved-skills cargo test --test dashboard_api_test automation_skills -- --nocapturenpm run test:dom -- curation-panel.vitest.tsxnpm testNotes: initial bare cargo test attempts hit
/scratch/cargo-targetdisk exhaustion, so verification was rerun with the repo-documented per-taskCARGO_TARGET_DIRunder/tmp.TraceDecay'd ~135k tokens during semantic diff/risk review.