Tell agents the truth about card column moves

[jeremy]

skills/basecamp/SKILL.md claimed:

Limitation: Basecamp does not track when cards are moved between columns.

It does. basecamp events <card_id> records an adopted event on every column
move, and a card crossing into or out of a Done column pairs that with
completed/uncompleted. This file installs to ~/.agents/skills/, so the
claim was actively misinforming agents — telling them to give up on a question
the CLI already answers.

Two gaps let it survive:

• basecamp events appeared nowhere in SKILL.md — zero grep hits. An agent
  reading the skill never learned the command exists, so nothing contradicted
  the limitation.
• internal/commands/events.go omitted adopted from the common actions in
  its Long help, so --help didn't fill the gap either.

All three are fixed here: the false limitation becomes the events recipe,
events gets its own section next to Timeline plus a Quick Reference row, and
adopted joins the command's own list of common actions.

.surface is unchanged — it doesn't capture Long help text — so
make check-skill-drift and make check-surface both pass, and bin/ci is
green.

Closes #595

---

Summary by cubic

Reflects that Basecamp cards record column moves via adopted events and documents how to use basecamp events to answer when a card moved or was completed, addressing #595. Adds adopted to events --help, replaces the false limitation in skills/basecamp/SKILL.md, and adds an Events section plus a Quick Reference entry.

^{Written for commit 068f3f6. Summary will update on new commits.}

[Copilot]

Pull request overview

This PR corrects a factual error in the agent skill documentation. Previously, skills/basecamp/SKILL.md claimed Basecamp does not track when cards move between columns. In reality, basecamp events <card_id> records an adopted event on every column move (paired with completed/uncompleted when crossing a Done column boundary). Since this skill file installs to ~/.agents/skills/, the false claim actively steered agents away from a working query. The PR replaces the false limitation with an accurate recipe, adds a dedicated Events section plus a Quick Reference row, and adds adopted to the events --help action list. It closes #595.

Changes:

• Replace the incorrect "Basecamp does not track column moves" limitation with guidance to use basecamp events for column-move/completion history.
• Add an "Events (change history)" section and a Quick Reference row documenting basecamp events usage and flags.
• Add adopted to the common event actions in internal/commands/events.go Long help, and note that completed/uncompleted applies to cards too.

I verified the documented flags (--limit, --all, --page, --json) against internal/commands/events.go and .surface, confirmed the --page "only 1" behavior matches the implementation (lines 50–52), and confirmed the internal link #events-change-history correctly resolves to the new unique ### Events (change history) header.

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.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
skills/basecamp/SKILL.md	Replaces the false column-move limitation with an accurate basecamp events recipe, adds an Events section, and adds a Quick Reference row.
internal/commands/events.go	Adds adopted to the common event actions in --help and clarifies that completed/uncompleted applies to cards as well as todos.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.