Skip to content

Fix launcher quickstart and align public product docs - #1096

Merged
abrichr merged 4 commits into
mainfrom
codex/quickstart-lifecycle
Aug 19, 2026
Merged

Fix launcher quickstart and align public product docs#1096
abrichr merged 4 commits into
mainfrom
codex/quickstart-lifecycle

Conversation

@abrichr

@abrichr abrichr commented Aug 18, 2026

Copy link
Copy Markdown
Member

Summary

  • delegate every openadapt flow ... argument to the installed Flow parser so current and future commands, options, help, and exit codes cannot drift behind the engine
  • make RDP deploy/doctor checks fail when the transport extra is absent
  • replace the false-success openadapt capture stop no-op with an explicit non-success result
  • raise the Capture compatibility floor to 1.2.0 and let Flow own the browser dependency contract
  • add a bounded weekly clean-wheel lifecycle for the public openadapt quickstart command
  • replace stale training-era package, architecture, installation, CLI, Capture, and permissions pages with the current governed-compiler product contract
  • state the supported Beta role of openadapt-capture in the package docstring, README, and package map

Root cause

The launcher duplicated selected Flow commands as Click wrappers. Those wrappers rejected valid current Flow options and hid new commands until the launcher changed. Several historical pages and the package docstring still described Capture as experimental. The raw Capture stop command also printed a success-shaped message without any cross-process control mechanism.

Capture currently exposes only in-process Recorder.stop() and internal stop events. It has no authenticated cross-process control channel. OpenAdaptAI/openadapt-capture#77 records the target owner-only IPC contract and acceptance tests. The launcher failure is the truthful interim behavior, not completion of that target feature.

Validation

  • 150 launcher tests passed; 6 platform-specific tests skipped
  • Ruff lint and format passed
  • source-availability boundary passed
  • wheel and source archive built and passed the release-artifact verifier
  • clean installed launcher + Flow lifecycle passed: Standard VERIFIED, 2/2 independent effects, 0 model calls, lint passed, and uninstall was verified

The docs rewrite introduced a Beta lifecycle for openadapt-capture in the
package map, the Capture package page, the README component list, and the
package docstring. The canonical organization registry
(OpenAdaptAI/.github REPOSITORY_LIFECYCLE.md) still records Capture as
Experimental, and no acceptance evidence exists yet.

Keep the stale-fact corrections in this PR and state the lifecycle the
registry actually records. The privacy extra keeps its experimental
qualifier for the same reason.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@abrichr
abrichr marked this pull request as ready for review August 19, 2026 22:02
@abrichr
abrichr merged commit 770e40f into main Aug 19, 2026
12 checks passed
@abrichr
abrichr deleted the codex/quickstart-lifecycle branch August 19, 2026 22:02
abrichr added a commit that referenced this pull request Aug 19, 2026
…1098)

The Notify docs job fails on main with "Request body length does not
match content-length header". peter-evans/repository-dispatch v4.0.1
sends a mismatched Content-Length for this payload on GitHub-hosted
runners, so the docs sync dispatch never reaches openadapt-ops.

This is not new breakage from the merge that surfaced it. #1053 already
pinned the action back to v3 for exactly this reason on 2026-07-25.
Dependabot PR #1079 bumped it to v4 again on 2026-08-12, and the job has
been latently broken since; the failure only appears on a push that
touches README.md or CHANGELOG.md, and the first such push after the
bump was #1096.

- pin the action to the v3.0.0 commit instead of a floating major tag
- ignore 4.x for this action in dependabot.yml so the bump cannot silently
  reintroduce the regression a third time
- address the dispatch to OpenAdaptAI/openadapt-ops directly rather than
  relying on the rename redirect from openadapt-maintenance

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant