[ark-archastro]

Review on ArchCode

Epic: Astroshots is a native archdev feature
Task: Update the Shots skill for current CLI defaults and sharing

Problem and author intent

The published Shots guide says capture is off by default and tells agents to enable it, although current CLI versions default the local setting on and check a server rollout flag. It also describes doctor as checking whether Shots is available. Agents can be given the wrong setup advice and may expect a locally captured screenshot to appear in the Company Room automatically.

What changed

Update the source skill and regenerate its eight harness plugin copies. Describe the local setting and rollout separately, doctor's offline limitation, manual versus automatic upload, and the possibility that a room post is unavailable. Retain the human-only settings rule and opt-outs. The guide now targets CLI 0.55.6 and later.

Scope: documentation and generated plugin copies. Risk: low; no CLI behavior changes.

Testing

• Plugin generator build, check, typecheck, and focused node --test test/generate.test.ts passed (45 tests); .github/workflows/plugins.yml runs the full generator test and check on PRs.
• Manual end-to-end archive proof: the installed Rust CLI 0.55.6 fetched a local HTTP archive containing the changed skill via archdev shots --skill; output matched the source byte for byte. This exercised CLI/archive/filesystem boundaries, not the public archive or a locally built CLI. The exact manual proof is not an automated regression test, and the requested on/off and missing-archive matrix has not been run against this changed guide.
• Full plugin suite: 128/129 passed locally; an unrelated nudge hook timing assertion failed (133–136 ms per invocation against a 100 ms budget). CI will run the unfiltered suite.
• Independent source review found and corrected wording about automatic room delivery and older CLI versions.

Follow-ups

The separate diagnostics task must update the guide after its implementation lands. Availability from the public archive requires this PR to merge; it has not been verified here.