Skip to content

feat: auto-refresh skills on container attach - #15

Merged
Tomeriko96 merged 2 commits into
mainfrom
fix/skills-post-attach
Sep 14, 2026
Merged

Tomeriko96 merged 2 commits into
mainfrom
fix/skills-post-attach

Conversation

@Tomeriko96

Copy link
Copy Markdown

Closes #14

Auto-refetch CEDA skills from cedanl/.github on every container attach, so skill updates are pulled in without a rebuild.

What

Adds .devcontainer/post-attach.sh, wired via postAttachCommand in devcontainer.json. It mirrors the skills add install from post-create.

Why low-noise

The skills CLI has no check-only mode and no quiet flag — any refresh works by cloning the source repo. So:

  • A cheap git ls-remote (~100ms, never clones) is compared against a stored source revision.
  • No change → the script exits silently and instantly (zero attach noise, zero clone).
  • Changed → only then does it run the full skills add, quietly, and prints a single [skills] Updated... line.

Verified

  • bash -n syntax check passes.
  • First run (no stored ref): refreshes + prints + records revision.
  • Stale-ref run: refreshes + prints + updates revision.
  • Current-ref run: silent, exit 0.
  • Offline / missing git: silent skip.

Tomer Iwan added 2 commits August 31, 2026 13:23
Add .devcontainer/post-attach.sh wired via postAttachCommand. It checks
cedanl/.github via a cheap git ls-remote and only runs the skills install
when the remote actually moved; otherwise it exits silently. This pulls in
skill updates on every attach without slowing or flooding an up-to-date
container.

Closes #14
Extract the skills install command into .devcontainer/install-skills.sh so
post-create.sh and post-attach.sh cannot drift. Aligns post-attach output
with the repo [OK]/[SKIPPED] marker convention and documents why the
pipefail/|| true guard is required on the ls-remote check.

Ref #14

Entire-Checkpoint: 8123988edbfb
@Tomeriko96
Tomeriko96 merged commit 7b32afd into main Sep 14, 2026
1 check passed
@Tomeriko96
Tomeriko96 deleted the fix/skills-post-attach branch September 14, 2026 08:34
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.

Update skills

1 participant