dojo.md 0.1.0 → 0.2.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/courses/GENERATION_LOG.md +27 -0
- package/courses/api-error-handling/course.yaml +16 -0
- package/courses/api-error-handling/scenarios/level-1/error-response-format.yaml +131 -0
- package/courses/api-error-handling/scenarios/level-1/http-status-codes-basics.yaml +90 -0
- package/courses/api-error-handling/scenarios/level-1/rate-limiting-basics.yaml +135 -0
- package/courses/api-error-handling/scenarios/level-1/request-validation-errors.yaml +208 -0
- package/courses/api-error-handling/scenarios/level-2/circuit-breaker-pattern.yaml +189 -0
- package/courses/api-error-handling/scenarios/level-2/idempotency-retry-logic.yaml +159 -0
- package/courses/api-error-handling/scenarios/level-2/rfc-7807-problem-details.yaml +178 -0
- package/courses/api-error-handling/scenarios/level-2/webhook-error-handling.yaml +211 -0
- package/courses/api-error-handling/scenarios/level-3/distributed-tracing-errors.yaml +275 -0
- package/courses/github-actions-cicd/course.yaml +10 -0
- package/courses/github-actions-cicd/scenarios/level-1/actions-and-runners.yaml +58 -0
- package/courses/github-actions-cicd/scenarios/level-1/basic-workflow-syntax.yaml +52 -0
- package/courses/github-actions-cicd/scenarios/level-1/branch-protection-checks.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-1/environment-variables-secrets.yaml +65 -0
- package/courses/github-actions-cicd/scenarios/level-1/first-cicd-shift.yaml +62 -0
- package/courses/github-actions-cicd/scenarios/level-1/job-dependencies-outputs.yaml +62 -0
- package/courses/github-actions-cicd/scenarios/level-1/simple-ci-pipeline.yaml +57 -0
- package/courses/github-actions-cicd/scenarios/level-1/workflow-debugging.yaml +90 -0
- package/courses/github-actions-cicd/scenarios/level-1/workflow-status-notifications.yaml +59 -0
- package/courses/github-actions-cicd/scenarios/level-1/workflow-triggers.yaml +56 -0
- package/courses/github-actions-cicd/scenarios/level-2/concurrency-control.yaml +58 -0
- package/courses/github-actions-cicd/scenarios/level-2/conditional-execution.yaml +60 -0
- package/courses/github-actions-cicd/scenarios/level-2/custom-actions-development.yaml +55 -0
- package/courses/github-actions-cicd/scenarios/level-2/dependency-caching.yaml +58 -0
- package/courses/github-actions-cicd/scenarios/level-2/deployment-workflows.yaml +61 -0
- package/courses/github-actions-cicd/scenarios/level-2/github-packages-publishing.yaml +59 -0
- package/courses/github-actions-cicd/scenarios/level-2/intermediate-cicd-shift.yaml +68 -0
- package/courses/github-actions-cicd/scenarios/level-2/matrix-builds.yaml +59 -0
- package/courses/github-actions-cicd/scenarios/level-2/reusable-workflows.yaml +61 -0
- package/courses/github-actions-cicd/scenarios/level-2/workflow-cost-optimization.yaml +61 -0
- package/courses/github-actions-cicd/scenarios/level-3/advanced-cicd-shift.yaml +64 -0
- package/courses/github-actions-cicd/scenarios/level-3/compliance-automation.yaml +68 -0
- package/courses/github-actions-cicd/scenarios/level-3/docker-action-development.yaml +65 -0
- package/courses/github-actions-cicd/scenarios/level-3/github-environments.yaml +65 -0
- package/courses/github-actions-cicd/scenarios/level-3/monorepo-ci.yaml +68 -0
- package/courses/github-actions-cicd/scenarios/level-3/oidc-cloud-deployments.yaml +55 -0
- package/courses/github-actions-cicd/scenarios/level-3/release-automation.yaml +61 -0
- package/courses/github-actions-cicd/scenarios/level-3/security-hardening.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-3/self-hosted-runners.yaml +60 -0
- package/courses/github-actions-cicd/scenarios/level-3/workflow-optimization.yaml +59 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-data-architecture.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-economics-roi.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-executive-communication.yaml +58 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-incident-response.yaml +60 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-org-design.yaml +59 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-platform-architecture.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-training-program.yaml +65 -0
- package/courses/github-actions-cicd/scenarios/level-4/cicd-vendor-evaluation.yaml +59 -0
- package/courses/github-actions-cicd/scenarios/level-4/enterprise-cicd-governance.yaml +55 -0
- package/courses/github-actions-cicd/scenarios/level-4/expert-cicd-shift.yaml +60 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-ai-future.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-behavioral-science.yaml +70 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-board-strategy.yaml +56 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-consulting-engagement.yaml +61 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-industry-benchmarks.yaml +63 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-ma-integration.yaml +73 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-product-development.yaml +68 -0
- package/courses/github-actions-cicd/scenarios/level-5/cicd-regulatory-landscape.yaml +72 -0
- package/courses/github-actions-cicd/scenarios/level-5/comprehensive-cicd-system.yaml +66 -0
- package/courses/github-actions-cicd/scenarios/level-5/master-cicd-shift.yaml +76 -0
- package/courses/github-pr-review/scenarios/level-2/api-change-review.yaml +82 -0
- package/courses/github-pr-review/scenarios/level-2/automated-review-tooling.yaml +53 -0
- package/courses/github-pr-review/scenarios/level-2/cross-team-review.yaml +61 -0
- package/courses/github-pr-review/scenarios/level-2/intermediate-review-shift.yaml +66 -0
- package/courses/github-pr-review/scenarios/level-2/performance-review-patterns.yaml +99 -0
- package/courses/github-pr-review/scenarios/level-2/review-disagreement-resolution.yaml +64 -0
- package/courses/github-pr-review/scenarios/level-2/review-metrics-analysis.yaml +63 -0
- package/courses/github-pr-review/scenarios/level-2/review-turnaround-sla.yaml +54 -0
- package/courses/github-pr-review/scenarios/level-2/stacked-pr-review.yaml +65 -0
- package/courses/github-pr-review/scenarios/level-3/advanced-review-shift.yaml +65 -0
- package/courses/github-pr-review/scenarios/level-3/ai-powered-review.yaml +58 -0
- package/courses/github-pr-review/scenarios/level-3/compliance-review-process.yaml +64 -0
- package/courses/github-pr-review/scenarios/level-3/cross-functional-review.yaml +60 -0
- package/courses/github-pr-review/scenarios/level-3/incident-driven-review.yaml +63 -0
- package/courses/github-pr-review/scenarios/level-3/large-scale-review-operations.yaml +55 -0
- package/courses/github-pr-review/scenarios/level-3/monorepo-review-process.yaml +68 -0
- package/courses/github-pr-review/scenarios/level-3/review-automation-platform.yaml +61 -0
- package/courses/github-pr-review/scenarios/level-3/review-culture-design.yaml +62 -0
- package/courses/github-pr-review/scenarios/level-3/review-data-pipeline.yaml +62 -0
- package/courses/github-pr-review/scenarios/level-4/enterprise-review-operations.yaml +61 -0
- package/courses/github-pr-review/scenarios/level-4/expert-review-shift.yaml +62 -0
- package/courses/github-pr-review/scenarios/level-4/review-data-architecture.yaml +69 -0
- package/courses/github-pr-review/scenarios/level-4/review-economics-roi.yaml +63 -0
- package/courses/github-pr-review/scenarios/level-4/review-executive-communication.yaml +61 -0
- package/courses/github-pr-review/scenarios/level-4/review-incident-postmortem.yaml +69 -0
- package/courses/github-pr-review/scenarios/level-4/review-org-design.yaml +62 -0
- package/courses/github-pr-review/scenarios/level-4/review-platform-architecture.yaml +64 -0
- package/courses/github-pr-review/scenarios/level-4/review-training-program.yaml +66 -0
- package/courses/github-pr-review/scenarios/level-4/review-vendor-evaluation.yaml +76 -0
- package/courses/github-pr-review/scenarios/level-5/comprehensive-review-system.yaml +68 -0
- package/courses/github-pr-review/scenarios/level-5/master-review-shift.yaml +73 -0
- package/courses/github-pr-review/scenarios/level-5/review-ai-future.yaml +69 -0
- package/courses/github-pr-review/scenarios/level-5/review-behavioral-science.yaml +66 -0
- package/courses/github-pr-review/scenarios/level-5/review-board-strategy.yaml +62 -0
- package/courses/github-pr-review/scenarios/level-5/review-consulting-engagement.yaml +62 -0
- package/courses/github-pr-review/scenarios/level-5/review-devtools-product.yaml +71 -0
- package/courses/github-pr-review/scenarios/level-5/review-industry-benchmarks.yaml +64 -0
- package/courses/github-pr-review/scenarios/level-5/review-ma-integration.yaml +76 -0
- package/courses/github-pr-review/scenarios/level-5/review-regulatory-landscape.yaml +78 -0
- package/courses/postgresql-query-optimization/course.yaml +11 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/explain-analyze-basics.yaml +80 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/first-optimization-shift.yaml +77 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/index-fundamentals.yaml +76 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/join-basics.yaml +73 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/n-plus-one-queries.yaml +62 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/query-rewriting-basics.yaml +69 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/select-star-problems.yaml +69 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/slow-query-diagnosis.yaml +63 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/vacuum-and-statistics.yaml +62 -0
- package/courses/postgresql-query-optimization/scenarios/level-1/where-clause-optimization.yaml +74 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/autovacuum-tuning.yaml +76 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/composite-index-design.yaml +81 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/covering-indexes.yaml +74 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/cte-optimization.yaml +83 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/intermediate-optimization-shift.yaml +66 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/join-optimization.yaml +72 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/partial-and-expression-indexes.yaml +75 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/query-planner-settings.yaml +62 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/subquery-optimization.yaml +67 -0
- package/courses/postgresql-query-optimization/scenarios/level-2/window-function-optimization.yaml +63 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/advanced-optimization-shift.yaml +71 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/connection-pooling.yaml +60 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/full-text-search-optimization.yaml +66 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/jsonb-optimization.yaml +88 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/lock-contention-analysis.yaml +80 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/materialized-view-optimization.yaml +73 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/parallel-query-execution.yaml +74 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/partitioning-strategies.yaml +71 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/specialized-index-types.yaml +67 -0
- package/courses/postgresql-query-optimization/scenarios/level-3/write-optimization.yaml +65 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/data-architecture-analytics.yaml +64 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/database-executive-communication.yaml +64 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/database-migration-planning.yaml +57 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/enterprise-database-governance.yaml +52 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/expert-optimization-shift.yaml +73 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/high-availability-architecture.yaml +62 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/optimizer-internals.yaml +69 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/performance-sla-design.yaml +58 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/read-replica-optimization.yaml +62 -0
- package/courses/postgresql-query-optimization/scenarios/level-4/vendor-evaluation.yaml +73 -0
- package/courses/rest-api-error-handling/course.yaml +11 -0
- package/courses/rest-api-error-handling/scenarios/level-1/authentication-errors.yaml +71 -0
- package/courses/rest-api-error-handling/scenarios/level-1/content-negotiation-errors.yaml +63 -0
- package/courses/rest-api-error-handling/scenarios/level-1/error-logging-basics.yaml +63 -0
- package/courses/rest-api-error-handling/scenarios/level-1/error-response-format.yaml +58 -0
- package/courses/rest-api-error-handling/scenarios/level-1/first-error-handling-shift.yaml +67 -0
- package/courses/rest-api-error-handling/scenarios/level-1/http-status-codes.yaml +46 -0
- package/courses/rest-api-error-handling/scenarios/level-1/not-found-errors.yaml +52 -0
- package/courses/rest-api-error-handling/scenarios/level-1/rate-limiting-errors.yaml +56 -0
- package/courses/rest-api-error-handling/scenarios/level-1/request-validation-errors.yaml +59 -0
- package/courses/rest-api-error-handling/scenarios/level-1/server-error-handling.yaml +55 -0
- package/courses/rest-api-error-handling/scenarios/level-2/api-versioning-errors.yaml +66 -0
- package/courses/rest-api-error-handling/scenarios/level-2/batch-request-errors.yaml +61 -0
- package/courses/rest-api-error-handling/scenarios/level-2/circuit-breaker-pattern.yaml +52 -0
- package/courses/rest-api-error-handling/scenarios/level-2/error-code-taxonomy.yaml +62 -0
- package/courses/rest-api-error-handling/scenarios/level-2/error-monitoring-alerting.yaml +53 -0
- package/courses/rest-api-error-handling/scenarios/level-2/intermediate-error-shift.yaml +69 -0
- package/courses/rest-api-error-handling/scenarios/level-2/pagination-errors.yaml +66 -0
- package/courses/rest-api-error-handling/scenarios/level-2/retry-and-idempotency.yaml +60 -0
- package/courses/rest-api-error-handling/scenarios/level-2/rfc7807-problem-details.yaml +60 -0
- package/courses/rest-api-error-handling/scenarios/level-2/webhook-error-handling.yaml +55 -0
- package/courses/rest-api-error-handling/scenarios/level-3/advanced-error-shift.yaml +72 -0
- package/courses/rest-api-error-handling/scenarios/level-3/api-gateway-errors.yaml +71 -0
- package/courses/rest-api-error-handling/scenarios/level-3/async-api-errors.yaml +67 -0
- package/courses/rest-api-error-handling/scenarios/level-3/caching-error-scenarios.yaml +65 -0
- package/courses/rest-api-error-handling/scenarios/level-3/chaos-engineering-apis.yaml +62 -0
- package/courses/rest-api-error-handling/scenarios/level-3/database-error-handling.yaml +79 -0
- package/courses/rest-api-error-handling/scenarios/level-3/distributed-error-propagation.yaml +63 -0
- package/courses/rest-api-error-handling/scenarios/level-3/error-budgets-sre.yaml +61 -0
- package/courses/rest-api-error-handling/scenarios/level-3/error-correlation.yaml +58 -0
- package/courses/rest-api-error-handling/scenarios/level-3/graphql-vs-rest-errors.yaml +73 -0
- package/courses/rest-api-error-handling/scenarios/level-4/compliance-error-handling.yaml +65 -0
- package/courses/rest-api-error-handling/scenarios/level-4/enterprise-error-governance.yaml +62 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-analytics-platform.yaml +65 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-cost-optimization.yaml +63 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-executive-communication.yaml +60 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-handling-architecture.yaml +67 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-org-design.yaml +68 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-sla-design.yaml +65 -0
- package/courses/rest-api-error-handling/scenarios/level-4/error-training-program.yaml +61 -0
- package/courses/rest-api-error-handling/scenarios/level-4/expert-error-shift.yaml +63 -0
- package/courses/rest-api-error-handling/scenarios/level-5/comprehensive-error-system.yaml +68 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-ai-future.yaml +75 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-behavioral-science.yaml +73 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-board-strategy.yaml +60 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-consulting-engagement.yaml +58 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-industry-benchmarks.yaml +72 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-ma-integration.yaml +68 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-product-development.yaml +66 -0
- package/courses/rest-api-error-handling/scenarios/level-5/error-regulatory-landscape.yaml +80 -0
- package/courses/rest-api-error-handling/scenarios/level-5/master-error-shift.yaml +73 -0
- package/dist/cli/commands/add.d.ts.map +1 -1
- package/dist/cli/commands/add.js +6 -5
- package/dist/cli/commands/add.js.map +1 -1
- package/dist/cli/commands/generate.d.ts.map +1 -1
- package/dist/cli/commands/generate.js +4 -0
- package/dist/cli/commands/generate.js.map +1 -1
- package/dist/cli/commands/list.d.ts.map +1 -1
- package/dist/cli/commands/list.js +6 -18
- package/dist/cli/commands/list.js.map +1 -1
- package/dist/cli/commands/train.d.ts.map +1 -1
- package/dist/cli/commands/train.js +18 -18
- package/dist/cli/commands/train.js.map +1 -1
- package/dist/cli/index.js +93 -55
- package/dist/cli/index.js.map +1 -1
- package/dist/cli/run-demo.js +2 -1
- package/dist/cli/run-demo.js.map +1 -1
- package/dist/cli/setup.d.ts +18 -0
- package/dist/cli/setup.d.ts.map +1 -0
- package/dist/cli/setup.js +154 -0
- package/dist/cli/setup.js.map +1 -0
- package/dist/engine/agent-bridge.d.ts +5 -2
- package/dist/engine/agent-bridge.d.ts.map +1 -1
- package/dist/engine/agent-bridge.js +36 -9
- package/dist/engine/agent-bridge.js.map +1 -1
- package/dist/engine/loader.d.ts +21 -0
- package/dist/engine/loader.d.ts.map +1 -1
- package/dist/engine/loader.js +54 -1
- package/dist/engine/loader.js.map +1 -1
- package/dist/engine/training-loop.d.ts.map +1 -1
- package/dist/engine/training-loop.js +1 -0
- package/dist/engine/training-loop.js.map +1 -1
- package/dist/engine/training.d.ts.map +1 -1
- package/dist/engine/training.js +1 -0
- package/dist/engine/training.js.map +1 -1
- package/dist/generator/skill-generator.d.ts +1 -1
- package/dist/generator/skill-generator.d.ts.map +1 -1
- package/dist/generator/skill-generator.js +21 -2
- package/dist/generator/skill-generator.js.map +1 -1
- package/dist/mcp/server.d.ts.map +1 -1
- package/dist/mcp/server.js +11 -26
- package/dist/mcp/server.js.map +1 -1
- package/dist/mcp/session-manager.d.ts +3 -1
- package/dist/mcp/session-manager.d.ts.map +1 -1
- package/dist/mcp/session-manager.js +44 -22
- package/dist/mcp/session-manager.js.map +1 -1
- package/dist/types/schemas.d.ts +38 -13
- package/dist/types/schemas.d.ts.map +1 -1
- package/dist/types/schemas.js +9 -5
- package/dist/types/schemas.js.map +1 -1
- package/package.json +1 -1
|
@@ -0,0 +1,63 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-economics-roi
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Analyze review economics — calculate the true cost of code review, model ROI of review improvements, and present to executive leadership"
|
|
7
|
+
tags: [github, pr-review, economics, ROI, cost-analysis, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the Director of Engineering presenting a business case to the
|
|
13
|
+
CFO for investing in code review infrastructure. The CFO's question:
|
|
14
|
+
"Code review costs us millions in engineer time. How do we know it's
|
|
15
|
+
worth it? And if we invest more, what's the return?"
|
|
16
|
+
|
|
17
|
+
Data you've collected:
|
|
18
|
+
- 600 engineers, average fully-loaded cost: $200K/year ($100/hour)
|
|
19
|
+
- Average engineer spends 6 hours/week on code review
|
|
20
|
+
- That's 600 × 6 × 52 = 187,200 review hours/year = $18.7M/year
|
|
21
|
+
- Average PR takes 25 minutes to review
|
|
22
|
+
- 78,000 PRs merged per year
|
|
23
|
+
|
|
24
|
+
Cost of review failures (incidents traced to review gaps):
|
|
25
|
+
- 12 production incidents/year attributed to review failures
|
|
26
|
+
- Average incident cost: $150K (engineering time, customer impact,
|
|
27
|
+
reputation)
|
|
28
|
+
- 3 compliance findings in last audit: $500K in remediation + fines
|
|
29
|
+
- 2 security vulnerabilities that passed review: $800K total impact
|
|
30
|
+
- Total cost of review failures: $3.5M/year
|
|
31
|
+
|
|
32
|
+
Proposed investments:
|
|
33
|
+
1. Review automation platform: $800K (build) + $200K/year (maintain)
|
|
34
|
+
Expected: Reduce review time by 30%, catch 60% of style issues
|
|
35
|
+
2. Reviewer training program: $150K/year
|
|
36
|
+
Expected: Reduce review rounds by 25%, improve bug detection by 15%
|
|
37
|
+
3. AI-assisted review: $300K/year (tooling)
|
|
38
|
+
Expected: Reduce first-review time by 40%, flag 50% of common issues
|
|
39
|
+
4. Review analytics dashboard: $200K (build) + $50K/year (maintain)
|
|
40
|
+
Expected: Identify bottlenecks, reduce merge time by 20%
|
|
41
|
+
|
|
42
|
+
Task: Build the complete economic model. Write: the true cost of
|
|
43
|
+
code review (including hidden costs like context switching, blocked
|
|
44
|
+
PRs, opportunity cost), the value model (what code review prevents,
|
|
45
|
+
quantified), the ROI analysis for each investment (with sensitivity
|
|
46
|
+
analysis), the 3-year financial projection, and the executive
|
|
47
|
+
presentation (2-page summary the CFO can present to the board).
|
|
48
|
+
Include the assumptions, risks, and what metrics to track to validate
|
|
49
|
+
the projections.
|
|
50
|
+
|
|
51
|
+
assertions:
|
|
52
|
+
- type: llm_judge
|
|
53
|
+
criteria: "Cost model includes hidden costs — beyond the $18.7M direct cost, includes context-switching costs (review interrupting deep work), blocked PR costs (engineers waiting for review), and opportunity cost (what engineers could build instead of reviewing)"
|
|
54
|
+
weight: 0.35
|
|
55
|
+
description: "Complete cost model with hidden costs"
|
|
56
|
+
- type: llm_judge
|
|
57
|
+
criteria: "ROI analysis is rigorous — each investment has NPV calculation, payback period, sensitivity analysis (what if improvements are 50% less than expected?), and the analysis accounts for interaction effects (automation + training together may have compounding benefits)"
|
|
58
|
+
weight: 0.35
|
|
59
|
+
description: "Rigorous ROI analysis"
|
|
60
|
+
- type: llm_judge
|
|
61
|
+
criteria: "Executive presentation is CFO-ready — leads with the business impact (not technical details), quantifies the 'do nothing' cost, presents investment options with clear ROI comparison, and includes metrics to track for accountability"
|
|
62
|
+
weight: 0.30
|
|
63
|
+
description: "CFO-ready executive presentation"
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-executive-communication
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Executive communication about review — present review process health, investments, and strategy to C-suite and board audiences"
|
|
7
|
+
tags: [github, pr-review, executive, communication, strategy, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the CTO preparing 3 different communications about code review
|
|
13
|
+
for executive audiences with different concerns.
|
|
14
|
+
|
|
15
|
+
Communication 1 — Board of Directors quarterly update:
|
|
16
|
+
The board wants to understand "engineering quality" as a competitive
|
|
17
|
+
advantage. They've heard competitors are shipping faster. Your data:
|
|
18
|
+
- Code review catches 65% of bugs before production
|
|
19
|
+
- Average merge time: 1.8 days (industry median: 2.5 days)
|
|
20
|
+
- Review-related incidents: 4/quarter (down from 12 last year)
|
|
21
|
+
- Review investment: $3.2M/year (5.3% of engineering budget)
|
|
22
|
+
- Developer satisfaction with review: 78% (up from 45% last year)
|
|
23
|
+
The board asks: "Are we over-investing in review at the expense of
|
|
24
|
+
shipping speed?"
|
|
25
|
+
|
|
26
|
+
Communication 2 — CEO 1:1 escalation:
|
|
27
|
+
The Head of Product is complaining that code review is the #1
|
|
28
|
+
bottleneck to shipping features. They want to reduce required reviews
|
|
29
|
+
from 2 to 0 for "low-risk" PRs and allow self-merge for senior
|
|
30
|
+
engineers. Your security team says this violates SOC 2 controls. The
|
|
31
|
+
CEO wants your recommendation by Friday.
|
|
32
|
+
|
|
33
|
+
Communication 3 — All-hands engineering presentation:
|
|
34
|
+
The annual engineering all-hands is next week. You want to present
|
|
35
|
+
the "State of Code Review" — celebrating improvements, acknowledging
|
|
36
|
+
remaining challenges, and building excitement for the review platform
|
|
37
|
+
investment. Your audience is 600 engineers ranging from interns to
|
|
38
|
+
distinguished engineers.
|
|
39
|
+
|
|
40
|
+
Task: Write all 3 communications. For each: adapt the message to the
|
|
41
|
+
audience (board speaks business/risk, CEO wants a decision framework,
|
|
42
|
+
engineers want to understand the "why"), use data effectively (board
|
|
43
|
+
gets trends and benchmarks, CEO gets risk analysis, engineers get
|
|
44
|
+
impact stories), and include clear asks or next steps. The board
|
|
45
|
+
update should be 1 page, the CEO memo should be 2 pages with a
|
|
46
|
+
recommendation, and the all-hands should be an outline with speaker
|
|
47
|
+
notes.
|
|
48
|
+
|
|
49
|
+
assertions:
|
|
50
|
+
- type: llm_judge
|
|
51
|
+
criteria: "Board communication speaks business language — frames review as risk management and competitive advantage (not engineering process), uses industry benchmarks to show strong performance, and answers the 'over-investing' question with data showing declining incidents and competitive merge times"
|
|
52
|
+
weight: 0.35
|
|
53
|
+
description: "Business-language board communication"
|
|
54
|
+
- type: llm_judge
|
|
55
|
+
criteria: "CEO memo provides a clear recommendation — doesn't just present options, takes a position on the self-merge request with risk analysis, proposes a middle ground that addresses product's velocity concerns while maintaining compliance, and includes a decision framework for future review policy changes"
|
|
56
|
+
weight: 0.35
|
|
57
|
+
description: "Decision-ready CEO memo"
|
|
58
|
+
- type: llm_judge
|
|
59
|
+
criteria: "All-hands presentation engages engineers — celebrates specific team and individual contributions, uses concrete impact stories (not just metrics), acknowledges pain points honestly, and builds excitement for the review platform investment with a clear vision"
|
|
60
|
+
weight: 0.30
|
|
61
|
+
description: "Engineer-engaging all-hands"
|
|
@@ -0,0 +1,69 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-incident-postmortem
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Conduct review process postmortems — lead blameless investigations into systemic review failures and implement organizational improvements"
|
|
7
|
+
tags: [github, pr-review, postmortem, incident-response, process-improvement, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the Head of Engineering conducting a postmortem after the most
|
|
13
|
+
serious review-related incident in company history. A payment
|
|
14
|
+
processing bug caused $2.3M in duplicate charges over 72 hours before
|
|
15
|
+
detection.
|
|
16
|
+
|
|
17
|
+
Incident timeline:
|
|
18
|
+
- Day 0: PR #4521 "Optimize payment batch processing" submitted by a
|
|
19
|
+
senior engineer (800 lines, refactoring payment job scheduler)
|
|
20
|
+
- Day 1: First review by a mid-level engineer — "LGTM, nice refactor"
|
|
21
|
+
(spent 8 minutes on 800 lines based on GitHub activity timestamps)
|
|
22
|
+
- Day 1: Second review by another mid-level — requested minor naming
|
|
23
|
+
change, then approved after the change was made
|
|
24
|
+
- Day 2: PR merged after CI passed (all 247 tests green)
|
|
25
|
+
- Day 3: Deployed to production via automated pipeline
|
|
26
|
+
- Day 5: Customer reports duplicate charges. Investigation begins
|
|
27
|
+
- Day 6: Root cause identified — race condition in the batch scheduler
|
|
28
|
+
that causes payment jobs to be processed twice under high load
|
|
29
|
+
- Day 8: Hotfix deployed. Customer refund process begins
|
|
30
|
+
|
|
31
|
+
Review failures identified:
|
|
32
|
+
1. Neither reviewer had payment domain expertise (CODEOWNERS didn't
|
|
33
|
+
include the batch scheduler directory)
|
|
34
|
+
2. The 8-minute review of 800 lines was clearly insufficient
|
|
35
|
+
3. The race condition was subtle but identifiable with careful review
|
|
36
|
+
of the concurrency logic
|
|
37
|
+
4. Test coverage was high (85%) but no tests simulated concurrent
|
|
38
|
+
execution or high-load conditions
|
|
39
|
+
5. The PR was marked "refactoring" which reduced perceived risk
|
|
40
|
+
6. The senior author's reputation created unconscious trust bias
|
|
41
|
+
|
|
42
|
+
Additional context:
|
|
43
|
+
- The payment team lost their tech lead 2 months ago (not replaced)
|
|
44
|
+
- CODEOWNERS for payment code points to a team that was reorganized
|
|
45
|
+
- The review SLA pressure (merge within 1 day) may have rushed reviews
|
|
46
|
+
- 3 similar but smaller incidents occurred in the past year
|
|
47
|
+
|
|
48
|
+
Task: Lead the postmortem. Write: the blameless incident analysis
|
|
49
|
+
(contributing factors at the individual, team, process, and
|
|
50
|
+
organizational levels), the immediate fixes (what changes this week),
|
|
51
|
+
the systemic improvements (what changes in 30/60/90 days), the review
|
|
52
|
+
process changes specifically for payment/financial code, and the
|
|
53
|
+
executive summary for the board (including financial impact and
|
|
54
|
+
remediation plan). Address the pattern of 3 similar incidents and why
|
|
55
|
+
previous fixes didn't prevent this one.
|
|
56
|
+
|
|
57
|
+
assertions:
|
|
58
|
+
- type: llm_judge
|
|
59
|
+
criteria: "Analysis is truly blameless — doesn't blame the 8-minute reviewer or the senior author, instead identifies systemic factors: outdated CODEOWNERS, missing domain expertise after tech lead departure, SLA pressure encouraging superficial reviews, and trust bias toward senior engineers"
|
|
60
|
+
weight: 0.35
|
|
61
|
+
description: "Truly blameless analysis"
|
|
62
|
+
- type: llm_judge
|
|
63
|
+
criteria: "Improvements address all levels — individual (reviewer training on concurrency), team (payment domain expertise gap), process (CODEOWNERS update, risk-based review requirements), and organizational (SLA policy that doesn't incentivize rubber-stamping)"
|
|
64
|
+
weight: 0.35
|
|
65
|
+
description: "Multi-level improvements"
|
|
66
|
+
- type: llm_judge
|
|
67
|
+
criteria: "Pattern analysis explains why previous fixes failed — the 3 prior incidents likely led to narrow fixes (fix the specific bug) rather than systemic changes, and this postmortem addresses the meta-problem of why postmortem action items aren't preventing recurrence"
|
|
68
|
+
weight: 0.30
|
|
69
|
+
description: "Pattern-breaking analysis"
|
|
@@ -0,0 +1,62 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-org-design
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Design review organization — build teams and roles dedicated to code review excellence, training, and tooling"
|
|
7
|
+
tags: [github, pr-review, org-design, teams, roles, career-paths, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the SVP of Engineering creating a dedicated "Developer
|
|
13
|
+
Productivity" organization that includes a Code Review Excellence
|
|
14
|
+
team. The company has 600 engineers and the CEO has approved headcount
|
|
15
|
+
for this new function.
|
|
16
|
+
|
|
17
|
+
The mandate:
|
|
18
|
+
- Reduce average merge time from 3 days to 1 day
|
|
19
|
+
- Reduce review-related production incidents by 80%
|
|
20
|
+
- Achieve 90%+ developer satisfaction with the review process (from
|
|
21
|
+
current 45%)
|
|
22
|
+
- Build review tooling and automation as a platform
|
|
23
|
+
- Create a reviewer training and certification program
|
|
24
|
+
- Establish review standards that scale with the company
|
|
25
|
+
|
|
26
|
+
Available headcount: 12 FTEs
|
|
27
|
+
Budget: $500K/year for tooling (separate from headcount)
|
|
28
|
+
|
|
29
|
+
Questions to answer:
|
|
30
|
+
1. What roles do you need? (Platform engineers, program managers,
|
|
31
|
+
trainers, data analysts?)
|
|
32
|
+
2. How does this team interact with the 50 product teams?
|
|
33
|
+
3. Where does this team sit in the org chart?
|
|
34
|
+
4. What does the career ladder look like for "review excellence"?
|
|
35
|
+
5. How do you avoid becoming a bottleneck or bureaucratic overhead?
|
|
36
|
+
6. What does success look like at 6 months, 1 year, and 2 years?
|
|
37
|
+
|
|
38
|
+
Constraint: The engineering culture values autonomy. Any centralized
|
|
39
|
+
function risks being seen as "the review police." You need buy-in from
|
|
40
|
+
team leads.
|
|
41
|
+
|
|
42
|
+
Task: Design the organization. Write: the team structure (roles,
|
|
43
|
+
responsibilities, reporting lines), the interaction model with product
|
|
44
|
+
teams (embedded vs centralized vs hybrid), the career ladder for review
|
|
45
|
+
excellence roles, the buy-in strategy (how to get team lead support
|
|
46
|
+
without being "review police"), the phased roadmap (what to deliver at
|
|
47
|
+
6 months, 1 year, 2 years), and the success metrics with accountability
|
|
48
|
+
framework.
|
|
49
|
+
|
|
50
|
+
assertions:
|
|
51
|
+
- type: llm_judge
|
|
52
|
+
criteria: "Team structure is well-designed — includes platform engineers (build tooling), program managers (process), trainers (education), and data analysts (metrics), with clear roles and not too top-heavy for a 12-person team"
|
|
53
|
+
weight: 0.35
|
|
54
|
+
description: "Well-designed team structure"
|
|
55
|
+
- type: llm_judge
|
|
56
|
+
criteria: "Interaction model preserves autonomy — uses an enabling/platform model (not command-and-control), product teams opt in to services, the team provides tools and training rather than mandates, and the 'review police' perception is actively countered"
|
|
57
|
+
weight: 0.35
|
|
58
|
+
description: "Autonomy-preserving interaction model"
|
|
59
|
+
- type: llm_judge
|
|
60
|
+
criteria: "Career ladder is viable — creates career progression for review excellence roles (not a dead-end), connects to broader developer productivity career paths, and provides incentives for engineers to specialize in this area"
|
|
61
|
+
weight: 0.30
|
|
62
|
+
description: "Viable career ladder"
|
|
@@ -0,0 +1,64 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-platform-architecture
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Architect a review platform — design the technical architecture for a company-wide review automation and analytics platform"
|
|
7
|
+
tags: [github, pr-review, architecture, platform, automation, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the Staff Engineer tasked with building an internal "Review
|
|
13
|
+
Platform" — a centralized system that orchestrates code review across
|
|
14
|
+
your 600-engineer organization. The platform replaces a patchwork of
|
|
15
|
+
scripts, bots, and manual processes.
|
|
16
|
+
|
|
17
|
+
Platform requirements:
|
|
18
|
+
1. Unified reviewer assignment across all repos and GitHub orgs
|
|
19
|
+
2. Risk-based review routing (auto-classify PRs by risk level and
|
|
20
|
+
route to appropriate reviewers)
|
|
21
|
+
3. Real-time review analytics dashboard
|
|
22
|
+
4. Compliance audit trail generator
|
|
23
|
+
5. Integration with GitHub, GitLab, Slack, Jira, PagerDuty
|
|
24
|
+
6. AI-assisted review suggestions (flag potential issues before human
|
|
25
|
+
review)
|
|
26
|
+
7. Custom review workflows (different flows for different teams/repos)
|
|
27
|
+
|
|
28
|
+
Scale requirements:
|
|
29
|
+
- 200+ repos across 3 GitHub orgs
|
|
30
|
+
- 1,500 PRs per week
|
|
31
|
+
- 50ms p99 latency for webhook processing
|
|
32
|
+
- 99.9% uptime (review platform down = all merges blocked)
|
|
33
|
+
- 30-day event replay for debugging
|
|
34
|
+
- Horizontal scalability for 3x growth
|
|
35
|
+
|
|
36
|
+
Technical constraints:
|
|
37
|
+
- Must run on existing Kubernetes cluster
|
|
38
|
+
- Must use existing PostgreSQL and Redis infrastructure
|
|
39
|
+
- GitHub API rate limits: 5,000/hour per installation
|
|
40
|
+
- Must handle GitHub webhook delivery guarantees (at-least-once)
|
|
41
|
+
- Must support gradual rollout (repo by repo)
|
|
42
|
+
|
|
43
|
+
Task: Design the platform architecture. Write: the system design
|
|
44
|
+
(components, data flow, storage, event processing), the API design
|
|
45
|
+
(internal APIs for team customization), the scalability strategy
|
|
46
|
+
(how to handle 1,500 PRs/week within rate limits, horizontal scaling
|
|
47
|
+
plan), the reliability design (what happens when the platform is down,
|
|
48
|
+
circuit breakers, graceful degradation), and the deployment strategy
|
|
49
|
+
(gradual rollout, feature flags, rollback plan). Include architecture
|
|
50
|
+
diagrams (text-based) and key technical decisions with trade-offs.
|
|
51
|
+
|
|
52
|
+
assertions:
|
|
53
|
+
- type: llm_judge
|
|
54
|
+
criteria: "Architecture handles the scale — event-driven design with webhook receivers, job queues, and workers that handle 1,500 PRs/week within API rate limits. Includes rate limit pooling across 3 GitHub orgs and back-pressure mechanisms"
|
|
55
|
+
weight: 0.35
|
|
56
|
+
description: "Scale-handling architecture"
|
|
57
|
+
- type: llm_judge
|
|
58
|
+
criteria: "Reliability is enterprise-grade — graceful degradation when platform is down (PRs can still be merged manually), circuit breakers for external services, idempotent webhook processing (at-least-once handling), and 30-day event replay for debugging"
|
|
59
|
+
weight: 0.35
|
|
60
|
+
description: "Enterprise-grade reliability"
|
|
61
|
+
- type: llm_judge
|
|
62
|
+
criteria: "Deployment strategy is low-risk — gradual rollout repo by repo, feature flags for each capability, rollback plan that doesn't disrupt ongoing reviews, and the platform can be disabled per-repo without affecting others"
|
|
63
|
+
weight: 0.30
|
|
64
|
+
description: "Low-risk deployment strategy"
|
|
@@ -0,0 +1,66 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-training-program
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Design reviewer training program — create a comprehensive curriculum for building review expertise across seniority levels"
|
|
7
|
+
tags: [github, pr-review, training, education, certification, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're designing a company-wide code review training program for your
|
|
13
|
+
600-engineer organization. The program should transform review from a
|
|
14
|
+
chore into a valued skill with career impact.
|
|
15
|
+
|
|
16
|
+
Current problems the training must address:
|
|
17
|
+
- New hires take 3+ months before their reviews add value
|
|
18
|
+
- 30% of review comments are preferences, not issues
|
|
19
|
+
- Reviewers miss bugs because they focus on style
|
|
20
|
+
- Senior engineers' reviews are rubber-stamped ("too experienced to
|
|
21
|
+
question")
|
|
22
|
+
- No one teaches how to write good review comments
|
|
23
|
+
- Reviewers don't know when to approve vs request changes
|
|
24
|
+
- Cross-domain reviews (backend reviewer on frontend PR) are shallow
|
|
25
|
+
|
|
26
|
+
Target audience tiers:
|
|
27
|
+
1. New hires / juniors: First-time reviewers who need fundamentals
|
|
28
|
+
2. Mid-level engineers: Competent reviewers who need depth and breadth
|
|
29
|
+
3. Senior / staff engineers: Experienced reviewers who need to review
|
|
30
|
+
architecture and mentor through reviews
|
|
31
|
+
4. Engineering managers: Need to evaluate review quality and coach
|
|
32
|
+
|
|
33
|
+
Available formats:
|
|
34
|
+
- Self-paced online modules (budget: $50K to build)
|
|
35
|
+
- Live workshops (budget: $30K/year, monthly cadence)
|
|
36
|
+
- Shadow reviewing program (pair reviewing with a mentor)
|
|
37
|
+
- Review certification (gamification to incentivize learning)
|
|
38
|
+
- Practice exercises with curated PRs (real anonymized examples)
|
|
39
|
+
|
|
40
|
+
Constraints:
|
|
41
|
+
- Maximum 2 hours/month of training time per engineer
|
|
42
|
+
- Must show measurable improvement within 3 months
|
|
43
|
+
- Must not feel punitive (training shouldn't be "you review badly")
|
|
44
|
+
- Must work for remote-first company (75% remote)
|
|
45
|
+
|
|
46
|
+
Task: Design the complete training program. Write: the curriculum for
|
|
47
|
+
each tier (learning objectives, modules, exercises), the shadow
|
|
48
|
+
reviewing program design (matching, structure, graduation criteria),
|
|
49
|
+
the certification system (levels, requirements, incentives, badge
|
|
50
|
+
visibility), the measurement framework (how to prove training works),
|
|
51
|
+
and the rollout plan (who goes first, how to drive adoption without
|
|
52
|
+
mandating). Include sample exercise materials for each tier.
|
|
53
|
+
|
|
54
|
+
assertions:
|
|
55
|
+
- type: llm_judge
|
|
56
|
+
criteria: "Curriculum is tier-appropriate — juniors learn fundamentals (comment types, approval criteria, etiquette), mid-level learn depth (security review, performance review, architecture), seniors learn strategic review and mentoring, and managers learn to evaluate and coach review quality"
|
|
57
|
+
weight: 0.35
|
|
58
|
+
description: "Tier-appropriate curriculum"
|
|
59
|
+
- type: llm_judge
|
|
60
|
+
criteria: "Shadow reviewing program is well-structured — includes matching criteria (expertise, personality, timezone), session structure (observe → co-review → independent with feedback), graduation criteria, and scales beyond 1:1 mentoring"
|
|
61
|
+
weight: 0.35
|
|
62
|
+
description: "Well-structured shadow program"
|
|
63
|
+
- type: llm_judge
|
|
64
|
+
criteria: "Measurement framework proves ROI — tracks leading indicators (comment quality scores, training completion) and lagging indicators (review-related incidents, merge time, developer satisfaction), with a control group or before/after comparison design"
|
|
65
|
+
weight: 0.30
|
|
66
|
+
description: "ROI-proving measurement framework"
|
|
@@ -0,0 +1,76 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: review-vendor-evaluation
|
|
3
|
+
level: 4
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Evaluate review tool vendors — conduct a thorough evaluation of code review platforms, AI review tools, and analytics vendors"
|
|
7
|
+
tags: [github, pr-review, vendor-evaluation, procurement, tooling, expert]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're leading the vendor evaluation for code review tooling. The
|
|
13
|
+
company has budget for 1-2 tools and needs a recommendation within
|
|
14
|
+
30 days. Your 600-engineer organization currently uses GitHub's native
|
|
15
|
+
review features with some custom scripts.
|
|
16
|
+
|
|
17
|
+
Vendors to evaluate:
|
|
18
|
+
|
|
19
|
+
Vendor A — ReviewBot Pro ($15/user/month):
|
|
20
|
+
- AI-powered code review with custom model training
|
|
21
|
+
- Automated reviewer assignment based on expertise
|
|
22
|
+
- Review analytics dashboard
|
|
23
|
+
- Claims: "Reduces review time by 50%"
|
|
24
|
+
- Integration: GitHub, GitLab, Bitbucket
|
|
25
|
+
- Security: SOC 2 Type II certified, code never leaves your network
|
|
26
|
+
- Concerns: requires GitHub App with broad repo permissions, new
|
|
27
|
+
startup (2 years old, Series A)
|
|
28
|
+
|
|
29
|
+
Vendor B — CodeGuard Enterprise ($25/user/month):
|
|
30
|
+
- Security-focused review automation (SAST, dependency scanning)
|
|
31
|
+
- Compliance reporting (SOC 2, PCI, HIPAA)
|
|
32
|
+
- Custom policy engine (enforce review rules per repo)
|
|
33
|
+
- Claims: "Catches 70% of security issues before human review"
|
|
34
|
+
- Integration: GitHub only
|
|
35
|
+
- Security: FedRAMP authorized, self-hosted option available
|
|
36
|
+
- Concerns: expensive, long implementation timeline (3-6 months)
|
|
37
|
+
|
|
38
|
+
Vendor C — DevMetrics ($8/user/month):
|
|
39
|
+
- Review analytics and team performance dashboards
|
|
40
|
+
- Review SLA tracking and alerting
|
|
41
|
+
- Developer experience surveys integrated with review data
|
|
42
|
+
- Claims: "Teams using DevMetrics improve merge time by 35%"
|
|
43
|
+
- Integration: GitHub, Jira, Slack
|
|
44
|
+
- Security: SOC 2 Type I, cloud only
|
|
45
|
+
- Concerns: analytics only (no automation), uses anonymized data for
|
|
46
|
+
their own ML model training
|
|
47
|
+
|
|
48
|
+
Vendor D — Open source alternative (Danger.js + custom GitHub Actions):
|
|
49
|
+
- Free, fully customizable
|
|
50
|
+
- Active community
|
|
51
|
+
- No vendor lock-in
|
|
52
|
+
- Concerns: maintenance burden (estimated 0.5 FTE), limited analytics,
|
|
53
|
+
no AI capabilities, no support SLA
|
|
54
|
+
|
|
55
|
+
Task: Conduct the vendor evaluation. Write: the evaluation framework
|
|
56
|
+
(weighted criteria matrix), the detailed assessment of each vendor
|
|
57
|
+
(strengths, weaknesses, risks), the total cost of ownership analysis
|
|
58
|
+
(3-year TCO including implementation, maintenance, training), the
|
|
59
|
+
security and privacy assessment (data handling, permissions, vendor
|
|
60
|
+
risk), and the recommendation memo (which vendor(s) to choose, with
|
|
61
|
+
implementation plan and exit strategy). Include the pilot program
|
|
62
|
+
design to validate the choice.
|
|
63
|
+
|
|
64
|
+
assertions:
|
|
65
|
+
- type: llm_judge
|
|
66
|
+
criteria: "Evaluation framework is rigorous — weighted criteria include functionality, security, cost, integration complexity, vendor viability (startup risk for Vendor A), scalability, and exit strategy. Weights reflect the organization's priorities"
|
|
67
|
+
weight: 0.35
|
|
68
|
+
description: "Rigorous evaluation framework"
|
|
69
|
+
- type: llm_judge
|
|
70
|
+
criteria: "TCO analysis is complete — includes license costs, implementation effort, ongoing maintenance, training, and hidden costs (productivity loss during migration, vendor lock-in costs). Compares 3-year TCO across all options including the open-source alternative"
|
|
71
|
+
weight: 0.35
|
|
72
|
+
description: "Complete TCO analysis"
|
|
73
|
+
- type: llm_judge
|
|
74
|
+
criteria: "Recommendation is well-reasoned — makes a specific recommendation (not 'it depends'), addresses security concerns (especially Vendor C's data usage), includes a pilot program to validate before full rollout, and has an exit strategy for each vendor"
|
|
75
|
+
weight: 0.30
|
|
76
|
+
description: "Well-reasoned recommendation"
|
|
@@ -0,0 +1,68 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: comprehensive-review-system
|
|
3
|
+
level: 5
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Design comprehensive review system — architect a 5-layer review system for a Fortune 500 technology company"
|
|
7
|
+
tags: [github, pr-review, comprehensive, enterprise, architecture, master]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the CTO of a Fortune 500 technology company ($8B revenue, 3,000
|
|
13
|
+
engineers) designing a unified code review system. The company has 6
|
|
14
|
+
business units acquired over 10 years, each with different tooling,
|
|
15
|
+
processes, and cultures. The board has mandated unification after a
|
|
16
|
+
regulatory finding that change management controls were "inconsistent
|
|
17
|
+
and insufficient."
|
|
18
|
+
|
|
19
|
+
Current state by business unit:
|
|
20
|
+
- Cloud Platform (600 eng): GitHub Enterprise, mature review process
|
|
21
|
+
- Consumer Apps (500 eng): GitHub Cloud, fast-moving, minimal process
|
|
22
|
+
- Enterprise Software (400 eng): GitLab, heavyweight review gates
|
|
23
|
+
- Data & Analytics (300 eng): Mix of GitHub and Jupyter, informal
|
|
24
|
+
- IoT/Embedded (200 eng): Gerrit, hardware-focused review needs
|
|
25
|
+
- Recently acquired AI startup (200 eng): No formal process, pair
|
|
26
|
+
programming culture
|
|
27
|
+
|
|
28
|
+
Design the review system in 5 layers:
|
|
29
|
+
|
|
30
|
+
Layer 1 — Governance: Universal policies, compliance mapping (SOX,
|
|
31
|
+
SOC 2, PCI DSS, HIPAA, FedRAMP across different BUs), audit trail
|
|
32
|
+
Layer 2 — Platform: Unified tooling, migration from 4 VCS platforms
|
|
33
|
+
to GitHub Enterprise, custom automation
|
|
34
|
+
Layer 3 — Process: Review workflows per risk level, SLAs, escalation,
|
|
35
|
+
cross-BU reviews for shared code
|
|
36
|
+
Layer 4 — Intelligence: AI-powered review, analytics, predictive
|
|
37
|
+
quality, anomaly detection
|
|
38
|
+
Layer 5 — Culture: Training, certification, incentives, career paths,
|
|
39
|
+
developer experience
|
|
40
|
+
|
|
41
|
+
Constraints:
|
|
42
|
+
- $10M budget over 3 years
|
|
43
|
+
- Cannot disrupt any BU's shipping velocity during migration
|
|
44
|
+
- Must satisfy 6 different compliance frameworks simultaneously
|
|
45
|
+
- Must handle 5,000+ PRs per week across all BUs
|
|
46
|
+
- IoT/Embedded team has legitimate needs for Gerrit's change-based
|
|
47
|
+
review model
|
|
48
|
+
- Board expects quarterly progress reports
|
|
49
|
+
|
|
50
|
+
Task: Design all 5 layers. For each, write: the detailed design, the
|
|
51
|
+
migration/implementation plan, the interaction with other layers, and
|
|
52
|
+
the success metrics. Then write: the 3-year phased roadmap with
|
|
53
|
+
quarterly milestones, the $10M budget allocation, the organizational
|
|
54
|
+
model (who owns each layer), and the board presentation for approval.
|
|
55
|
+
|
|
56
|
+
assertions:
|
|
57
|
+
- type: llm_judge
|
|
58
|
+
criteria: "5-layer design is coherent — each layer builds on the one below, there are clear interfaces between layers, and the design handles the complexity of 6 BUs without over-simplifying. The IoT/Gerrit exception is handled gracefully (bridge or adapter, not forced migration)"
|
|
59
|
+
weight: 0.35
|
|
60
|
+
description: "Coherent 5-layer design"
|
|
61
|
+
- type: llm_judge
|
|
62
|
+
criteria: "Migration plan is realistic — handles the 4-to-1 VCS consolidation without disrupting shipping, phases the migration BU by BU, has rollback plans for each phase, and the 3-year timeline with quarterly milestones is achievable"
|
|
63
|
+
weight: 0.35
|
|
64
|
+
description: "Realistic migration plan"
|
|
65
|
+
- type: llm_judge
|
|
66
|
+
criteria: "Budget allocation is justified — the $10M is distributed across layers and years with clear rationale, the build-vs-buy decisions are made for each component, and the ROI projection shows when the investment pays back through reduced incidents, compliance costs, and engineering productivity"
|
|
67
|
+
weight: 0.30
|
|
68
|
+
description: "Justified budget allocation"
|
|
@@ -0,0 +1,73 @@
|
|
|
1
|
+
meta:
|
|
2
|
+
id: master-review-shift
|
|
3
|
+
level: 5
|
|
4
|
+
course: github-pr-review
|
|
5
|
+
type: output
|
|
6
|
+
description: "Master review shift — navigate existential threats to the review function while maintaining organizational trust and shipping velocity"
|
|
7
|
+
tags: [github, pr-review, shift-simulation, crisis, existential, master]
|
|
8
|
+
|
|
9
|
+
state: {}
|
|
10
|
+
|
|
11
|
+
trigger: |
|
|
12
|
+
You're the CTO facing a perfect storm of review-related challenges
|
|
13
|
+
that threaten the company's engineering capability, compliance
|
|
14
|
+
posture, and competitive position simultaneously.
|
|
15
|
+
|
|
16
|
+
Crisis 1 — Regulatory investigation:
|
|
17
|
+
The SEC is investigating your company's software change management
|
|
18
|
+
practices after a financial reporting error was traced to a code
|
|
19
|
+
change that bypassed review. They've subpoenaed 2 years of code
|
|
20
|
+
review records. Your legal team discovers that 12% of production
|
|
21
|
+
changes in the financial reporting codebase have incomplete review
|
|
22
|
+
trails. The SEC hearing is in 6 weeks.
|
|
23
|
+
|
|
24
|
+
Crisis 2 — Competitive disruption:
|
|
25
|
+
Your main competitor just announced "Zero-Review Deployment" — they
|
|
26
|
+
claim AI reviews 100% of their code and human review is optional.
|
|
27
|
+
Their merge time is 30 minutes vs your 2 days. Your board is asking
|
|
28
|
+
why you're "falling behind on AI adoption." Three VP-level candidates
|
|
29
|
+
withdrew from your hiring pipeline citing your "slow engineering
|
|
30
|
+
culture."
|
|
31
|
+
|
|
32
|
+
Crisis 3 — Platform outage:
|
|
33
|
+
Your GitHub Enterprise instance suffered a data corruption event.
|
|
34
|
+
The review history for 6 months of PRs is unrecoverable. This affects
|
|
35
|
+
your SOC 2 audit evidence, your review analytics baseline, and your
|
|
36
|
+
AI review model training data. GitHub's recovery ETA is "weeks."
|
|
37
|
+
|
|
38
|
+
Crisis 4 — Talent exodus:
|
|
39
|
+
Your top 3 review tooling engineers (out of 12) just gave notice —
|
|
40
|
+
they're joining the competitor from Crisis 2. They built the core
|
|
41
|
+
review automation platform and have institutional knowledge that's
|
|
42
|
+
not documented. Their last day is in 2 weeks.
|
|
43
|
+
|
|
44
|
+
Crisis 5 — Board pressure:
|
|
45
|
+
The board is meeting in 48 hours for an emergency session. They want
|
|
46
|
+
a unified briefing on all 4 crises above and a strategic response.
|
|
47
|
+
Two board members are pushing to "just adopt AI review like the
|
|
48
|
+
competitor." One board member (former regulator) is pushing to "freeze
|
|
49
|
+
all deployments until review gaps are fixed."
|
|
50
|
+
|
|
51
|
+
Task: Navigate all 5 crises. Write: the 48-hour action plan for the
|
|
52
|
+
board meeting, the SEC response strategy (what to present, what to
|
|
53
|
+
remediate, legal coordination), the competitive response (how to
|
|
54
|
+
address the AI review narrative without panic), the platform recovery
|
|
55
|
+
plan (data recovery, interim process, audit evidence reconstruction),
|
|
56
|
+
the talent retention and knowledge transfer plan, and the board
|
|
57
|
+
presentation that unifies all crises into a coherent strategic
|
|
58
|
+
response. Show how the crises interconnect and the risks of solving
|
|
59
|
+
one at the expense of others.
|
|
60
|
+
|
|
61
|
+
assertions:
|
|
62
|
+
- type: llm_judge
|
|
63
|
+
criteria: "Board presentation unifies the crises — shows the interconnections (SEC investigation + data loss = compounded compliance risk, competitor + talent loss = capability risk), doesn't present 5 separate plans but a unified strategy, and addresses both board member extremes (AI adoption vs deployment freeze)"
|
|
64
|
+
weight: 0.35
|
|
65
|
+
description: "Unified crisis board presentation"
|
|
66
|
+
- type: llm_judge
|
|
67
|
+
criteria: "SEC response is legally sound — coordinates with legal counsel, presents a proactive remediation plan rather than defensive posture, addresses the 12% review gap with root cause analysis and corrective actions, and the audit evidence reconstruction plan is credible"
|
|
68
|
+
weight: 0.35
|
|
69
|
+
description: "Legally sound SEC response"
|
|
70
|
+
- type: llm_judge
|
|
71
|
+
criteria: "Competitive response is strategic not reactive — doesn't panic-adopt AI review to match competitor, analyzes the competitor's claim critically (is 'zero-review' actually safe for financial software?), proposes an AI adoption roadmap that's faster but responsible, and addresses the hiring pipeline impact"
|
|
72
|
+
weight: 0.30
|
|
73
|
+
description: "Strategic competitive response"
|