Skip to content

ci: add deterministic Isobar maps to PR review signals - #655

Open
eipasteur wants to merge 3 commits into
mainfrom
feat/pr-isobar-map
Open

eipasteur wants to merge 3 commits into
mainfrom
feat/pr-isobar-map

Conversation

@eipasteur

@eipasteur eipasteur commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Adds a deterministic Isobar map to the existing PR quality comment, following #634. Quality and map sections are independently collapsible; the existing grading and 10-row/all-purple rule stay unchanged. One image updates in place as commits arrive.

The static reporter reads PR Git objects without executing contributor code. Existing fork test approvals remain unchanged; activation and the external-fork smoke test require the workflow to be merged into main.

Validation: tooling tests, lint, formatting, secretlint, docs build and three historical PR previews.

Comment thread scripts/pr-impact/reader-guard.mjs Fixed
Comment thread scripts/pr-impact/render.mjs Fixed
@jeromevdl jeromevdl added area/ci GitHub Actions, hooks, lint, release tooling, E2E harness size/XL 3000+ changed lines (lockfiles excluded), consider splitting labels Oct 30, 2026
@jeromevdl jeromevdl changed the title feat: add deterministic Isobar maps to PR review signals ci: add deterministic Isobar maps to PR review signals Oct 30, 2026

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

area/ci GitHub Actions, hooks, lint, release tooling, E2E harness size/XL 3000+ changed lines (lockfiles excluded), consider splitting

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants