Repository navigation
feat(install): share skill sources across local agents - #5
sungyongcho wants to merge 2 commits into
Conversation
Summary Install one shared Commit It source for Codex, Claude Code and Gemini. Changes - Add guarded installation, explicit updates and provider discovery links. - Document per-agent setup and release-time version selection. - Preserve the merged conflict-resolution policy and upstream version 2.9.0. Verification - 16 package tests passed; release metadata and shell syntax checks passed. - Codex discovery and invocation passed; Claude discovery passed. - Claude invocation blocked by expired OAuth; Gemini CLI unavailable. Refs: #3
Summary Verify physical source identity without assuming macOS temporary paths are canonical. Changes - Compare linked installation targets using samefile instead of raw path spelling. Verification - 16 local package tests passed; diff check passed. - The original macOS CI failure showed /var resolving to /private/var. - Cross-platform CI will verify this correction on the pushed commit. Refs: #3
sungyongcho-ops
left a comment
There was a problem hiding this comment.
Self-review: LGTM
Worker: 01a07ca1-b13b-7e52-89af-30369e17ee08
Reviewed head: 880710098336ab2e8bfdc82d06126474ab89fefe.
Reviewed the exact installer and documentation delta. The macOS alias correction preserves physical file identity checks. Linux and macOS CI passed for this head; the inherited conflict-resolution policy and release version remain intact. Runtime authentication limitations are documented.
|
Superseded by merged PR #4 at 13274c5. That release incorporates this installation work and adds stronger source, inventory and recovery checks. Its macOS fixtures also canonicalize the temporary directory, covering the path-alias correction from this branch. No unique implementation remains to merge from this PR; preserve the newer published source. |
Codex, Claude Code and Gemini now use one local Commit It source with provider discovery links. The standalone installer preserves existing work, requires explicit updates and ships copyable per-agent setup instructions. Release numbering is decided at release time; the inherited 2.9.0 version and merged conflict-resolution policy are preserved.
Validation: 16 package tests, release metadata, shell syntax and diff checks passed. Codex discovery and invocation passed. Claude discovery passed, but invocation is blocked by the existing expired OAuth session; Gemini CLI is not installed.
Closes #3