Add --position flag to cards move

[jeremy]

Summary

• Adds --position (alias --pos) flag to basecamp cards move for placing a card at a specific position within the target column (1-indexed)
• Positioned moves use the CardColumns().Move endpoint (POST /card_tables/{id}/moves.json) with card-as-source, column-as-target; unpositioned moves use the existing Cards().Move path unchanged
• When --position is used with a numeric --to column ID, the card table is auto-resolved (single table) or an ambiguous error guides the user to provide --card-table

Test plan

• TestCardsMovePositionPayload — transport-backed test verifies CLI sends {source_id, target_id, position} to /card_tables/{id}/moves.json
• TestCardsMoveWithoutPositionUsesCardsMove — transport-backed test verifies old path /card_tables/cards/{id}/moves.json with {column_id} when --position absent
• TestCardsMovePositionRejectsNonPositive — validates --position -1
• TestCardsMovePositionRejectsZero — validates --position 0
• TestCardsMovePositionNumericToMultiTableAmbiguous — multi-table + no --card-table returns ambiguous error
• Smoke e2e test: cards move with position moves card to position
• bin/ci passes (all checks including skill drift)
• Manual validation against live BC3 API that the moves endpoint accepts card-as-source, column-as-target (transport tests prove CLI contract, not API acceptance)

---

Summary by cubic

Adds a --position flag to basecamp cards move to place a card at a specific position in the target column. When omitted, existing move behavior is unchanged.

• New Features
  • --position (alias --pos) sets 1-indexed position in the target column.
  • Positioned moves use CardColumns().Move (POST /card_tables/{id}/moves.json) with {source_id, target_id, position}; otherwise uses existing Cards().Move.
  • For positioned moves with numeric --to, auto-resolves the card table; errors if multiple tables and no --card-table.
  • Validates --position > 0 and includes position in JSON output and CLI summary.
  • Updated .surface, e2e smoke test, and skills/basecamp/SKILL.md.

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

[cubic-dev-ai]

No issues found across 6 files

[Copilot]

Pull request overview

Adds optional positional placement for basecamp cards move so users can move a card into a specific index within the target column, while preserving the existing move behavior when no position is requested.

Changes:

• Add --position / --pos to basecamp cards move and route positioned moves through the card table moves endpoint.
• Extend unit tests with transport-backed coverage for positioned vs unpositioned move request contracts and for multi-table ambiguity.
• Update skill docs, smoke tests, allowlist, and surface snapshot to reflect the new flags and example usage.

Reviewed changes

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

Show a summary per file

File	Description
skills/basecamp/SKILL.md	Documents cards move with optional --position and adds examples.
internal/commands/cards.go	Implements --position/--pos and switches endpoint/payload when set.
internal/commands/cards_test.go	Adds transport-backed tests for positioned payload, legacy path, validation, and ambiguity.
e2e/smoke/smoke_cards_write.bats	Adds a smoke test covering cards move --position.
e2e/smoke/.qa-allowlist	Allowlists the new smoke test as unverifiable in some environments.
.surface	Updates CLI surface snapshot to include the new flags.

---

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