[ark-archastro]

Review on ArchCode

Problem and author intent

The public Shots skill still describes missing upload preferences as manual and tells users to install helper npm packages. ArchDev CLI 0.56.8 defaults uploads to automatic behind the local setting and server rollout flag, and includes the helper tools. Agents following the old guide can misinterpret settings and install unnecessary packages.

What changed

Describe the 0.56.8 default, retained manual opt-outs, and the difference between unsetting the preference and turning uploads off. Explain session-only Stop uploads and auth/rollout admission. Give reviewed unattended harnesses an explicit manual preference so they continue to keep captures local.

Replace obsolete helper-install instructions with bundled-distribution diagnostics, fixture-owned React/Ink dependencies, and supported Node-runtime requirements. Passive doctor output does not prove runtime compatibility. Regenerate the eight harness skill copies from the root guide.

Scope: documentation only. No CLI runtime changes. The canonical CLI fetches the public skill from main; version-tagged plugin consumers receive it through their normal plugin update/release path.

Verification

• Generator build/check: all 220 generated files match.
• TypeScript typecheck passed; generator/skill suite: 139 passed, one skipped.
• Released 0.56.8 shots doctor --json confirms bundled helper readiness and passive-runtime limitations.
• Semantic source: the released settings parser defaults absent upload preferences to Auto and preserves Manual/invalid-value opt-outs. The released real-process tests, specifically a_stop_hook_defaults_to_uploading_only_the_sessions_captures, manual_mode_uploads_nothing_on_stop, and automatic_upload_auth_and_offline_failures_are_silent_and_retryable, exercise the default and its gates through the actual CLI/worker and fixture HTTP. These passed in implementation PR 21977; generator tests alone are not evidence of runtime semantics.
• Independent review checked claims against the actual 0.56.8 source: its helper includes both CPU architectures and Linux libc variants; release fixtures cover Ink 5/React 18 and Ink 7/React 19.
• Automatic coverage: .github/workflows/plugins.yml runs generator typecheck, tests, and consistency checks on PRs. Existing bootstrap/installer workflows also run. No new tests or targets; documentation-only change, with no new end-to-end behavior to prove.

Risk and follow-up

Low runtime risk: instructions and generated copies only. Upload permission wording preserves explicit manual/off choices. This does not enable the server rollout flag. Shots remains staff/development-gated; the companion firstlanding changelog PR keeps its announcement as a draft until verified general availability.