Skip to content

Advance daily review workflows and simplify installation - #136

Draft
Paffin wants to merge 42 commits into
mainfrom
codex/roadmap-review-workflows
Draft

Paffin wants to merge 42 commits into
mainfrom
codex/roadmap-review-workflows

Conversation

@Paffin

@Paffin Paffin commented Oct 29, 2026 •

Copy link
Copy Markdown
Owner

This batch adds daily review workflows across Web, CLI, SDKs and MCP: requested reviewers, range/batched suggestions, private account drafts for reviews, general comments and existing-file edits, and pinned conflict resolution with correct rename source paths. File drafts retain their original revision and file metadata; saving remains separate from publishing a change request.

The Web review page puts code first, with compact secondary controls. Single-file editing now supports account save/restore and discovery of saved drafts after a file disappears from the current branch. Installation and English/Russian READMEs lead directly to the first working change. Native Windows packaging remains behind its qualification workflow.

Development metadata schema 12 requires a pre-upgrade backup. Published beta.7 remains schema 8. The batch also fixes structural sync losing independent incoming changes and adds bounded file/directory choices in Web and CLI.

Keep this PR in draft: the user explicitly requested accumulating changes on a separate branch, with no merge or release until instructed. Current progress and remaining roadmap scope are in dev/continuous-progress.md. Earlier review/comment slices have focused integration coverage; the latest file-draft slice received basic builds/type checks only, following the user's request to prioritize implementation over additional test loops.

codex and others added 30 commits October 29, 2026 21:55
Preview exact replacements across files and publish one changeset using an inspected-head update. Preserve line endings, BOM and mode; leave discussions open. Reuse existing APIs without schema changes.

Validation: 779 web tests on reconciled main, typecheck, lint, production build and four real-server desktop/phone tests. Earlier mutation pass killed all 34 sites before grouping extraction; behavior tests rerun after extraction.
Use an explicit range fence in existing discussion bodies, preserve source format, retain range drafts and reject overlapping batches. Show the original file and require author confirmation. No protocol or storage change.

Validation: 785 full web tests before boundary additions, 24 focused final tests, typecheck, six real-server desktop/phone scenarios. Mutation pass killed 66 of 67 candidates; remaining single-line parser default mutation produces the same public result.
Persist author-owned reviewer routing with inspected-set updates, idempotent retries and pre-pagination filtering. Share stream authorization and principal normalization across HTTP/gRPC, CLI, TypeScript/Python SDKs and MCP. Requests grant neither access nor approval. Schema 9 requires a pre-upgrade backup.

Validation: 295 storage/engine tests passed (one existing ignored test); HTTP/gRPC and scoped-token regressions; CLI local/remote and generated contracts; 102 MCP tests; 169 TypeScript and 192 Python SDK tests. Workspace clippy passes. Optional TS response field has an explicit generator regression.
Move secondary actions and context to a quiet rail, keep primary Review/Land actions and compact readiness visible, and preserve all operational flows. Add author-managed reviewers and a capability-guarded personal review filter with retained conflict drafts.

Validation: 796 web tests, typecheck/lint/build, real-server desktop/phone reviewer flow, final layout/header and accessibility checks, plus retained existing proposal regressions. Hallmark before/after review in light and dark. No main merge.
Keep original blobs, types and executable modes when choosing conflict versions. Compare the inspected branch before merge preparation and in the workspace commit transaction; preserve proposal-head CAS and session-bound cleanup. Expose the target in HTTP/gRPC, SDKs and MCP, and retry sync automatically only when pinned. No storage migration.

Validation: 150 engine tests, HTTP/gRPC pin regression, 812 web tests and six real-server browser cases, 107 MCP tests, 174 TypeScript and 198 Python SDK tests. Type/lint/build, workspace clippy, generator and documentation checks pass.
Keep unsubmitted notes isolated by actor with transactional save and clear preconditions. Add schema 10, HTTP/gRPC contracts and no-store responses without shared audit or event publication.

Validated: storage/integration suites, full server suite, migration backup fixture and workspace Clippy.
Add current-account read, versioned save and clear with safe retry semantics and capability checks. Keep private draft tools behind the review permission group.

Validated: TypeScript 177 tests and package checks, Python 202 tests and lint/types, MCP 108 tests.
Preserve tab-local notes while supporting cross-device account drafts. Require a choice for differing notes and guard post-review cleanup against newer edits.

Validated: 830 unit tests, typecheck, lint, build, four browser cases and 77 killed controller mutations.
Add remote review-draft get/save/clear, UTF-8 body files and capability guidance. Document cross-client draft behavior and the remaining roadmap work.

Validated: real-server CLI lifecycle, body-boundary unit coverage and documented JSON shapes.
Expose the merge engine's exact per-side source paths through core, API and clients. Read each Web preview at its original path and keep rename/delete choices tied to the inspected revisions.

Validated: full integration and Web suites, API/gRPC regressions in both rename directions, 12 browser cases, CLI/SDK/MCP checks and workspace Clippy.
Choose a published prerelease when no stable exists, preserve verification and explain exact launch/PATH steps. Rewrite both READMEs and getting-started around a verified beta.7 workflow instead of release chronology.

Validated: 54 release-tool tests, ShellCheck, five killed shell mutations, isolated published archive installation and full first-change smoke, README quickstart and documentation checks.
Package the supported Windows clients in a deterministic ZIP, verify downloads before per-user installation, and require native first-use smoke before future release publication. Keep current beta.7 availability explicit.

Validated: 15 offline PowerShell/package tests, 54 POSIX regression tests, workflow/docs checks and nine killed targeted mutations. Native Windows and PowerShell 5.1 remain gated in CI.
Keep the current fast promotion pipeline while moving Windows packaging into candidate preparation. Require its checksum-covered ZIP in candidate manifests, enforce PowerShell availability and preserve historical beta.7 release recognition. Persist the continuous backlog and audit status.

Validated: fast tooling suite, candidate11, request26, Windows15, CI policy18, workflow18 and skip-source gate.
Resolve virtual file/directory conflicts against the full candidate instead of an incomplete fallback. Refuse unsafe local sync and legacy pending snapshots before acknowledging lost changes, including forced snapshots.

Validated: 239 merge/integration tests, targeted structural and legacy regressions, clippy and independent review.
Preserve historical POSIX-only release recognition while detecting and repairing incomplete uploads after beta.7. Validate SemVer precedence rather than comparing version text.

Validated: 28 release-request tests, candidate/CI contracts and six killed ordering mutants.
Expose complete candidate path scopes for explicit file/directory choices, including synthetic rename destinations. Bound path copies, mark truncated scopes incomplete and preserve legacy optional metadata in clients.

Validated: 243 core tests, HTTP/gRPC scope regressions, client suites, bounded-scope mutations and independent review.
Offer explicit whole-source choices from complete pinned previews, preserving exact blobs and clean changes. Bound requests and paths, show removed destinations, and refuse unsupported or incomplete scopes.

Validated: 849 Web tests, twelve browser cases, types/build/lint, mutations and independent review.
Add explicit lifecycle commands and guarded resolution JSON. Preserve scope metadata without making conflict choices or moving proposals; retain binary/type/mode fields and reject ambiguous or duplicate inputs.

Validated: real-server structural lifecycle, stale-head behavior, CLI contracts and generated help; record OS facts in native CI.

This branch has not been deployed

No deployments
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.

2 participants