feat: Architecture Review action V1 - #1

Closed

brovatten wants to merge 24 commits into

mainfrom

feat/architecture-review-v1

Commits on May 25, 2026

feat: rewrite as Architecture Review action
brovatten
committed
5b6b478
chore: trigger CI
brovatten
committed
8e512d4
fix: remove timeout-minutes from composite step (not allowed)
brovatten
committed
7c10e9b
fix: bundle webview-ui dist (CodeBoarding-vscode is private)
brovatten
committed
8581a59
docs: update README to match shipped V1 (no vscode_ref, bundled dist)
brovatten
committed
5ffc288
chore: trigger CI with new baseline on main
brovatten
committed
cba8719
fix: use run_incremental directly against checked-out PR head
brovatten
committed
b695bc6
test: pass agent/parsing models matching local .env presets
brovatten
committed
ecce426
test: read AGENT_MODEL and PARSING_MODEL from repo secrets
brovatten
committed
83ac257
fix: pre-flight OpenRouter auth check (fail in 2s, not 9min)
brovatten
committed
6938cc7
fix: strip whitespace from OPENROUTER_API_KEY / models before use
brovatten
committed
03fdc9a
fix: also strip quotes/KEY= prefix from secret; report prefix sanity
brovatten
committed
560dfaf
fix: fall back to run_full on IncrementalCacheMissingError
brovatten
committed
df92624
fix: move Install Playwright AFTER seed step (it referenced base.found)
brovatten
committed
609c0c4
debug: dump DOM + console logs on no-nodes; screenshot regardless
brovatten
committed
a5f75c6
fix: serve analysis at /sample-analysis.json so browser-dev-mock loads it
brovatten
committed
f803c55
chore: drop unused analysis read (now served via http)
brovatten
committed
22990fb
fix: hide welcome / demo / help / side chrome before screenshot
brovatten
committed
b7e9865
feat: crop screenshot to node bounds + 80px padding
brovatten
committed
716fd6e
feat(comment): use branch name + smart pluralization in PR comment
brovatten
committed
8d0ec07
feat: append code-health warnings footer to PR comment
brovatten
committed
53272a5

Navigation Menu

Uh oh!

feat: Architecture Review action V1 - #1

feat: Architecture Review action V1#1
brovatten wants to merge 24 commits into
mainfrom
feat/architecture-review-v1

Commits on May 25, 2026

feat: rewrite as Architecture Review action

chore: trigger CI

fix: remove timeout-minutes from composite step (not allowed)

fix: bundle webview-ui dist (CodeBoarding-vscode is private)

docs: update README to match shipped V1 (no vscode_ref, bundled dist)

chore: trigger CI with new baseline on main

fix: use run_incremental directly against checked-out PR head

test: pass agent/parsing models matching local .env presets

test: read AGENT_MODEL and PARSING_MODEL from repo secrets

fix: pre-flight OpenRouter auth check (fail in 2s, not 9min)

fix: strip whitespace from OPENROUTER_API_KEY / models before use

fix: also strip quotes/KEY= prefix from secret; report prefix sanity

fix: fall back to run_full on IncrementalCacheMissingError

fix: move Install Playwright AFTER seed step (it referenced base.found)

debug: dump DOM + console logs on no-nodes; screenshot regardless

fix: serve analysis at /sample-analysis.json so browser-dev-mock loads it

chore: drop unused analysis read (now served via http)

fix: hide welcome / demo / help / side chrome before screenshot

feat: crop screenshot to node bounds + 80px padding

feat(comment): use branch name + smart pluralization in PR comment

feat: append code-health warnings footer to PR comment

Commits on May 26, 2026

feat(comment): expand warnings footer with examples + trace link

fix(warnings): better entity-name extraction for examples

chore: ignore webview-dist from CodeBoarding analysis