Skip to content

Tell agents how to follow a move or copy, and load the skill for one - #984

Merged
robzolkos merged 2 commits into
mainfrom
skill-moves-copies
Oct 28, 2026
Merged

robzolkos merged 2 commits into
mainfrom
skill-moves-copies

Conversation

@robzolkos

@robzolkos robzolkos commented Oct 28, 2026 •

Copy link
Copy Markdown
Collaborator

The skill triggered only on "move card", so a request to move or copy something else into another project might not load it. And the rule that a copy is never repeated lived only in the move and copy commands' own help, which an agent retrying out of habit never reads.

  • Triggers: move to another project, copy to another project, copy todo, copy card.
  • One non-negotiable rule: use the new ID a move or copy prints, and never repeat a copy. A copy still running or one that lost its answer may already exist, so check it with basecamp filings show <id> or the hint it printed.
  • Eval case copy-still-running.yml: asked whether a running copy finished, the agent runs filings show and never todos copy. Not run against a model here (no API key).

A live headless-agent check of v0.20.0's moves (22 natural-language requests, every family) picked the right command first time throughout; this closes the two gaps that check couldn't reach: skill loading, and a lost copy answer.

SKILL.md is 15.2 KB of its 16 KiB budget. make check-skill-drift, make check-eval-patterns, make check-eval-harness and bin/ci pass.


Summary by cubic

The Basecamp skill only triggered on "move card", so moves or copies into another project could miss it. It now loads for those requests and tells agents to follow the new ID a move prints and never repeat a copy, with an eval case covering the lost-answer case.

  • Adds triggers for move to/copy to another project, copy todo, and copy card.
  • After a move into another container, sends writes to the printed new ID since the old one stops taking them; copies leave the original writable, so their ID matters only for checking progress.
  • Never repeats a copy — Basecamp files it in the background, so a still-running or unanswered copy may already exist; check it with basecamp filings show <id>.
  • Adds copy-still-running.yml, where the agent must run filings show and reject any todos copy when asked whether a copy finished.

Written for commit 6d56662. Summary will update on new commits.

View guided diff Turn on auto-fix

The skill only triggered on "move card", so a request to move or copy
something else into another project might not load it. And the rule that a
copy is never repeated lived only in the move and copy commands' own help,
which an agent retrying out of habit never reads. Add the triggers, a rule to
use the new ID and check a running copy with `filings show`, and an eval
case for it.
Copilot AI balanced review requested due to automatic review settings October 28, 2026 06:30
@robzolkos
robzolkos requested a review from a team as a code owner October 28, 2026 06:30
@github-actions github-actions Bot added the skills Agent skills label Oct 28, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

The regression fixture uses a nonexistent copy-ID field rather than the real response structure, and the ID guidance can redirect edits intended for the original item to its copy.

2 open issues
What changed in this PR

This PR expands the Basecamp agent skill’s move/copy triggers and adds guidance for following background copies without creating duplicates.

Changes:

  • Adds triggers for cross-project moves and copies, plus copying to-dos and cards.
  • Adds guidance on returned IDs and checking copy status rather than repeating the operation.
  • Adds an evaluation case requiring filings show and rejecting todos copy.

[!TIP]
If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

File Description
skills/​basecamp/​SKILL.md Expands triggers and documents move/copy follow-up rules.
skill-evals/​cases/​copy-still-running.yml Adds a status-check scenario that rejects repeating a copy.

Review effort: Balanced

Comment thread skill-evals/cases/copy-still-running.yml Outdated
Comment thread skills/basecamp/SKILL.md Outdated
…moves

A copy leaves the original writable, and a same-project reposition keeps its
ID, so the rule now sends writes to the new ID only after a move into another
container. The eval mock answers with destination_recording, as filings show
does.
Copilot AI balanced review requested due to automatic review settings October 28, 2026 06:34

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approved

No active review issues are reported for the Basecamp move/copy guidance and regression-case changes.

No open issues 🎉

2 resolved since last review

Review effort: Balanced

@robzolkos
robzolkos merged commit d1239a5 into main Oct 28, 2026
29 checks passed
@robzolkos
robzolkos deleted the skill-moves-copies branch October 28, 2026 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skills Agent skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants