@pigcloud/skills 1.0.10 → 1.1.2
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/CHANGELOG.md +25 -20
- package/README.en.md +41 -75
- package/README.md +26 -39
- package/bin/cli.js +282 -188
- package/bin/rules-loader.js +271 -484
- package/codex-commands/README.md +25 -23
- package/codex-commands/commands/analyze.md +21 -22
- package/codex-commands/commands/build.md +22 -22
- package/codex-commands/commands/design.md +21 -22
- package/codex-commands/commands/distill.md +21 -21
- package/codex-commands/commands/doc.md +21 -22
- package/codex-commands/commands/infra.md +21 -21
- package/codex-commands/commands/init.md +20 -20
- package/codex-commands/commands/kb.md +21 -20
- package/codex-commands/commands/perf.md +21 -21
- package/codex-commands/commands/prd.md +21 -22
- package/codex-commands/commands/review.md +21 -22
- package/codex-commands/commands/security.md +21 -22
- package/codex-commands/commands/test.md +21 -21
- package/codex-commands/commands/workflow.md +21 -20
- package/package.json +5 -2
- package/rules/core/index.md +26 -41
- package/rules/delivery/index.md +25 -0
- package/rules/design/index.md +25 -0
- package/rules/discovery/index.md +25 -0
- package/rules/implementation/index.md +25 -0
- package/rules/index.md +24 -39
- package/rules/overlays/index.md +19 -19
- package/rules/overlays/pig-cloud.md +20 -45
- package/rules/shared/index.md +25 -0
- package/rules/skill-stage-map.json +26 -0
- package/rules/stages.json +48 -0
- package/rules/validation/index.md +25 -0
- package/scripts/add-skill-reference-nav.js +3 -0
- package/scripts/bootstrap-skill-specs.js +96 -0
- package/scripts/ci-validator.sh +51 -114
- package/scripts/generate-skill-prompt-library.js +3 -0
- package/scripts/golden-prompt-suite.current.js +211 -0
- package/scripts/migrate-skill-packages.js +309 -0
- package/scripts/run-golden-replays.js +110 -79
- package/scripts/validate-rules.js +128 -125
- package/scripts/validate-skill-replay-signals.js +45 -57
- package/scripts/validate-skill-shapes.js +153 -127
- package/scripts/validate-skill-stop-rules.js +54 -46
- package/skills/01-discovery/ambiguity-detection/SKILL.md +30 -0
- package/skills/01-discovery/ambiguity-detection/assets/golden-prompt-suite.current.js +22 -0
- package/skills/01-discovery/ambiguity-detection/references/README.md +17 -0
- package/skills/01-discovery/ambiguity-detection/references/cases.md +26 -0
- package/skills/01-discovery/ambiguity-detection/references/prompt-template.md +18 -0
- package/skills/01-discovery/ambiguity-detection/skill-spec.json +26 -0
- package/skills/01-discovery/business-analysis/SKILL.md +30 -0
- package/skills/01-discovery/business-analysis/assets/golden-prompt-suite.current.js +22 -0
- package/skills/01-discovery/business-analysis/references/README.md +17 -0
- package/skills/01-discovery/business-analysis/references/cases.md +26 -0
- package/skills/01-discovery/business-analysis/references/prompt-template.md +18 -0
- package/skills/01-discovery/business-analysis/skill-spec.json +26 -0
- package/skills/01-discovery/impact-analysis/SKILL.md +30 -0
- package/skills/01-discovery/impact-analysis/assets/golden-prompt-suite.current.js +22 -0
- package/skills/01-discovery/impact-analysis/references/README.md +17 -0
- package/skills/01-discovery/impact-analysis/references/cases.md +26 -0
- package/skills/01-discovery/impact-analysis/references/prompt-template.md +18 -0
- package/skills/01-discovery/impact-analysis/skill-spec.json +26 -0
- package/skills/01-discovery/requirement-discovery/SKILL.md +30 -0
- package/skills/01-discovery/requirement-discovery/assets/golden-prompt-suite.current.js +24 -0
- package/skills/01-discovery/requirement-discovery/references/README.md +17 -0
- package/skills/01-discovery/requirement-discovery/references/cases.md +28 -0
- package/skills/01-discovery/requirement-discovery/references/prompt-template.md +18 -0
- package/skills/01-discovery/requirement-discovery/skill-spec.json +26 -0
- package/skills/02-design/api-design/SKILL.md +29 -0
- package/skills/02-design/api-design/assets/golden-prompt-suite.current.js +22 -0
- package/skills/02-design/api-design/references/README.md +17 -0
- package/skills/02-design/api-design/references/cases.md +26 -0
- package/skills/02-design/api-design/references/prompt-template.md +18 -0
- package/skills/02-design/api-design/skill-spec.json +25 -0
- package/skills/02-design/architecture-design/SKILL.md +29 -0
- package/skills/02-design/architecture-design/assets/golden-prompt-suite.current.js +22 -0
- package/skills/02-design/architecture-design/references/README.md +17 -0
- package/skills/02-design/architecture-design/references/cases.md +26 -0
- package/skills/02-design/architecture-design/references/prompt-template.md +18 -0
- package/skills/02-design/architecture-design/skill-spec.json +25 -0
- package/skills/02-design/database-design/SKILL.md +29 -0
- package/skills/02-design/database-design/assets/golden-prompt-suite.current.js +22 -0
- package/skills/02-design/database-design/references/README.md +17 -0
- package/skills/02-design/database-design/references/cases.md +26 -0
- package/skills/02-design/database-design/references/prompt-template.md +18 -0
- package/skills/02-design/database-design/skill-spec.json +25 -0
- package/skills/02-design/task-breakdown/SKILL.md +29 -0
- package/skills/02-design/task-breakdown/assets/golden-prompt-suite.current.js +22 -0
- package/skills/02-design/task-breakdown/references/README.md +17 -0
- package/skills/02-design/task-breakdown/references/cases.md +26 -0
- package/skills/02-design/task-breakdown/references/prompt-template.md +18 -0
- package/skills/02-design/task-breakdown/skill-spec.json +25 -0
- package/skills/03-implementation/backend-development/SKILL.md +29 -0
- package/skills/03-implementation/backend-development/assets/golden-prompt-suite.current.js +22 -0
- package/skills/03-implementation/backend-development/references/README.md +17 -0
- package/skills/03-implementation/backend-development/references/cases.md +26 -0
- package/skills/03-implementation/backend-development/references/prompt-template.md +18 -0
- package/skills/03-implementation/backend-development/skill-spec.json +25 -0
- package/skills/03-implementation/bug-fix/SKILL.md +29 -0
- package/skills/03-implementation/bug-fix/assets/golden-prompt-suite.current.js +22 -0
- package/skills/03-implementation/bug-fix/references/README.md +17 -0
- package/skills/03-implementation/bug-fix/references/cases.md +26 -0
- package/skills/03-implementation/bug-fix/references/prompt-template.md +18 -0
- package/skills/03-implementation/bug-fix/skill-spec.json +25 -0
- package/skills/03-implementation/database-change/SKILL.md +29 -0
- package/skills/03-implementation/database-change/assets/golden-prompt-suite.current.js +22 -0
- package/skills/03-implementation/database-change/references/README.md +17 -0
- package/skills/03-implementation/database-change/references/cases.md +26 -0
- package/skills/03-implementation/database-change/references/prompt-template.md +18 -0
- package/skills/03-implementation/database-change/skill-spec.json +25 -0
- package/skills/03-implementation/frontend-development/SKILL.md +29 -0
- package/skills/03-implementation/frontend-development/assets/golden-prompt-suite.current.js +22 -0
- package/skills/03-implementation/frontend-development/references/README.md +17 -0
- package/skills/03-implementation/frontend-development/references/cases.md +26 -0
- package/skills/03-implementation/frontend-development/references/prompt-template.md +18 -0
- package/skills/03-implementation/frontend-development/skill-spec.json +25 -0
- package/skills/04-validation/code-review/SKILL.md +29 -0
- package/skills/04-validation/code-review/assets/golden-prompt-suite.current.js +22 -0
- package/skills/04-validation/code-review/references/README.md +17 -0
- package/skills/04-validation/code-review/references/cases.md +26 -0
- package/skills/04-validation/code-review/references/prompt-template.md +18 -0
- package/skills/04-validation/code-review/skill-spec.json +25 -0
- package/skills/04-validation/performance-review/SKILL.md +29 -0
- package/skills/04-validation/performance-review/assets/golden-prompt-suite.current.js +22 -0
- package/skills/04-validation/performance-review/references/README.md +17 -0
- package/skills/04-validation/performance-review/references/cases.md +26 -0
- package/skills/04-validation/performance-review/references/prompt-template.md +18 -0
- package/skills/04-validation/performance-review/skill-spec.json +25 -0
- package/skills/04-validation/regression-check/SKILL.md +29 -0
- package/skills/04-validation/regression-check/assets/golden-prompt-suite.current.js +22 -0
- package/skills/04-validation/regression-check/references/README.md +17 -0
- package/skills/04-validation/regression-check/references/cases.md +26 -0
- package/skills/04-validation/regression-check/references/prompt-template.md +18 -0
- package/skills/04-validation/regression-check/skill-spec.json +25 -0
- package/skills/04-validation/security-review/SKILL.md +29 -0
- package/skills/04-validation/security-review/assets/golden-prompt-suite.current.js +22 -0
- package/skills/04-validation/security-review/references/README.md +17 -0
- package/skills/04-validation/security-review/references/cases.md +26 -0
- package/skills/04-validation/security-review/references/prompt-template.md +18 -0
- package/skills/04-validation/security-review/skill-spec.json +25 -0
- package/skills/04-validation/unit-test/SKILL.md +29 -0
- package/skills/04-validation/unit-test/assets/golden-prompt-suite.current.js +22 -0
- package/skills/04-validation/unit-test/references/README.md +17 -0
- package/skills/04-validation/unit-test/references/cases.md +26 -0
- package/skills/04-validation/unit-test/references/prompt-template.md +18 -0
- package/skills/04-validation/unit-test/skill-spec.json +25 -0
- package/skills/05-delivery/change-log/SKILL.md +29 -0
- package/skills/05-delivery/change-log/assets/golden-prompt-suite.current.js +22 -0
- package/skills/05-delivery/change-log/references/README.md +17 -0
- package/skills/05-delivery/change-log/references/cases.md +26 -0
- package/skills/05-delivery/change-log/references/prompt-template.md +18 -0
- package/skills/05-delivery/change-log/skill-spec.json +25 -0
- package/skills/05-delivery/deployment-guide/SKILL.md +29 -0
- package/skills/05-delivery/deployment-guide/assets/golden-prompt-suite.current.js +22 -0
- package/skills/05-delivery/deployment-guide/references/README.md +17 -0
- package/skills/05-delivery/deployment-guide/references/cases.md +26 -0
- package/skills/05-delivery/deployment-guide/references/prompt-template.md +18 -0
- package/skills/05-delivery/deployment-guide/skill-spec.json +25 -0
- package/skills/05-delivery/release-check/SKILL.md +29 -0
- package/skills/05-delivery/release-check/assets/golden-prompt-suite.current.js +22 -0
- package/skills/05-delivery/release-check/references/README.md +17 -0
- package/skills/05-delivery/release-check/references/cases.md +26 -0
- package/skills/05-delivery/release-check/references/prompt-template.md +18 -0
- package/skills/05-delivery/release-check/skill-spec.json +25 -0
- package/skills/05-delivery/release-validation/SKILL.md +29 -0
- package/skills/05-delivery/release-validation/assets/golden-prompt-suite.current.js +22 -0
- package/skills/05-delivery/release-validation/references/README.md +17 -0
- package/skills/05-delivery/release-validation/references/cases.md +26 -0
- package/skills/05-delivery/release-validation/references/prompt-template.md +18 -0
- package/skills/05-delivery/release-validation/skill-spec.json +25 -0
- package/skills/shared/codebase-learning/SKILL.md +29 -0
- package/skills/shared/codebase-learning/assets/golden-prompt-suite.current.js +22 -0
- package/skills/shared/codebase-learning/references/README.md +17 -0
- package/skills/shared/codebase-learning/references/cases.md +26 -0
- package/skills/shared/codebase-learning/references/prompt-template.md +18 -0
- package/skills/shared/codebase-learning/skill-spec.json +25 -0
- package/skills/shared/evidence-collector/SKILL.md +29 -0
- package/skills/shared/evidence-collector/assets/golden-prompt-suite.current.js +22 -0
- package/skills/shared/evidence-collector/references/README.md +17 -0
- package/skills/shared/evidence-collector/references/cases.md +26 -0
- package/skills/shared/evidence-collector/references/prompt-template.md +18 -0
- package/skills/shared/evidence-collector/skill-spec.json +25 -0
- package/skills/shared/framework-guide/SKILL.md +28 -0
- package/skills/shared/framework-guide/assets/golden-prompt-suite.current.js +22 -0
- package/skills/shared/framework-guide/references/README.md +17 -0
- package/skills/shared/framework-guide/references/cases.md +26 -0
- package/skills/shared/framework-guide/references/prompt-template.md +18 -0
- package/skills/shared/framework-guide/skill-spec.json +24 -0
- package/rules/bundles.json +0 -358
- package/rules/coding/analysis.md +0 -27
- package/rules/coding/backend/cache-invalidation.md +0 -30
- package/rules/coding/backend/cache-keying.md +0 -30
- package/rules/coding/backend/cache.md +0 -37
- package/rules/coding/backend/database.md +0 -32
- package/rules/coding/backend/feign.md +0 -30
- package/rules/coding/backend/index.md +0 -42
- package/rules/coding/backend/query.md +0 -32
- package/rules/coding/backend/remote.md +0 -33
- package/rules/coding/backend/transaction-boundary.md +0 -30
- package/rules/coding/backend/transaction-rollback.md +0 -30
- package/rules/coding/backend/transaction.md +0 -38
- package/rules/coding/boundary.md +0 -25
- package/rules/coding/implementation.md +0 -26
- package/rules/coding/index.md +0 -38
- package/rules/coding/scaffold.md +0 -28
- package/rules/coding/testing.md +0 -29
- package/rules/coding/validation.md +0 -29
- package/rules/core/code-quality.md +0 -30
- package/rules/core/evidence.md +0 -26
- package/rules/core/interface.md +0 -26
- package/rules/core/iteration.md +0 -26
- package/rules/core/layer-boundary.md +0 -25
- package/rules/core/logging.md +0 -26
- package/rules/core/security.md +0 -26
- package/rules/core/task-boundary.md +0 -27
- package/rules/docs/api.md +0 -34
- package/rules/docs/capture-summary.md +0 -29
- package/rules/docs/capture.md +0 -34
- package/rules/docs/contract.md +0 -30
- package/rules/docs/decision-log.md +0 -32
- package/rules/docs/examples.md +0 -28
- package/rules/docs/index.md +0 -49
- package/rules/docs/reference.md +0 -32
- package/rules/overlays/pig-cloud/controller.md +0 -33
- package/rules/overlays/pig-cloud/dto-vo.md +0 -33
- package/rules/overlays/pig-cloud/entity.md +0 -32
- package/rules/overlays/pig-cloud/exception.md +0 -32
- package/rules/overlays/pig-cloud/layering.md +0 -31
- package/rules/overlays/pig-cloud/mapper.md +0 -32
- package/rules/overlays/pig-cloud/query-style.md +0 -32
- package/rules/overlays/pig-cloud/rest-response.md +0 -33
- package/rules/overlays/pig-cloud/service.md +0 -33
- package/rules/overlays/pig-cloud/transactions.md +0 -32
- package/rules/overlays/pig-cloud/validation.md +0 -33
- package/rules/product/acceptance.md +0 -25
- package/rules/product/briefing.md +0 -27
- package/rules/product/index.md +0 -36
- package/rules/product/intake.md +0 -27
- package/rules/product/modeling.md +0 -25
- package/rules/product/project-context.md +0 -29
- package/rules/review/code.md +0 -35
- package/rules/review/evidence.md +0 -31
- package/rules/review/index.md +0 -50
- package/rules/review/java.md +0 -42
- package/rules/review/performance.md +0 -38
- package/rules/review/rubric.md +0 -28
- package/rules/review/security.md +0 -38
- package/rules/review/ts.md +0 -33
- package/rules/review/vue.md +0 -33
- package/rules/skill-profile-map.json +0 -59
- package/rules/skill-profile-map.md +0 -29
- package/rules/workflow/handoff.md +0 -25
- package/rules/workflow/index.md +0 -37
- package/rules/workflow/refinement.md +0 -29
- package/rules/workflow/router.md +0 -25
- package/rules/workflow/selection.md +0 -25
- package/rules/workflow/stop.md +0 -25
- package/skills/api-contract-docs/SKILL.md +0 -77
- package/skills/business-fact-extraction/SKILL.md +0 -337
- package/skills/business-fact-extraction/scripts/write-knowledge-base.js +0 -228
- package/skills/code-review/SKILL.md +0 -136
- package/skills/code-review/references/findings-template.md +0 -51
- package/skills/code-review/references/performance-checklist.md +0 -213
- package/skills/code-review/references/rubric.md +0 -232
- package/skills/code-review/references/rules.md +0 -32
- package/skills/code-review/references/security-checklist.md +0 -178
- package/skills/code-review/references/stack-notes.md +0 -25
- package/skills/code-review/references/template-review.md +0 -39
- package/skills/code-review/scripts/lint-code-review.mjs +0 -431
- package/skills/domain-modeling/SKILL.md +0 -81
- package/skills/domain-modeling/references/README.md +0 -134
- package/skills/domain-modeling/references/distillation-checklist.md +0 -44
- package/skills/domain-modeling/references/test-cases-template.md +0 -128
- package/skills/environment-deploy/SKILL.md +0 -81
- package/skills/feature-build/SKILL.md +0 -122
- package/skills/feature-build/references/coding-checklist.md +0 -97
- package/skills/feature-build/references/comment-specification.md +0 -89
- package/skills/knowledge-capture/SKILL.md +0 -93
- package/skills/performance-audit/SKILL.md +0 -118
- package/skills/project-bootstrap/SKILL.md +0 -81
- package/skills/references/anti-rationalization.md +0 -144
- package/skills/references/business-fact-extraction.md +0 -415
- package/skills/references/engineering-delivery-method.md +0 -64
- package/skills/references/engineering-delivery-template.md +0 -81
- package/skills/references/golden-prompt-suite.js +0 -436
- package/skills/references/golden-prompt-suite.md +0 -33
- package/skills/references/project-requirement-alignment.md +0 -42
- package/skills/references/rule-loading-map.md +0 -117
- package/skills/references/skill-authoring-standard.md +0 -74
- package/skills/references/skill-boundary-template.md +0 -71
- package/skills/references/skill-enhanced-template.md +0 -103
- package/skills/references/skill-reference-matrix.md +0 -62
- package/skills/security-audit/SKILL.md +0 -119
- package/skills/spec-refinement/SKILL.md +0 -149
- package/skills/technical-design/SKILL.md +0 -106
- package/skills/technical-design/references/solid-checklist.md +0 -199
- package/skills/test-design/SKILL.md +0 -92
- package/skills/workflow-router/SKILL.md +0 -86
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
{
|
|
2
|
+
"group": "03-implementation",
|
|
3
|
+
"name": "database-change",
|
|
4
|
+
"description": "Apply schema, index, or data migration changes with care for compatibility.",
|
|
5
|
+
"title": "Database Change",
|
|
6
|
+
"purpose": "Execute database modifications safely and keep compatibility, rollback, and validation in view.",
|
|
7
|
+
"whenToUse": [
|
|
8
|
+
"A migration, schema change, index change, or data correction is required.",
|
|
9
|
+
"Data risk matters and must be controlled.",
|
|
10
|
+
"Backfill, rollout, or rollback planning is part of the work."
|
|
11
|
+
],
|
|
12
|
+
"how": [
|
|
13
|
+
"Plan the migration, backfill, and rollback path before touching persistent data.",
|
|
14
|
+
"Preserve forward and backward compatibility when the release path depends on it.",
|
|
15
|
+
"Validate the data shape, row counts, and any critical invariants after the change.",
|
|
16
|
+
"Keep application code changes separate unless the migration depends on them."
|
|
17
|
+
],
|
|
18
|
+
"whenToAvoid": [
|
|
19
|
+
"The requirements are still unclear or incomplete.",
|
|
20
|
+
"The task belongs to planning or validation instead of coding.",
|
|
21
|
+
"You only need documentation or evidence collection."
|
|
22
|
+
],
|
|
23
|
+
"caseId": "database-migration-safety",
|
|
24
|
+
"source": "bootstrap-from-skill-md"
|
|
25
|
+
}
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: frontend-development
|
|
3
|
+
description: Implement UI behavior, components, and client-side data flow changes.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Frontend Development
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Build frontend code that follows the approved UX, contract, and repository conventions.
|
|
11
|
+
|
|
12
|
+
## When to Use
|
|
13
|
+
|
|
14
|
+
- You need to change a UI, client-side workflow, or view-level state.
|
|
15
|
+
- The interface shape is already known.
|
|
16
|
+
- Accessibility, responsiveness, or client-side validation matters.
|
|
17
|
+
|
|
18
|
+
## How
|
|
19
|
+
|
|
20
|
+
- Implement the component or view change with the smallest safe UI surface area.
|
|
21
|
+
- Keep state, effects, and data fetching close to the feature boundary.
|
|
22
|
+
- Preserve layout, keyboard, and accessibility behavior unless the change requires otherwise.
|
|
23
|
+
- Validate the path in a browser or UI test before handing off.
|
|
24
|
+
|
|
25
|
+
## When to Avoid
|
|
26
|
+
|
|
27
|
+
- The requirements are still unclear or incomplete.
|
|
28
|
+
- The task belongs to planning or validation instead of coding.
|
|
29
|
+
- You only need documentation or evidence collection.
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
#!/usr/bin/env node
|
|
2
|
+
|
|
3
|
+
module.exports = {
|
|
4
|
+
"name": "frontend-development.golden-prompt-suite.current",
|
|
5
|
+
"skill": "frontend-development",
|
|
6
|
+
"description": "Implement UI behavior, components, and client-side data flow changes.",
|
|
7
|
+
"cases": [
|
|
8
|
+
{
|
|
9
|
+
"id": "frontend-implementation-plan",
|
|
10
|
+
"prompt": "This frontend page or component needs implementation. Outline the key steps and constraints.",
|
|
11
|
+
"expectedFirstSkill": "frontend-development",
|
|
12
|
+
"expectedOutput": [
|
|
13
|
+
"component changes",
|
|
14
|
+
"state handling",
|
|
15
|
+
"validation"
|
|
16
|
+
],
|
|
17
|
+
"expectedStop": "Do not change unrelated UI.",
|
|
18
|
+
"expectedHandoff": "unit-test"
|
|
19
|
+
}
|
|
20
|
+
]
|
|
21
|
+
};
|
|
22
|
+
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
# frontend-development References
|
|
2
|
+
|
|
3
|
+
This directory is local to the skill and is the only reference entry point for this skill.
|
|
4
|
+
|
|
5
|
+
## Contents
|
|
6
|
+
|
|
7
|
+
- `../skill-spec.json`: design source for regenerating this skill
|
|
8
|
+
- `prompt-template.md`: reusable prompt wording for this skill
|
|
9
|
+
- `cases.md`: human-readable replay case for this skill
|
|
10
|
+
- `../assets/golden-prompt-suite.current.js`: machine-readable local case fixture
|
|
11
|
+
|
|
12
|
+
## Guidance
|
|
13
|
+
|
|
14
|
+
- Keep skill-specific explanation here instead of pushing it into a stage-level shared library.
|
|
15
|
+
- Keep shared hard constraints in `../../../rules/index.md`.
|
|
16
|
+
- Keep canonical names unchanged.
|
|
17
|
+
- Skill summary: Implement UI behavior, components, and client-side data flow changes.
|
|
@@ -0,0 +1,26 @@
|
|
|
1
|
+
# frontend-development
|
|
2
|
+
|
|
3
|
+
## Replay Case
|
|
4
|
+
|
|
5
|
+
- Case: `frontend-implementation-plan`
|
|
6
|
+
- First skill: `frontend-development`
|
|
7
|
+
|
|
8
|
+
### Prompt
|
|
9
|
+
|
|
10
|
+
```text
|
|
11
|
+
This frontend page or component needs implementation. Outline the key steps and constraints.
|
|
12
|
+
```
|
|
13
|
+
|
|
14
|
+
### Expected Output
|
|
15
|
+
|
|
16
|
+
- component changes
|
|
17
|
+
- state handling
|
|
18
|
+
- validation
|
|
19
|
+
|
|
20
|
+
### Stop Rule
|
|
21
|
+
|
|
22
|
+
- Do not change unrelated UI.
|
|
23
|
+
|
|
24
|
+
### Handoff
|
|
25
|
+
|
|
26
|
+
- unit-test
|
|
@@ -0,0 +1,18 @@
|
|
|
1
|
+
# frontend-development
|
|
2
|
+
|
|
3
|
+
## Prompt Template
|
|
4
|
+
|
|
5
|
+
```text
|
|
6
|
+
You are handling the frontend-development skill.
|
|
7
|
+
Goal: Implement UI behavior, components, and client-side data flow changes
|
|
8
|
+
Inputs: the request, supporting evidence, constraints, and existing repository facts.
|
|
9
|
+
Output contract: component changes, state handling, validation.
|
|
10
|
+
Stop rule: Do not change unrelated UI.
|
|
11
|
+
Handoff: unit-test.
|
|
12
|
+
Keep the response scoped to this stage and do not absorb the work of the next stage.
|
|
13
|
+
```
|
|
14
|
+
|
|
15
|
+
## Notes
|
|
16
|
+
|
|
17
|
+
- Keep the template aligned with the local replay case and local assets.
|
|
18
|
+
- Read `cases.md` for the matching replay example.
|
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
{
|
|
2
|
+
"group": "03-implementation",
|
|
3
|
+
"name": "frontend-development",
|
|
4
|
+
"description": "Implement UI behavior, components, and client-side data flow changes.",
|
|
5
|
+
"title": "Frontend Development",
|
|
6
|
+
"purpose": "Build frontend code that follows the approved UX, contract, and repository conventions.",
|
|
7
|
+
"whenToUse": [
|
|
8
|
+
"You need to change a UI, client-side workflow, or view-level state.",
|
|
9
|
+
"The interface shape is already known.",
|
|
10
|
+
"Accessibility, responsiveness, or client-side validation matters."
|
|
11
|
+
],
|
|
12
|
+
"how": [
|
|
13
|
+
"Implement the component or view change with the smallest safe UI surface area.",
|
|
14
|
+
"Keep state, effects, and data fetching close to the feature boundary.",
|
|
15
|
+
"Preserve layout, keyboard, and accessibility behavior unless the change requires otherwise.",
|
|
16
|
+
"Validate the path in a browser or UI test before handing off."
|
|
17
|
+
],
|
|
18
|
+
"whenToAvoid": [
|
|
19
|
+
"The requirements are still unclear or incomplete.",
|
|
20
|
+
"The task belongs to planning or validation instead of coding.",
|
|
21
|
+
"You only need documentation or evidence collection."
|
|
22
|
+
],
|
|
23
|
+
"caseId": "frontend-implementation-plan",
|
|
24
|
+
"source": "bootstrap-from-skill-md"
|
|
25
|
+
}
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: code-review
|
|
3
|
+
description: Review a diff or pull request for correctness, maintainability, and risk.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Code Review
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Assess the change for bugs, regressions, clarity, and maintainability before merge.
|
|
11
|
+
|
|
12
|
+
## When to Use
|
|
13
|
+
|
|
14
|
+
- You need review findings for a completed diff.
|
|
15
|
+
- The task is to judge quality, not implement code.
|
|
16
|
+
- You need findings that are tied to evidence in the patch.
|
|
17
|
+
|
|
18
|
+
## How
|
|
19
|
+
|
|
20
|
+
- Inspect the diff for correctness, maintainability, and regression risk, then report actionable findings.
|
|
21
|
+
- Anchor each finding to a concrete location, impact, and fix path.
|
|
22
|
+
- Rank findings by severity and keep the report focused on the submitted change.
|
|
23
|
+
- If evidence is weak, collect more proof before calling it a finding.
|
|
24
|
+
|
|
25
|
+
## When to Avoid
|
|
26
|
+
|
|
27
|
+
- You still need design or implementation work.
|
|
28
|
+
- The task is about a different validation subtype.
|
|
29
|
+
- You need to rewrite code instead of reviewing it.
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
#!/usr/bin/env node
|
|
2
|
+
|
|
3
|
+
module.exports = {
|
|
4
|
+
"name": "code-review.golden-prompt-suite.current",
|
|
5
|
+
"skill": "code-review",
|
|
6
|
+
"description": "Review a diff or pull request for correctness, maintainability, and risk.",
|
|
7
|
+
"cases": [
|
|
8
|
+
{
|
|
9
|
+
"id": "review-diff-correctness",
|
|
10
|
+
"prompt": "Review this diff for correctness, maintainability, and regression risk.",
|
|
11
|
+
"expectedFirstSkill": "code-review",
|
|
12
|
+
"expectedOutput": [
|
|
13
|
+
"findings",
|
|
14
|
+
"severity",
|
|
15
|
+
"fix recommendations"
|
|
16
|
+
],
|
|
17
|
+
"expectedStop": "Do not refactor the code inside the review step.",
|
|
18
|
+
"expectedHandoff": "evidence-collector"
|
|
19
|
+
}
|
|
20
|
+
]
|
|
21
|
+
};
|
|
22
|
+
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
# code-review References
|
|
2
|
+
|
|
3
|
+
This directory is local to the skill and is the only reference entry point for this skill.
|
|
4
|
+
|
|
5
|
+
## Contents
|
|
6
|
+
|
|
7
|
+
- `../skill-spec.json`: design source for regenerating this skill
|
|
8
|
+
- `prompt-template.md`: reusable prompt wording for this skill
|
|
9
|
+
- `cases.md`: human-readable replay case for this skill
|
|
10
|
+
- `../assets/golden-prompt-suite.current.js`: machine-readable local case fixture
|
|
11
|
+
|
|
12
|
+
## Guidance
|
|
13
|
+
|
|
14
|
+
- Keep skill-specific explanation here instead of pushing it into a stage-level shared library.
|
|
15
|
+
- Keep shared hard constraints in `../../../rules/index.md`.
|
|
16
|
+
- Keep canonical names unchanged.
|
|
17
|
+
- Skill summary: Review a diff or pull request for correctness, maintainability, and risk.
|
|
@@ -0,0 +1,26 @@
|
|
|
1
|
+
# code-review
|
|
2
|
+
|
|
3
|
+
## Replay Case
|
|
4
|
+
|
|
5
|
+
- Case: `review-diff-correctness`
|
|
6
|
+
- First skill: `code-review`
|
|
7
|
+
|
|
8
|
+
### Prompt
|
|
9
|
+
|
|
10
|
+
```text
|
|
11
|
+
Review this diff for correctness, maintainability, and regression risk.
|
|
12
|
+
```
|
|
13
|
+
|
|
14
|
+
### Expected Output
|
|
15
|
+
|
|
16
|
+
- findings
|
|
17
|
+
- severity
|
|
18
|
+
- fix recommendations
|
|
19
|
+
|
|
20
|
+
### Stop Rule
|
|
21
|
+
|
|
22
|
+
- Do not refactor the code inside the review step.
|
|
23
|
+
|
|
24
|
+
### Handoff
|
|
25
|
+
|
|
26
|
+
- evidence-collector
|
|
@@ -0,0 +1,18 @@
|
|
|
1
|
+
# code-review
|
|
2
|
+
|
|
3
|
+
## Prompt Template
|
|
4
|
+
|
|
5
|
+
```text
|
|
6
|
+
You are handling the code-review skill.
|
|
7
|
+
Goal: Review a diff or pull request for correctness, maintainability, and risk
|
|
8
|
+
Inputs: the request, supporting evidence, constraints, and existing repository facts.
|
|
9
|
+
Output contract: findings, severity, fix recommendations.
|
|
10
|
+
Stop rule: Do not refactor the code inside the review step.
|
|
11
|
+
Handoff: evidence-collector.
|
|
12
|
+
Keep the response scoped to this stage and do not absorb the work of the next stage.
|
|
13
|
+
```
|
|
14
|
+
|
|
15
|
+
## Notes
|
|
16
|
+
|
|
17
|
+
- Keep the template aligned with the local replay case and local assets.
|
|
18
|
+
- Read `cases.md` for the matching replay example.
|
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
{
|
|
2
|
+
"group": "04-validation",
|
|
3
|
+
"name": "code-review",
|
|
4
|
+
"description": "Review a diff or pull request for correctness, maintainability, and risk.",
|
|
5
|
+
"title": "Code Review",
|
|
6
|
+
"purpose": "Assess the change for bugs, regressions, clarity, and maintainability before merge.",
|
|
7
|
+
"whenToUse": [
|
|
8
|
+
"You need review findings for a completed diff.",
|
|
9
|
+
"The task is to judge quality, not implement code.",
|
|
10
|
+
"You need findings that are tied to evidence in the patch."
|
|
11
|
+
],
|
|
12
|
+
"how": [
|
|
13
|
+
"Inspect the diff for correctness, maintainability, and regression risk, then report actionable findings.",
|
|
14
|
+
"Anchor each finding to a concrete location, impact, and fix path.",
|
|
15
|
+
"Rank findings by severity and keep the report focused on the submitted change.",
|
|
16
|
+
"If evidence is weak, collect more proof before calling it a finding."
|
|
17
|
+
],
|
|
18
|
+
"whenToAvoid": [
|
|
19
|
+
"You still need design or implementation work.",
|
|
20
|
+
"The task is about a different validation subtype.",
|
|
21
|
+
"You need to rewrite code instead of reviewing it."
|
|
22
|
+
],
|
|
23
|
+
"caseId": "review-diff-correctness",
|
|
24
|
+
"source": "bootstrap-from-skill-md"
|
|
25
|
+
}
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: performance-review
|
|
3
|
+
description: Review changes for latency, throughput, memory, and load risk.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Performance Review
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Assess whether the change introduces a measurable performance problem or hot path risk.
|
|
11
|
+
|
|
12
|
+
## When to Use
|
|
13
|
+
|
|
14
|
+
- The change may affect response time or resource usage.
|
|
15
|
+
- You need performance-oriented findings.
|
|
16
|
+
- The task mentions latency, throughput, memory, load, or hot paths.
|
|
17
|
+
|
|
18
|
+
## How
|
|
19
|
+
|
|
20
|
+
- Trace hot paths, expensive calls, and resource pressure before recommending optimizations.
|
|
21
|
+
- Name the concrete bottleneck path and what resource it stresses.
|
|
22
|
+
- Estimate the likely impact and keep the fix suggestion small.
|
|
23
|
+
- Avoid vague "might be slow" comments without a measurable path.
|
|
24
|
+
|
|
25
|
+
## When to Avoid
|
|
26
|
+
|
|
27
|
+
- You still need design or implementation work.
|
|
28
|
+
- The task is about a different validation subtype.
|
|
29
|
+
- You need to rewrite code instead of reviewing it.
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
#!/usr/bin/env node
|
|
2
|
+
|
|
3
|
+
module.exports = {
|
|
4
|
+
"name": "performance-review.golden-prompt-suite.current",
|
|
5
|
+
"skill": "performance-review",
|
|
6
|
+
"description": "Review changes for latency, throughput, memory, and load risk.",
|
|
7
|
+
"cases": [
|
|
8
|
+
{
|
|
9
|
+
"id": "performance-bottleneck-check",
|
|
10
|
+
"prompt": "Check whether this change introduces latency, throughput, or resource pressure risks.",
|
|
11
|
+
"expectedFirstSkill": "performance-review",
|
|
12
|
+
"expectedOutput": [
|
|
13
|
+
"performance findings",
|
|
14
|
+
"bottleneck map",
|
|
15
|
+
"optimization advice"
|
|
16
|
+
],
|
|
17
|
+
"expectedStop": "Do not rewrite code.",
|
|
18
|
+
"expectedHandoff": "regression-check"
|
|
19
|
+
}
|
|
20
|
+
]
|
|
21
|
+
};
|
|
22
|
+
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
# performance-review References
|
|
2
|
+
|
|
3
|
+
This directory is local to the skill and is the only reference entry point for this skill.
|
|
4
|
+
|
|
5
|
+
## Contents
|
|
6
|
+
|
|
7
|
+
- `../skill-spec.json`: design source for regenerating this skill
|
|
8
|
+
- `prompt-template.md`: reusable prompt wording for this skill
|
|
9
|
+
- `cases.md`: human-readable replay case for this skill
|
|
10
|
+
- `../assets/golden-prompt-suite.current.js`: machine-readable local case fixture
|
|
11
|
+
|
|
12
|
+
## Guidance
|
|
13
|
+
|
|
14
|
+
- Keep skill-specific explanation here instead of pushing it into a stage-level shared library.
|
|
15
|
+
- Keep shared hard constraints in `../../../rules/index.md`.
|
|
16
|
+
- Keep canonical names unchanged.
|
|
17
|
+
- Skill summary: Review changes for latency, throughput, memory, and load risk.
|
|
@@ -0,0 +1,26 @@
|
|
|
1
|
+
# performance-review
|
|
2
|
+
|
|
3
|
+
## Replay Case
|
|
4
|
+
|
|
5
|
+
- Case: `performance-bottleneck-check`
|
|
6
|
+
- First skill: `performance-review`
|
|
7
|
+
|
|
8
|
+
### Prompt
|
|
9
|
+
|
|
10
|
+
```text
|
|
11
|
+
Check whether this change introduces latency, throughput, or resource pressure risks.
|
|
12
|
+
```
|
|
13
|
+
|
|
14
|
+
### Expected Output
|
|
15
|
+
|
|
16
|
+
- performance findings
|
|
17
|
+
- bottleneck map
|
|
18
|
+
- optimization advice
|
|
19
|
+
|
|
20
|
+
### Stop Rule
|
|
21
|
+
|
|
22
|
+
- Do not rewrite code.
|
|
23
|
+
|
|
24
|
+
### Handoff
|
|
25
|
+
|
|
26
|
+
- regression-check
|
|
@@ -0,0 +1,18 @@
|
|
|
1
|
+
# performance-review
|
|
2
|
+
|
|
3
|
+
## Prompt Template
|
|
4
|
+
|
|
5
|
+
```text
|
|
6
|
+
You are handling the performance-review skill.
|
|
7
|
+
Goal: Review changes for latency, throughput, memory, and load risk
|
|
8
|
+
Inputs: the request, supporting evidence, constraints, and existing repository facts.
|
|
9
|
+
Output contract: performance findings, bottleneck map, optimization advice.
|
|
10
|
+
Stop rule: Do not rewrite code.
|
|
11
|
+
Handoff: regression-check.
|
|
12
|
+
Keep the response scoped to this stage and do not absorb the work of the next stage.
|
|
13
|
+
```
|
|
14
|
+
|
|
15
|
+
## Notes
|
|
16
|
+
|
|
17
|
+
- Keep the template aligned with the local replay case and local assets.
|
|
18
|
+
- Read `cases.md` for the matching replay example.
|
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
{
|
|
2
|
+
"group": "04-validation",
|
|
3
|
+
"name": "performance-review",
|
|
4
|
+
"description": "Review changes for latency, throughput, memory, and load risk.",
|
|
5
|
+
"title": "Performance Review",
|
|
6
|
+
"purpose": "Assess whether the change introduces a measurable performance problem or hot path risk.",
|
|
7
|
+
"whenToUse": [
|
|
8
|
+
"The change may affect response time or resource usage.",
|
|
9
|
+
"You need performance-oriented findings.",
|
|
10
|
+
"The task mentions latency, throughput, memory, load, or hot paths."
|
|
11
|
+
],
|
|
12
|
+
"how": [
|
|
13
|
+
"Trace hot paths, expensive calls, and resource pressure before recommending optimizations.",
|
|
14
|
+
"Name the concrete bottleneck path and what resource it stresses.",
|
|
15
|
+
"Estimate the likely impact and keep the fix suggestion small.",
|
|
16
|
+
"Avoid vague \"might be slow\" comments without a measurable path."
|
|
17
|
+
],
|
|
18
|
+
"whenToAvoid": [
|
|
19
|
+
"You still need design or implementation work.",
|
|
20
|
+
"The task is about a different validation subtype.",
|
|
21
|
+
"You need to rewrite code instead of reviewing it."
|
|
22
|
+
],
|
|
23
|
+
"caseId": "performance-bottleneck-check",
|
|
24
|
+
"source": "bootstrap-from-skill-md"
|
|
25
|
+
}
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: regression-check
|
|
3
|
+
description: Verify that the change does not break existing behavior or release assumptions.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Regression Check
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Confirm the change is safe for surrounding behaviors, flows, and release expectations.
|
|
11
|
+
|
|
12
|
+
## When to Use
|
|
13
|
+
|
|
14
|
+
- You need a final safety pass after implementation.
|
|
15
|
+
- The change could affect unrelated paths.
|
|
16
|
+
- Nearby flows, defaults, or release assumptions could change.
|
|
17
|
+
|
|
18
|
+
## How
|
|
19
|
+
|
|
20
|
+
- Compare the change against nearby behaviors, then confirm the surrounding flows still hold.
|
|
21
|
+
- Check the happy path, the fallback path, and the failure path around the change.
|
|
22
|
+
- Confirm any shared contract, flag, or default still behaves as expected.
|
|
23
|
+
- Note anything that should be monitored after release.
|
|
24
|
+
|
|
25
|
+
## When to Avoid
|
|
26
|
+
|
|
27
|
+
- You still need design or implementation work.
|
|
28
|
+
- The task is about a different validation subtype.
|
|
29
|
+
- You need to rewrite code instead of reviewing it.
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
#!/usr/bin/env node
|
|
2
|
+
|
|
3
|
+
module.exports = {
|
|
4
|
+
"name": "regression-check.golden-prompt-suite.current",
|
|
5
|
+
"skill": "regression-check",
|
|
6
|
+
"description": "Verify that the change does not break existing behavior or release assumptions.",
|
|
7
|
+
"cases": [
|
|
8
|
+
{
|
|
9
|
+
"id": "regression-behavior-scan",
|
|
10
|
+
"prompt": "The change is complete. Check whether related behavior broke.",
|
|
11
|
+
"expectedFirstSkill": "regression-check",
|
|
12
|
+
"expectedOutput": [
|
|
13
|
+
"regression matrix",
|
|
14
|
+
"safety check",
|
|
15
|
+
"remaining risk"
|
|
16
|
+
],
|
|
17
|
+
"expectedStop": "Do not change production code.",
|
|
18
|
+
"expectedHandoff": "release-check"
|
|
19
|
+
}
|
|
20
|
+
]
|
|
21
|
+
};
|
|
22
|
+
|
|
@@ -0,0 +1,17 @@
|
|
|
1
|
+
# regression-check References
|
|
2
|
+
|
|
3
|
+
This directory is local to the skill and is the only reference entry point for this skill.
|
|
4
|
+
|
|
5
|
+
## Contents
|
|
6
|
+
|
|
7
|
+
- `../skill-spec.json`: design source for regenerating this skill
|
|
8
|
+
- `prompt-template.md`: reusable prompt wording for this skill
|
|
9
|
+
- `cases.md`: human-readable replay case for this skill
|
|
10
|
+
- `../assets/golden-prompt-suite.current.js`: machine-readable local case fixture
|
|
11
|
+
|
|
12
|
+
## Guidance
|
|
13
|
+
|
|
14
|
+
- Keep skill-specific explanation here instead of pushing it into a stage-level shared library.
|
|
15
|
+
- Keep shared hard constraints in `../../../rules/index.md`.
|
|
16
|
+
- Keep canonical names unchanged.
|
|
17
|
+
- Skill summary: Verify that the change does not break existing behavior or release assumptions.
|
|
@@ -0,0 +1,26 @@
|
|
|
1
|
+
# regression-check
|
|
2
|
+
|
|
3
|
+
## Replay Case
|
|
4
|
+
|
|
5
|
+
- Case: `regression-behavior-scan`
|
|
6
|
+
- First skill: `regression-check`
|
|
7
|
+
|
|
8
|
+
### Prompt
|
|
9
|
+
|
|
10
|
+
```text
|
|
11
|
+
The change is complete. Check whether related behavior broke.
|
|
12
|
+
```
|
|
13
|
+
|
|
14
|
+
### Expected Output
|
|
15
|
+
|
|
16
|
+
- regression matrix
|
|
17
|
+
- safety check
|
|
18
|
+
- remaining risk
|
|
19
|
+
|
|
20
|
+
### Stop Rule
|
|
21
|
+
|
|
22
|
+
- Do not change production code.
|
|
23
|
+
|
|
24
|
+
### Handoff
|
|
25
|
+
|
|
26
|
+
- release-check
|
|
@@ -0,0 +1,18 @@
|
|
|
1
|
+
# regression-check
|
|
2
|
+
|
|
3
|
+
## Prompt Template
|
|
4
|
+
|
|
5
|
+
```text
|
|
6
|
+
You are handling the regression-check skill.
|
|
7
|
+
Goal: Verify that the change does not break existing behavior or release assumptions
|
|
8
|
+
Inputs: the request, supporting evidence, constraints, and existing repository facts.
|
|
9
|
+
Output contract: regression matrix, safety check, remaining risk.
|
|
10
|
+
Stop rule: Do not change production code.
|
|
11
|
+
Handoff: release-check.
|
|
12
|
+
Keep the response scoped to this stage and do not absorb the work of the next stage.
|
|
13
|
+
```
|
|
14
|
+
|
|
15
|
+
## Notes
|
|
16
|
+
|
|
17
|
+
- Keep the template aligned with the local replay case and local assets.
|
|
18
|
+
- Read `cases.md` for the matching replay example.
|
|
@@ -0,0 +1,25 @@
|
|
|
1
|
+
{
|
|
2
|
+
"group": "04-validation",
|
|
3
|
+
"name": "regression-check",
|
|
4
|
+
"description": "Verify that the change does not break existing behavior or release assumptions.",
|
|
5
|
+
"title": "Regression Check",
|
|
6
|
+
"purpose": "Confirm the change is safe for surrounding behaviors, flows, and release expectations.",
|
|
7
|
+
"whenToUse": [
|
|
8
|
+
"You need a final safety pass after implementation.",
|
|
9
|
+
"The change could affect unrelated paths.",
|
|
10
|
+
"Nearby flows, defaults, or release assumptions could change."
|
|
11
|
+
],
|
|
12
|
+
"how": [
|
|
13
|
+
"Compare the change against nearby behaviors, then confirm the surrounding flows still hold.",
|
|
14
|
+
"Check the happy path, the fallback path, and the failure path around the change.",
|
|
15
|
+
"Confirm any shared contract, flag, or default still behaves as expected.",
|
|
16
|
+
"Note anything that should be monitored after release."
|
|
17
|
+
],
|
|
18
|
+
"whenToAvoid": [
|
|
19
|
+
"You still need design or implementation work.",
|
|
20
|
+
"The task is about a different validation subtype.",
|
|
21
|
+
"You need to rewrite code instead of reviewing it."
|
|
22
|
+
],
|
|
23
|
+
"caseId": "regression-behavior-scan",
|
|
24
|
+
"source": "bootstrap-from-skill-md"
|
|
25
|
+
}
|
|
@@ -0,0 +1,29 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: security-review
|
|
3
|
+
description: Review changes for authorization, data exposure, secrets, and abuse risk.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Security Review
|
|
7
|
+
|
|
8
|
+
## Purpose
|
|
9
|
+
|
|
10
|
+
Find security weaknesses and recommend the smallest safe mitigation.
|
|
11
|
+
|
|
12
|
+
## When to Use
|
|
13
|
+
|
|
14
|
+
- The change touches auth, secrets, data access, user input, or trust boundaries.
|
|
15
|
+
- You need a security-specific review.
|
|
16
|
+
- The risk is about exposure, privilege, or abuse rather than correctness alone.
|
|
17
|
+
|
|
18
|
+
## How
|
|
19
|
+
|
|
20
|
+
- Check auth, data exposure, input handling, and trust boundaries before merge.
|
|
21
|
+
- Trace how untrusted input reaches sensitive operations or secrets.
|
|
22
|
+
- State the attack path, impacted asset, and minimum fix needed to close it.
|
|
23
|
+
- Keep findings focused on concrete exploitability or exposure.
|
|
24
|
+
|
|
25
|
+
## When to Avoid
|
|
26
|
+
|
|
27
|
+
- You still need design or implementation work.
|
|
28
|
+
- The task is about a different validation subtype.
|
|
29
|
+
- You need to rewrite code instead of reviewing it.
|
|
@@ -0,0 +1,22 @@
|
|
|
1
|
+
#!/usr/bin/env node
|
|
2
|
+
|
|
3
|
+
module.exports = {
|
|
4
|
+
"name": "security-review.golden-prompt-suite.current",
|
|
5
|
+
"skill": "security-review",
|
|
6
|
+
"description": "Review changes for authorization, data exposure, secrets, and abuse risk.",
|
|
7
|
+
"cases": [
|
|
8
|
+
{
|
|
9
|
+
"id": "security-risk-check",
|
|
10
|
+
"prompt": "Check whether this change introduces auth, injection, or data exposure risks.",
|
|
11
|
+
"expectedFirstSkill": "security-review",
|
|
12
|
+
"expectedOutput": [
|
|
13
|
+
"security findings",
|
|
14
|
+
"risk level",
|
|
15
|
+
"mitigation"
|
|
16
|
+
],
|
|
17
|
+
"expectedStop": "Do not rewrite code.",
|
|
18
|
+
"expectedHandoff": "regression-check"
|
|
19
|
+
}
|
|
20
|
+
]
|
|
21
|
+
};
|
|
22
|
+
|