From ab4682d32c5b420775e01cf2bb3bf4d6720ace3e Mon Sep 17 00:00:00 2001 From: Garry Tan Date: Fri, 3 Apr 2026 07:58:58 -0700 Subject: [PATCH] feat: add DX Review row to review dashboard Adds plan-devex-review and devex-review schema entries to the review dashboard resolver and placeholder table in the preamble. All existing SKILL.md files regenerated to include the new DX Review row. Co-Authored-By: Claude Opus 4.6 (1M context) --- SKILL.md | 1 + autoplan/SKILL.md | 1 + benchmark/SKILL.md | 1 + browse/SKILL.md | 1 + canary/SKILL.md | 1 + checkpoint/SKILL.md | 1 + codex/SKILL.md | 6 ++++++ connect-chrome/SKILL.md | 1 + cso/SKILL.md | 1 + design-consultation/SKILL.md | 1 + design-html/SKILL.md | 1 + design-review/SKILL.md | 1 + design-shotgun/SKILL.md | 1 + document-release/SKILL.md | 1 + health/SKILL.md | 1 + investigate/SKILL.md | 1 + land-and-deploy/SKILL.md | 1 + learn/SKILL.md | 1 + office-hours/SKILL.md | 1 + plan-ceo-review/SKILL.md | 6 ++++++ plan-design-review/SKILL.md | 6 ++++++ plan-eng-review/SKILL.md | 6 ++++++ qa-only/SKILL.md | 1 + qa/SKILL.md | 1 + retro/SKILL.md | 1 + review/SKILL.md | 1 + scripts/resolvers/preamble.ts | 1 + scripts/resolvers/review.ts | 5 +++++ setup-browser-cookies/SKILL.md | 1 + setup-deploy/SKILL.md | 1 + ship/SKILL.md | 1 + 31 files changed, 55 insertions(+) diff --git a/SKILL.md b/SKILL.md index d63e8a83..1bb898aa 100644 --- a/SKILL.md +++ b/SKILL.md @@ -337,6 +337,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/autoplan/SKILL.md b/autoplan/SKILL.md index 9acb6d43..8ad8e37c 100644 --- a/autoplan/SKILL.md +++ b/autoplan/SKILL.md @@ -475,6 +475,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/benchmark/SKILL.md b/benchmark/SKILL.md index c32151e0..6f7dc894 100644 --- a/benchmark/SKILL.md +++ b/benchmark/SKILL.md @@ -340,6 +340,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/browse/SKILL.md b/browse/SKILL.md index f9af93e5..3931c7f8 100644 --- a/browse/SKILL.md +++ b/browse/SKILL.md @@ -339,6 +339,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/canary/SKILL.md b/canary/SKILL.md index b72a13eb..f1dca423 100644 --- a/canary/SKILL.md +++ b/canary/SKILL.md @@ -449,6 +449,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/checkpoint/SKILL.md b/checkpoint/SKILL.md index baa40e1a..96a41dc7 100644 --- a/checkpoint/SKILL.md +++ b/checkpoint/SKILL.md @@ -452,6 +452,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/codex/SKILL.md b/codex/SKILL.md index 4bcb5100..47d7521a 100644 --- a/codex/SKILL.md +++ b/codex/SKILL.md @@ -469,6 +469,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` @@ -686,6 +687,10 @@ Parse each JSONL entry. Each skill logs different fields: → Findings: "{issues_found} issues, {critical_gaps} critical gaps" - **plan-design-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`unresolved\`, \`decisions_made\`, \`commit\` → Findings: "score: {initial_score}/10 → {overall_score}/10, {decisions_made} decisions" +- **plan-devex-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`product_type\`, \`tthw_current\`, \`tthw_target\`, \`unresolved\`, \`commit\` + → Findings: "score: {initial_score}/10 → {overall_score}/10, TTHW: {tthw_current} → {tthw_target}" +- **devex-review**: \`status\`, \`overall_score\`, \`product_type\`, \`tthw_measured\`, \`dimensions_tested\`, \`dimensions_inferred\`, \`boomerang\`, \`commit\` + → Findings: "score: {overall_score}/10, TTHW: {tthw_measured}, {dimensions_tested} tested/{dimensions_inferred} inferred" - **codex-review**: \`status\`, \`gate\`, \`findings\`, \`findings_fixed\` → Findings: "{findings} findings, {findings_fixed}/{findings} fixed" @@ -704,6 +709,7 @@ Produce this markdown table: | Codex Review | \`/codex review\` | Independent 2nd opinion | {runs} | {status} | {findings} | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | {runs} | {status} | {findings} | | Design Review | \`/plan-design-review\` | UI/UX gaps | {runs} | {status} | {findings} | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | {runs} | {status} | {findings} | \`\`\` Below the table, add these lines (omit any that are empty/not applicable): diff --git a/connect-chrome/SKILL.md b/connect-chrome/SKILL.md index 1297374b..4e627931 100644 --- a/connect-chrome/SKILL.md +++ b/connect-chrome/SKILL.md @@ -466,6 +466,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/cso/SKILL.md b/cso/SKILL.md index 6540eac1..b1a1346d 100644 --- a/cso/SKILL.md +++ b/cso/SKILL.md @@ -454,6 +454,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/design-consultation/SKILL.md b/design-consultation/SKILL.md index 7052ba7d..05b8acc8 100644 --- a/design-consultation/SKILL.md +++ b/design-consultation/SKILL.md @@ -472,6 +472,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/design-html/SKILL.md b/design-html/SKILL.md index ec8142ed..1d98c345 100644 --- a/design-html/SKILL.md +++ b/design-html/SKILL.md @@ -456,6 +456,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/design-review/SKILL.md b/design-review/SKILL.md index b634d187..ec1eb940 100644 --- a/design-review/SKILL.md +++ b/design-review/SKILL.md @@ -472,6 +472,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/design-shotgun/SKILL.md b/design-shotgun/SKILL.md index 0f8f716e..8312a12b 100644 --- a/design-shotgun/SKILL.md +++ b/design-shotgun/SKILL.md @@ -451,6 +451,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/document-release/SKILL.md b/document-release/SKILL.md index e274cc28..7fb3228a 100644 --- a/document-release/SKILL.md +++ b/document-release/SKILL.md @@ -451,6 +451,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/health/SKILL.md b/health/SKILL.md index 68ade8e2..227b80bd 100644 --- a/health/SKILL.md +++ b/health/SKILL.md @@ -451,6 +451,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/investigate/SKILL.md b/investigate/SKILL.md index 3f57ded9..30f695d8 100644 --- a/investigate/SKILL.md +++ b/investigate/SKILL.md @@ -466,6 +466,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/land-and-deploy/SKILL.md b/land-and-deploy/SKILL.md index 4a13ca10..106007e8 100644 --- a/land-and-deploy/SKILL.md +++ b/land-and-deploy/SKILL.md @@ -466,6 +466,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/learn/SKILL.md b/learn/SKILL.md index e8f6055c..710330cf 100644 --- a/learn/SKILL.md +++ b/learn/SKILL.md @@ -451,6 +451,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/office-hours/SKILL.md b/office-hours/SKILL.md index 2fb28fad..6417cd25 100644 --- a/office-hours/SKILL.md +++ b/office-hours/SKILL.md @@ -476,6 +476,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/plan-ceo-review/SKILL.md b/plan-ceo-review/SKILL.md index 2e692ed3..2130cde2 100644 --- a/plan-ceo-review/SKILL.md +++ b/plan-ceo-review/SKILL.md @@ -472,6 +472,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` @@ -1613,6 +1614,10 @@ Parse each JSONL entry. Each skill logs different fields: → Findings: "{issues_found} issues, {critical_gaps} critical gaps" - **plan-design-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`unresolved\`, \`decisions_made\`, \`commit\` → Findings: "score: {initial_score}/10 → {overall_score}/10, {decisions_made} decisions" +- **plan-devex-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`product_type\`, \`tthw_current\`, \`tthw_target\`, \`unresolved\`, \`commit\` + → Findings: "score: {initial_score}/10 → {overall_score}/10, TTHW: {tthw_current} → {tthw_target}" +- **devex-review**: \`status\`, \`overall_score\`, \`product_type\`, \`tthw_measured\`, \`dimensions_tested\`, \`dimensions_inferred\`, \`boomerang\`, \`commit\` + → Findings: "score: {overall_score}/10, TTHW: {tthw_measured}, {dimensions_tested} tested/{dimensions_inferred} inferred" - **codex-review**: \`status\`, \`gate\`, \`findings\`, \`findings_fixed\` → Findings: "{findings} findings, {findings_fixed}/{findings} fixed" @@ -1631,6 +1636,7 @@ Produce this markdown table: | Codex Review | \`/codex review\` | Independent 2nd opinion | {runs} | {status} | {findings} | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | {runs} | {status} | {findings} | | Design Review | \`/plan-design-review\` | UI/UX gaps | {runs} | {status} | {findings} | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | {runs} | {status} | {findings} | \`\`\` Below the table, add these lines (omit any that are empty/not applicable): diff --git a/plan-design-review/SKILL.md b/plan-design-review/SKILL.md index 43c065a9..d4d16a64 100644 --- a/plan-design-review/SKILL.md +++ b/plan-design-review/SKILL.md @@ -470,6 +470,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` @@ -1347,6 +1348,10 @@ Parse each JSONL entry. Each skill logs different fields: → Findings: "{issues_found} issues, {critical_gaps} critical gaps" - **plan-design-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`unresolved\`, \`decisions_made\`, \`commit\` → Findings: "score: {initial_score}/10 → {overall_score}/10, {decisions_made} decisions" +- **plan-devex-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`product_type\`, \`tthw_current\`, \`tthw_target\`, \`unresolved\`, \`commit\` + → Findings: "score: {initial_score}/10 → {overall_score}/10, TTHW: {tthw_current} → {tthw_target}" +- **devex-review**: \`status\`, \`overall_score\`, \`product_type\`, \`tthw_measured\`, \`dimensions_tested\`, \`dimensions_inferred\`, \`boomerang\`, \`commit\` + → Findings: "score: {overall_score}/10, TTHW: {tthw_measured}, {dimensions_tested} tested/{dimensions_inferred} inferred" - **codex-review**: \`status\`, \`gate\`, \`findings\`, \`findings_fixed\` → Findings: "{findings} findings, {findings_fixed}/{findings} fixed" @@ -1365,6 +1370,7 @@ Produce this markdown table: | Codex Review | \`/codex review\` | Independent 2nd opinion | {runs} | {status} | {findings} | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | {runs} | {status} | {findings} | | Design Review | \`/plan-design-review\` | UI/UX gaps | {runs} | {status} | {findings} | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | {runs} | {status} | {findings} | \`\`\` Below the table, add these lines (omit any that are empty/not applicable): diff --git a/plan-eng-review/SKILL.md b/plan-eng-review/SKILL.md index e05d8342..9a0e03cf 100644 --- a/plan-eng-review/SKILL.md +++ b/plan-eng-review/SKILL.md @@ -472,6 +472,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` @@ -1259,6 +1260,10 @@ Parse each JSONL entry. Each skill logs different fields: → Findings: "{issues_found} issues, {critical_gaps} critical gaps" - **plan-design-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`unresolved\`, \`decisions_made\`, \`commit\` → Findings: "score: {initial_score}/10 → {overall_score}/10, {decisions_made} decisions" +- **plan-devex-review**: \`status\`, \`initial_score\`, \`overall_score\`, \`product_type\`, \`tthw_current\`, \`tthw_target\`, \`unresolved\`, \`commit\` + → Findings: "score: {initial_score}/10 → {overall_score}/10, TTHW: {tthw_current} → {tthw_target}" +- **devex-review**: \`status\`, \`overall_score\`, \`product_type\`, \`tthw_measured\`, \`dimensions_tested\`, \`dimensions_inferred\`, \`boomerang\`, \`commit\` + → Findings: "score: {overall_score}/10, TTHW: {tthw_measured}, {dimensions_tested} tested/{dimensions_inferred} inferred" - **codex-review**: \`status\`, \`gate\`, \`findings\`, \`findings_fixed\` → Findings: "{findings} findings, {findings_fixed}/{findings} fixed" @@ -1277,6 +1282,7 @@ Produce this markdown table: | Codex Review | \`/codex review\` | Independent 2nd opinion | {runs} | {status} | {findings} | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | {runs} | {status} | {findings} | | Design Review | \`/plan-design-review\` | UI/UX gaps | {runs} | {status} | {findings} | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | {runs} | {status} | {findings} | \`\`\` Below the table, add these lines (omit any that are empty/not applicable): diff --git a/qa-only/SKILL.md b/qa-only/SKILL.md index 336e5c20..6549bcf9 100644 --- a/qa-only/SKILL.md +++ b/qa-only/SKILL.md @@ -468,6 +468,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/qa/SKILL.md b/qa/SKILL.md index aba5f8f9..a526b3b7 100644 --- a/qa/SKILL.md +++ b/qa/SKILL.md @@ -474,6 +474,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/retro/SKILL.md b/retro/SKILL.md index bd99a762..ef86bc3a 100644 --- a/retro/SKILL.md +++ b/retro/SKILL.md @@ -449,6 +449,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/review/SKILL.md b/review/SKILL.md index eeb3c2ec..9d69790d 100644 --- a/review/SKILL.md +++ b/review/SKILL.md @@ -470,6 +470,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/scripts/resolvers/preamble.ts b/scripts/resolvers/preamble.ts index 49288500..7202d17a 100644 --- a/scripts/resolvers/preamble.ts +++ b/scripts/resolvers/preamble.ts @@ -508,6 +508,7 @@ Then write a \`## GSTACK REVIEW REPORT\` section to the end of the plan file: | Codex Review | \\\`/codex review\\\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \\\`/plan-eng-review\\\` | Architecture & tests (required) | 0 | — | — | | Design Review | \\\`/plan-design-review\\\` | UI/UX gaps | 0 | — | — | +| DX Review | \\\`/plan-devex-review\\\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \\\`/autoplan\\\` for full review pipeline, or individual reviews above. \\\`\\\`\\\` diff --git a/scripts/resolvers/review.ts b/scripts/resolvers/review.ts index de01698a..fe3a1318 100644 --- a/scripts/resolvers/review.ts +++ b/scripts/resolvers/review.ts @@ -94,6 +94,10 @@ Parse each JSONL entry. Each skill logs different fields: → Findings: "{issues_found} issues, {critical_gaps} critical gaps" - **plan-design-review**: \\\`status\\\`, \\\`initial_score\\\`, \\\`overall_score\\\`, \\\`unresolved\\\`, \\\`decisions_made\\\`, \\\`commit\\\` → Findings: "score: {initial_score}/10 → {overall_score}/10, {decisions_made} decisions" +- **plan-devex-review**: \\\`status\\\`, \\\`initial_score\\\`, \\\`overall_score\\\`, \\\`product_type\\\`, \\\`tthw_current\\\`, \\\`tthw_target\\\`, \\\`unresolved\\\`, \\\`commit\\\` + → Findings: "score: {initial_score}/10 → {overall_score}/10, TTHW: {tthw_current} → {tthw_target}" +- **devex-review**: \\\`status\\\`, \\\`overall_score\\\`, \\\`product_type\\\`, \\\`tthw_measured\\\`, \\\`dimensions_tested\\\`, \\\`dimensions_inferred\\\`, \\\`boomerang\\\`, \\\`commit\\\` + → Findings: "score: {overall_score}/10, TTHW: {tthw_measured}, {dimensions_tested} tested/{dimensions_inferred} inferred" - **codex-review**: \\\`status\\\`, \\\`gate\\\`, \\\`findings\\\`, \\\`findings_fixed\\\` → Findings: "{findings} findings, {findings_fixed}/{findings} fixed" @@ -112,6 +116,7 @@ Produce this markdown table: | Codex Review | \\\`/codex review\\\` | Independent 2nd opinion | {runs} | {status} | {findings} | | Eng Review | \\\`/plan-eng-review\\\` | Architecture & tests (required) | {runs} | {status} | {findings} | | Design Review | \\\`/plan-design-review\\\` | UI/UX gaps | {runs} | {status} | {findings} | +| DX Review | \\\`/plan-devex-review\\\` | Developer experience gaps | {runs} | {status} | {findings} | \\\`\\\`\\\` Below the table, add these lines (omit any that are empty/not applicable): diff --git a/setup-browser-cookies/SKILL.md b/setup-browser-cookies/SKILL.md index 91828dac..dc8dd9fa 100644 --- a/setup-browser-cookies/SKILL.md +++ b/setup-browser-cookies/SKILL.md @@ -336,6 +336,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/setup-deploy/SKILL.md b/setup-deploy/SKILL.md index a186aa33..f09b6404 100644 --- a/setup-deploy/SKILL.md +++ b/setup-deploy/SKILL.md @@ -452,6 +452,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\` diff --git a/ship/SKILL.md b/ship/SKILL.md index 92524582..4886ea03 100644 --- a/ship/SKILL.md +++ b/ship/SKILL.md @@ -471,6 +471,7 @@ Then write a `## GSTACK REVIEW REPORT` section to the end of the plan file: | Codex Review | \`/codex review\` | Independent 2nd opinion | 0 | — | — | | Eng Review | \`/plan-eng-review\` | Architecture & tests (required) | 0 | — | — | | Design Review | \`/plan-design-review\` | UI/UX gaps | 0 | — | — | +| DX Review | \`/plan-devex-review\` | Developer experience gaps | 0 | — | — | **VERDICT:** NO REVIEWS YET — run \`/autoplan\` for full review pipeline, or individual reviews above. \`\`\`