Repository navigation
Remove the automated PR reviewer - #1230
Merged
Merged
Conversation
Remove the PR Review and Repo Analysis workflows, the @executablemd/code-review-agent workspace package, the .reviews entrypoints, policies and sensor profile, and the code-review agent specification, together with their tasks, workspace and lock facts, publication job, compile exclusion and documentation registration. The Oxlint gate keeps its shared policy; its contract moves from the retired sensor specification to specs/oxlint-policy-spec.md. General Plugin coverage that used the reviewer as its fixture now uses the existing declaring.mjs fixture, which gains one reserved registration, and the documentation witness reads the bundled Git Plugin instead.
Delete the root devDependency, the packages/code-review-agent workspace record and its alias entry. No other entry is re-resolved.
The provider shipped only with the removed code-review package. The JSDoc example and the missing-provider diagnostic now name OllamaProvider alone; the middleware contract is unchanged.
This was referenced Nov 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Why
Retire the automated PR reviewer at the maintainer's request. It repeatedly proposed deleting upstream patch context, lint directives and required provenance comments, creating review work without demonstrated correctness findings in the recent sample. Fixes #1226.
What changes
code-review-agentworkspace,.reviewsentrypoints and policies, and their dedicated specifications/tests.declaring.mjsfixture. Reviewer-only tests are deleted.specs/oxlint-policy-spec.md. Normal CI and the implementation workflow's independent Fable/Astra reviews remain.Both hosted workflows were disabled in GitHub before publication. There are no running or queued PR Review jobs. Historical reviews, credentials and recorded test timings are retained; deleted-file timing entries are ignored by the existing measurement contract.
Review guide
Start with the workflow/package removals and workspace locks. Then inspect
scripts/validate-documentation.ts, publication generation,declaring.mjs, and the plugin/compiled-plan test migrations. The source deletion accounts for most of the diff: 115 files, +309 / -11,410. No replacement reviewer or public API is added.How to verify it
Independent Fable review passed on exact final head
6e2dd3df3385e29bac309c6094c08889def9a5ec, after reviewing the retirement on parent99b22b60and then the two-line Sample diagnostic delta in the same reviewer session.Focused evidence:
Stampregistration non-reserved causes the expected precedence test failure; the fixture was restored.deno task setup,deno task build, the actual plugin/compiled-plan suites and npm CLI package suite pass on99b22b60.deno task verify:clean: PASS on exact final commit6e2dd3df, including clean setup, offline builds/release compilation, resolution and Deno/Node/Bun interference proof with unchanged tracked files, node_modules and lockfile.The first dependency/network, npm-cache permission and sandboxed process-inspection failures are preserved. Setup recovered with an owned cache and a fresh prepared layout. The final verification runs outside the sandbox so its existing process-join step can inspect its own children. No preflight, assertion, deadline or signing rule was bypassed. Required CI and current main health remain delivery gates.
Scope
The existing lint engine and its resolved optional peer remain pinned; this retirement performs no unrelated dependency upgrades. Shared runtime APIs keep their existing consumers and contracts. Original user changes, #1203 and the two planned pilots are untouched.
Scope confirmation