Skip to content

feat(authoring): report compatibility and static readiness - #158

Merged
777genius merged 3 commits into
mainfrom
feat/authoring-readiness-checkpoint
Sep 6, 2026
Merged

feat(authoring): report compatibility and static readiness#158
777genius merged 3 commits into
mainfrom
feat/authoring-readiness-checkpoint

Conversation

@777genius

@777genius 777genius commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Adds explicit client compatibility, engine capabilities and static source-project doctor commands through both internal authoring entrypoints. Compatibility uses the existing client registry without installed clients; doctor uses captured metadata and leaves executable versions, dependency consistency, runtime and OAuth unproven.

Follows merged #157. Based on main at e92f409. This is Phase 4, not the Phase 6 public cutover; default v1 builds stay unchanged.

Validation at 783ab8d (tree identical to tested 3cc9dcc):

  • Authoring, factory and planner suites passed after rebasing onto the first PR fixes.
  • Both actual native binaries passed compatibility/capabilities/doctor E2E. Added commands ran under a calibrated Linux Landlock/seccomp fixture that denies network, child execution, HOME file access and source writes while allowing private reader scratch. This proves denial, not a trace of every attempted access.
  • Independent review found one P2 involving empty MCP server names. The correction separates item identities from document-wide invalidity; baseline-failing regressions now pass through compat and inspect at both mounts. Post-commit planner/commands and confined native checks passed. Earlier focused/race/vet and CLI regression checks also passed.
  • Final required CI, CodeQL, security checks, documentation checks and Linux/Windows installer smoke passed on this head. The draft CodeRabbit check is a skip, not independent review evidence.

The bounded diff includes the pure compatibility API and tests. Native platform support, Skills authoring and release packaging remain separate gates. Revert this PR independently to remove these read-only commands while retaining #157.

@coderabbitai

coderabbitai Bot commented Sep 6, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@777genius
777genius force-pushed the feat/authoring-readiness-checkpoint branch from 3cc9dcc to 783ab8d Compare September 6, 2026 05:37
@777genius
777genius changed the base branch from feat/standard-authoring-core to main September 6, 2026 05:37
@777genius 777genius closed this Sep 6, 2026
@777genius 777genius reopened this Sep 6, 2026
@777genius
777genius marked this pull request as ready for review September 6, 2026 05:50
@777genius
777genius merged commit 1b77dc8 into main Sep 6, 2026
16 checks passed
777genius added a commit that referenced this pull request Sep 6, 2026
)

Add metadata-first NTFS acquisition, trusted scratch alias handling and native offline authoring gates for both entrypoints. Preserve default v1 behavior and read-only APFS restriction.

Validated PR head: f0e5995
Validated tree: 845e794
Native evidence: https://github.com/777genius/universal-agent-plugins/actions/runs/34022971411
All PR CI jobs passed. Optional CodeRabbit review was pending; branch rules do not require it. Independent hosted review is complete. Native Windows: 826 test/subtest passes, no required skips. Native Linux: 801 passes, one previously classified optional Unix-socket fixture skip. Independent gpt-6-astra xhigh review accepts this internal checkpoint; both identified test P2 findings are corrected.

Residual risk: a prior isolated Windows scratch_unavailable failure has no established cause. Current public and staged cleanup tests pass, and acquisition fails closed; no causal fix or zero-recurrence claim is made. Public v2 and writable macOS release acceptance remain outside this checkpoint.

Related PRs: #157, #158, #159, #161.
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