@sniper.ai/core 2.0.0 → 3.0.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/README.md +88 -98
- package/agents/analyst.md +30 -0
- package/agents/architect.md +36 -0
- package/agents/backend-dev.md +43 -0
- package/agents/code-reviewer.md +72 -0
- package/agents/frontend-dev.md +43 -0
- package/agents/fullstack-dev.md +44 -0
- package/agents/gate-reviewer.md +62 -0
- package/agents/lead-orchestrator.md +51 -0
- package/agents/product-manager.md +38 -0
- package/agents/qa-engineer.md +37 -0
- package/agents/retro-analyst.md +98 -0
- package/checklists/discover.yaml +23 -0
- package/checklists/implement.yaml +28 -0
- package/checklists/ingest-document.yaml +18 -0
- package/checklists/ingest-extract.yaml +13 -0
- package/checklists/ingest-scan.yaml +18 -0
- package/checklists/multi-faceted-review.yaml +56 -0
- package/checklists/plan.yaml +36 -0
- package/checklists/refactor-analyze.yaml +18 -0
- package/checklists/review.yaml +28 -0
- package/claude-md.template +42 -0
- package/config.template.yaml +156 -0
- package/hooks/settings-hooks.json +31 -0
- package/hooks/signal-hooks.json +11 -0
- package/package.json +23 -5
- package/personas/cognitive/devils-advocate.md +24 -0
- package/personas/cognitive/performance-focused.md +23 -0
- package/personas/cognitive/security-first.md +24 -0
- package/protocols/explore.yaml +18 -0
- package/protocols/feature.yaml +45 -0
- package/protocols/full.yaml +63 -0
- package/protocols/hotfix.yaml +19 -0
- package/protocols/ingest.yaml +39 -0
- package/protocols/patch.yaml +30 -0
- package/protocols/refactor.yaml +41 -0
- package/schemas/checkpoint.schema.yaml +133 -0
- package/schemas/cost.schema.yaml +97 -0
- package/schemas/dependency-graph.schema.yaml +37 -0
- package/schemas/gate-result.schema.yaml +101 -0
- package/schemas/knowledge-manifest.schema.yaml +39 -0
- package/schemas/live-status.schema.yaml +122 -0
- package/schemas/protocol.schema.yaml +100 -0
- package/schemas/retro.schema.yaml +95 -0
- package/schemas/revert-plan.schema.yaml +40 -0
- package/schemas/signal.schema.yaml +39 -0
- package/schemas/velocity.schema.yaml +52 -0
- package/schemas/workspace-lock.schema.yaml +34 -0
- package/schemas/workspace.schema.yaml +82 -0
- package/skills/sniper-flow/SKILL.md +243 -0
- package/skills/sniper-flow-headless/SKILL.md +105 -0
- package/skills/sniper-init/SKILL.md +103 -0
- package/skills/sniper-review/SKILL.md +49 -0
- package/skills/sniper-status/SKILL.md +79 -0
- package/templates/architecture.md +23 -0
- package/templates/checkpoint.yaml +27 -0
- package/templates/codebase-overview.md +19 -0
- package/templates/cost.yaml +23 -0
- package/templates/custom-protocol.yaml +98 -0
- package/templates/knowledge-manifest.yaml +32 -0
- package/templates/live-status.yaml +26 -0
- package/templates/multi-faceted-review-report.md +28 -0
- package/templates/review-report.md +25 -0
- package/templates/signal-record.yaml +37 -0
- package/templates/spec.md +28 -0
- package/templates/story.md +19 -0
- package/templates/velocity.yaml +9 -0
- package/templates/workspace-config.yaml +44 -0
- package/framework/checklists/code-review.md +0 -33
- package/framework/checklists/debug-review.md +0 -34
- package/framework/checklists/discover-review.md +0 -33
- package/framework/checklists/doc-review.md +0 -39
- package/framework/checklists/feature-review.md +0 -42
- package/framework/checklists/ingest-review.md +0 -42
- package/framework/checklists/memory-review.md +0 -30
- package/framework/checklists/perf-review.md +0 -33
- package/framework/checklists/plan-review.md +0 -52
- package/framework/checklists/refactor-review.md +0 -33
- package/framework/checklists/security-review.md +0 -34
- package/framework/checklists/sprint-review.md +0 -41
- package/framework/checklists/story-review.md +0 -30
- package/framework/checklists/test-review.md +0 -32
- package/framework/checklists/workspace-review.md +0 -34
- package/framework/claude-md.template +0 -37
- package/framework/commands/sniper-audit.md +0 -1549
- package/framework/commands/sniper-compose.md +0 -323
- package/framework/commands/sniper-debug.md +0 -337
- package/framework/commands/sniper-discover.md +0 -423
- package/framework/commands/sniper-doc.md +0 -441
- package/framework/commands/sniper-feature.md +0 -515
- package/framework/commands/sniper-ingest.md +0 -506
- package/framework/commands/sniper-init.md +0 -388
- package/framework/commands/sniper-memory.md +0 -219
- package/framework/commands/sniper-plan.md +0 -630
- package/framework/commands/sniper-review.md +0 -369
- package/framework/commands/sniper-solve.md +0 -408
- package/framework/commands/sniper-sprint.md +0 -716
- package/framework/commands/sniper-status.md +0 -481
- package/framework/commands/sniper-workspace-feature.md +0 -267
- package/framework/commands/sniper-workspace-init.md +0 -252
- package/framework/commands/sniper-workspace-status.md +0 -112
- package/framework/commands/sniper-workspace-validate.md +0 -138
- package/framework/config.template.yaml +0 -196
- package/framework/personas/cognitive/devils-advocate.md +0 -30
- package/framework/personas/cognitive/mentor-explainer.md +0 -29
- package/framework/personas/cognitive/performance-focused.md +0 -30
- package/framework/personas/cognitive/security-first.md +0 -29
- package/framework/personas/cognitive/systems-thinker.md +0 -29
- package/framework/personas/cognitive/user-empathetic.md +0 -29
- package/framework/personas/domain/.gitkeep +0 -0
- package/framework/personas/process/analyst.md +0 -29
- package/framework/personas/process/architect.md +0 -30
- package/framework/personas/process/architecture-cartographer.md +0 -25
- package/framework/personas/process/code-archaeologist.md +0 -22
- package/framework/personas/process/code-investigator.md +0 -29
- package/framework/personas/process/code-reviewer.md +0 -26
- package/framework/personas/process/contract-designer.md +0 -31
- package/framework/personas/process/convention-miner.md +0 -27
- package/framework/personas/process/coverage-analyst.md +0 -24
- package/framework/personas/process/developer.md +0 -32
- package/framework/personas/process/doc-analyst.md +0 -63
- package/framework/personas/process/doc-reviewer.md +0 -62
- package/framework/personas/process/doc-writer.md +0 -42
- package/framework/personas/process/flake-hunter.md +0 -30
- package/framework/personas/process/impact-analyst.md +0 -23
- package/framework/personas/process/integration-validator.md +0 -29
- package/framework/personas/process/log-analyst.md +0 -22
- package/framework/personas/process/migration-architect.md +0 -24
- package/framework/personas/process/perf-profiler.md +0 -27
- package/framework/personas/process/product-manager.md +0 -32
- package/framework/personas/process/qa-engineer.md +0 -31
- package/framework/personas/process/release-manager.md +0 -23
- package/framework/personas/process/retro-analyst.md +0 -30
- package/framework/personas/process/scrum-master.md +0 -31
- package/framework/personas/process/threat-modeler.md +0 -30
- package/framework/personas/process/triage-lead.md +0 -23
- package/framework/personas/process/ux-designer.md +0 -31
- package/framework/personas/process/vuln-scanner.md +0 -27
- package/framework/personas/process/workspace-orchestrator.md +0 -30
- package/framework/personas/technical/ai-ml.md +0 -33
- package/framework/personas/technical/api-design.md +0 -32
- package/framework/personas/technical/backend.md +0 -32
- package/framework/personas/technical/database.md +0 -32
- package/framework/personas/technical/frontend.md +0 -33
- package/framework/personas/technical/infrastructure.md +0 -32
- package/framework/personas/technical/security.md +0 -34
- package/framework/settings.template.json +0 -6
- package/framework/spawn-prompts/_template.md +0 -25
- package/framework/teams/debug.yaml +0 -56
- package/framework/teams/discover.yaml +0 -57
- package/framework/teams/doc.yaml +0 -76
- package/framework/teams/feature-plan.yaml +0 -61
- package/framework/teams/ingest.yaml +0 -85
- package/framework/teams/perf.yaml +0 -33
- package/framework/teams/plan.yaml +0 -86
- package/framework/teams/refactor.yaml +0 -34
- package/framework/teams/retro.yaml +0 -30
- package/framework/teams/review-pr.yaml +0 -73
- package/framework/teams/review-release.yaml +0 -70
- package/framework/teams/security.yaml +0 -59
- package/framework/teams/solve.yaml +0 -48
- package/framework/teams/sprint.yaml +0 -68
- package/framework/teams/test.yaml +0 -59
- package/framework/teams/workspace-feature.yaml +0 -69
- package/framework/teams/workspace-validation.yaml +0 -27
- package/framework/templates/arch-delta.md +0 -74
- package/framework/templates/architecture.md +0 -95
- package/framework/templates/brief.md +0 -73
- package/framework/templates/bug-report.md +0 -55
- package/framework/templates/contract-validation-report.md +0 -68
- package/framework/templates/contract.yaml +0 -60
- package/framework/templates/conventions.md +0 -59
- package/framework/templates/coverage-report.md +0 -67
- package/framework/templates/doc-api.md +0 -53
- package/framework/templates/doc-guide.md +0 -35
- package/framework/templates/doc-readme.md +0 -49
- package/framework/templates/epic.md +0 -47
- package/framework/templates/feature-brief.md +0 -54
- package/framework/templates/feature-spec.md +0 -53
- package/framework/templates/flaky-report.md +0 -64
- package/framework/templates/investigation.md +0 -49
- package/framework/templates/memory-anti-pattern.yaml +0 -16
- package/framework/templates/memory-convention.yaml +0 -17
- package/framework/templates/memory-decision.yaml +0 -16
- package/framework/templates/migration-plan.md +0 -47
- package/framework/templates/optimization-plan.md +0 -59
- package/framework/templates/performance-profile.md +0 -64
- package/framework/templates/personas.md +0 -118
- package/framework/templates/postmortem.md +0 -69
- package/framework/templates/pr-review.md +0 -50
- package/framework/templates/prd.md +0 -92
- package/framework/templates/refactor-scope.md +0 -52
- package/framework/templates/release-readiness.md +0 -66
- package/framework/templates/retro.yaml +0 -44
- package/framework/templates/risks.md +0 -64
- package/framework/templates/security.md +0 -111
- package/framework/templates/sprint-review.md +0 -32
- package/framework/templates/story.md +0 -53
- package/framework/templates/threat-model.md +0 -71
- package/framework/templates/ux-spec.md +0 -71
- package/framework/templates/vulnerability-report.md +0 -56
- package/framework/templates/workspace-brief.md +0 -52
- package/framework/templates/workspace-plan.md +0 -50
- package/framework/workflows/discover-only.md +0 -39
- package/framework/workflows/full-lifecycle.md +0 -56
- package/framework/workflows/quick-feature.md +0 -44
- package/framework/workflows/sprint-cycle.md +0 -47
- package/framework/workflows/workspace-feature.md +0 -71
|
@@ -1,30 +0,0 @@
|
|
|
1
|
-
# Memory Review Checklist
|
|
2
|
-
|
|
3
|
-
Gate mode: **flexible** (memory entries are informational, auto-advance)
|
|
4
|
-
|
|
5
|
-
## Conventions (`.sniper/memory/conventions.yaml`)
|
|
6
|
-
- [ ] Every convention has a non-empty `rule` field with a clear, imperative statement
|
|
7
|
-
- [ ] Every convention has a `rationale` explaining why it exists
|
|
8
|
-
- [ ] Every convention has at least one role in `applies_to`
|
|
9
|
-
- [ ] Every convention has `enforcement` set to one of: review_gate, spawn_prompt, both
|
|
10
|
-
- [ ] No two conventions have identical or contradictory rules
|
|
11
|
-
- [ ] Convention IDs are unique and follow the `conv-XXX` pattern
|
|
12
|
-
|
|
13
|
-
## Anti-Patterns (`.sniper/memory/anti-patterns.yaml`)
|
|
14
|
-
- [ ] Every anti-pattern has a non-empty `description`
|
|
15
|
-
- [ ] Every anti-pattern has a `fix_pattern` describing the correct approach
|
|
16
|
-
- [ ] Every anti-pattern has `severity` set to one of: high, medium, low
|
|
17
|
-
- [ ] Every anti-pattern has at least one role in `applies_to`
|
|
18
|
-
- [ ] Anti-pattern IDs are unique and follow the `ap-XXX` pattern
|
|
19
|
-
|
|
20
|
-
## Decisions (`.sniper/memory/decisions.yaml`)
|
|
21
|
-
- [ ] Every decision has `title`, `context`, and `decision` fields filled
|
|
22
|
-
- [ ] Every decision has at least one alternative in `alternatives_considered`
|
|
23
|
-
- [ ] Every decision has `status` set to one of: active, superseded, deprecated
|
|
24
|
-
- [ ] Superseded decisions reference the replacing decision in `superseded_by`
|
|
25
|
-
- [ ] Decision IDs are unique and follow the `dec-XXX` pattern
|
|
26
|
-
|
|
27
|
-
## Overall
|
|
28
|
-
- [ ] No duplicate entries across conventions, anti-patterns, and decisions
|
|
29
|
-
- [ ] All source references are valid (type and ref fields populated)
|
|
30
|
-
- [ ] Candidate entries have sufficient evidence to remain as candidates
|
|
@@ -1,33 +0,0 @@
|
|
|
1
|
-
# Performance Audit Review Checklist
|
|
2
|
-
|
|
3
|
-
Use this checklist to review artifacts produced during a performance audit lifecycle.
|
|
4
|
-
|
|
5
|
-
## Profile Report (`docs/audits/PERF-{NNN}/profile-report.md`)
|
|
6
|
-
|
|
7
|
-
- [ ] **Actual bottlenecks:** Findings are based on code analysis, not premature optimization guesses
|
|
8
|
-
- [ ] **Specific locations:** Each bottleneck includes file:line or file:function references
|
|
9
|
-
- [ ] **Categorized:** Bottlenecks are categorized (N+1, missing index, sync I/O, etc.)
|
|
10
|
-
- [ ] **Impact assessed:** Severity based on path frequency and latency contribution
|
|
11
|
-
- [ ] **Critical paths traced:** Request handling chains are fully traced
|
|
12
|
-
- [ ] **Existing optimizations noted:** Current caching and indexing acknowledged
|
|
13
|
-
|
|
14
|
-
## Optimization Plan (`docs/audits/PERF-{NNN}/optimization-plan.md`)
|
|
15
|
-
|
|
16
|
-
- [ ] **Expected improvement stated:** Each recommendation includes specific improvement description
|
|
17
|
-
- [ ] **Trade-offs documented:** Caching complexity, consistency risks, etc. are noted
|
|
18
|
-
- [ ] **Quick wins identified:** Low-effort, high-impact optimizations separated
|
|
19
|
-
- [ ] **Benchmark requirements:** Each optimization has a corresponding benchmark requirement
|
|
20
|
-
- [ ] **Priority matrix sensible:** Impact/effort ratio drives the ordering
|
|
21
|
-
|
|
22
|
-
## Stories (`docs/audits/PERF-{NNN}/stories/`)
|
|
23
|
-
|
|
24
|
-
- [ ] **Paired with benchmarks:** Each optimization story has a companion benchmark story
|
|
25
|
-
- [ ] **Quick wins first:** Low-effort optimizations are prioritized
|
|
26
|
-
- [ ] **Self-contained:** Each story can be implemented and verified independently
|
|
27
|
-
- [ ] **Measurable:** Success criteria include specific performance metrics
|
|
28
|
-
|
|
29
|
-
## Overall
|
|
30
|
-
|
|
31
|
-
- [ ] **Data-driven:** No premature optimization — all changes backed by profiling evidence
|
|
32
|
-
- [ ] **Consistency:** Profile, plan, and stories tell a coherent story
|
|
33
|
-
- [ ] **Verification plan:** Clear strategy for measuring improvement after implementation
|
|
@@ -1,52 +0,0 @@
|
|
|
1
|
-
# Planning Review Checklist
|
|
2
|
-
|
|
3
|
-
Gate mode: **STRICT** (human MUST approve before Phase 3)
|
|
4
|
-
|
|
5
|
-
## PRD (`docs/prd.md`)
|
|
6
|
-
- [ ] Problem statement includes evidence (not just assertions)
|
|
7
|
-
- [ ] User stories follow "As a [persona], I want, so that" format
|
|
8
|
-
- [ ] P0 requirements are minimal — only what's critical for v1
|
|
9
|
-
- [ ] Every requirement has testable acceptance criteria
|
|
10
|
-
- [ ] Non-functional requirements have specific measurable targets
|
|
11
|
-
- [ ] Success metrics have numbers (not vague "improve X")
|
|
12
|
-
- [ ] Out-of-scope explicitly names features users might expect
|
|
13
|
-
- [ ] No duplicate requirements
|
|
14
|
-
|
|
15
|
-
## Architecture (`docs/architecture.md`)
|
|
16
|
-
- [ ] Every technology choice includes: what, why, alternatives considered
|
|
17
|
-
- [ ] Component diagram shows clear boundaries and interfaces
|
|
18
|
-
- [ ] Data models include field types, constraints, indexes, relationships
|
|
19
|
-
- [ ] API contracts are specific enough for independent frontend/backend implementation
|
|
20
|
-
- [ ] Infrastructure specifies sizing, scaling triggers, cost estimates
|
|
21
|
-
- [ ] Cross-cutting concerns addressed (auth, logging, errors, config)
|
|
22
|
-
- [ ] Non-functional targets have implementation strategies
|
|
23
|
-
- [ ] Security architecture aligns with `docs/security.md`
|
|
24
|
-
|
|
25
|
-
## UX Specification (`docs/ux-spec.md`)
|
|
26
|
-
- [ ] Information architecture maps all pages/views
|
|
27
|
-
- [ ] Screen inventory covers all user-facing screens
|
|
28
|
-
- [ ] User flows include error paths, not just happy paths
|
|
29
|
-
- [ ] Component specs include all states (default, hover, active, disabled, loading, error)
|
|
30
|
-
- [ ] Responsive breakpoints specify actual layout changes
|
|
31
|
-
- [ ] Accessibility requirements name specific WCAG criteria
|
|
32
|
-
- [ ] UX flows align with PRD user stories
|
|
33
|
-
|
|
34
|
-
## Security Requirements (`docs/security.md`)
|
|
35
|
-
- [ ] Authentication model specified (OAuth, JWT, session, etc.)
|
|
36
|
-
- [ ] Authorization model specified (RBAC, ABAC, etc.)
|
|
37
|
-
- [ ] Data encryption strategy covers at-rest and in-transit
|
|
38
|
-
- [ ] Compliance requirements name specific regulations
|
|
39
|
-
- [ ] Threat model identifies top attack vectors
|
|
40
|
-
- [ ] Security testing requirements are defined
|
|
41
|
-
|
|
42
|
-
## Cross-Document Consistency
|
|
43
|
-
- [ ] Architecture API contracts match UX component data needs
|
|
44
|
-
- [ ] Security requirements are implementable within architecture choices
|
|
45
|
-
- [ ] PRD requirements are fully coverable by architecture design
|
|
46
|
-
- [ ] No orphaned requirements (in PRD but not in architecture)
|
|
47
|
-
|
|
48
|
-
## Approval
|
|
49
|
-
**Reviewer:** _______________
|
|
50
|
-
**Date:** _______________
|
|
51
|
-
**Decision:** APPROVED / NEEDS REVISION
|
|
52
|
-
**Feedback:**
|
|
@@ -1,33 +0,0 @@
|
|
|
1
|
-
# Refactor Review Checklist
|
|
2
|
-
|
|
3
|
-
Use this checklist to review artifacts produced during a refactoring lifecycle.
|
|
4
|
-
|
|
5
|
-
## Impact Analysis (`docs/refactors/REF-{NNN}/scope.md`)
|
|
6
|
-
|
|
7
|
-
- [ ] **Complete inventory:** All instances of the pattern being changed are counted
|
|
8
|
-
- [ ] **Files listed:** Every affected file is listed, not estimated
|
|
9
|
-
- [ ] **Risk assessment:** Risks are specific, not generic boilerplate
|
|
10
|
-
- [ ] **Compatibility concerns:** API consumers and downstream dependencies are identified
|
|
11
|
-
- [ ] **Effort estimate:** Effort is justified by actual file/instance counts
|
|
12
|
-
|
|
13
|
-
## Migration Plan (`docs/refactors/REF-{NNN}/plan.md`)
|
|
14
|
-
|
|
15
|
-
- [ ] **Strategy justified:** Migration strategy choice (big-bang/incremental/strangler) has clear rationale
|
|
16
|
-
- [ ] **Order makes sense:** Steps follow dependency order (e.g., shared code before consuming code)
|
|
17
|
-
- [ ] **Coexistence plan:** Describes how old and new patterns coexist during migration
|
|
18
|
-
- [ ] **Verification at each step:** Each step has specific tests or checks
|
|
19
|
-
- [ ] **Rollback plan:** Each step can be reversed independently
|
|
20
|
-
- [ ] **No data loss risk:** Database migrations are reversible or have a safety net
|
|
21
|
-
|
|
22
|
-
## Stories (`docs/refactors/REF-{NNN}/stories/`)
|
|
23
|
-
|
|
24
|
-
- [ ] **Coverage:** Stories cover all instances from the pattern inventory
|
|
25
|
-
- [ ] **Order matches plan:** Story dependencies follow the migration plan order
|
|
26
|
-
- [ ] **Self-contained:** Each story can be implemented and verified independently
|
|
27
|
-
- [ ] **Tests included:** Each story includes test updates for the migrated code
|
|
28
|
-
|
|
29
|
-
## Overall
|
|
30
|
-
|
|
31
|
-
- [ ] **Consistency:** Scope, plan, and stories tell a coherent story
|
|
32
|
-
- [ ] **Completeness:** No instances of the old pattern will remain after all stories complete
|
|
33
|
-
- [ ] **Safety:** At no point during the migration will the system be in a broken state
|
|
@@ -1,34 +0,0 @@
|
|
|
1
|
-
# Security Audit Review Checklist
|
|
2
|
-
|
|
3
|
-
Use this checklist to review artifacts produced during a security audit lifecycle.
|
|
4
|
-
|
|
5
|
-
## Threat Model (`docs/audits/SEC-{NNN}/threat-model.md`)
|
|
6
|
-
|
|
7
|
-
- [ ] **Complete attack surface:** All entry points from the architecture doc are mapped
|
|
8
|
-
- [ ] **Trust boundaries identified:** Authentication and authorization boundaries are clearly documented
|
|
9
|
-
- [ ] **Data classified:** Sensitive data types are identified with storage and flow documentation
|
|
10
|
-
- [ ] **STRIDE applied systematically:** All six categories evaluated for each component
|
|
11
|
-
- [ ] **Dependencies assessed:** High-risk packages identified with CVE status
|
|
12
|
-
- [ ] **Threats prioritized:** Top threats ranked by likelihood x impact with justification
|
|
13
|
-
|
|
14
|
-
## Vulnerability Report (`docs/audits/SEC-{NNN}/vulnerability-report.md`)
|
|
15
|
-
|
|
16
|
-
- [ ] **Evidence-based findings:** Each vulnerability includes specific file:line and code pattern
|
|
17
|
-
- [ ] **Severity justified:** Severity ratings account for context (auth requirements, exposure)
|
|
18
|
-
- [ ] **OWASP categorized:** Findings mapped to OWASP Top 10 categories
|
|
19
|
-
- [ ] **Remediation included:** Each finding has a concrete fix with code example
|
|
20
|
-
- [ ] **Patterns identified:** Systemic issues (not just individual instances) are flagged
|
|
21
|
-
- [ ] **Positive findings noted:** Existing security practices are acknowledged
|
|
22
|
-
|
|
23
|
-
## Stories (`docs/audits/SEC-{NNN}/stories/`)
|
|
24
|
-
|
|
25
|
-
- [ ] **Severity-ordered:** Critical remediation stories come before high, medium, low
|
|
26
|
-
- [ ] **Systemic first:** Middleware and validation layer fixes before individual fixes
|
|
27
|
-
- [ ] **Secure code:** Remediation code follows secure coding practices
|
|
28
|
-
- [ ] **Test coverage:** Each story includes security test requirements
|
|
29
|
-
|
|
30
|
-
## Overall
|
|
31
|
-
|
|
32
|
-
- [ ] **Consistency:** Threat model and vulnerability report align (threats have corresponding findings)
|
|
33
|
-
- [ ] **Completeness:** All critical and high findings have remediation stories
|
|
34
|
-
- [ ] **Practicality:** Recommendations are actionable within the project's constraints
|
|
@@ -1,41 +0,0 @@
|
|
|
1
|
-
# Sprint Review Checklist
|
|
2
|
-
|
|
3
|
-
Gate mode: **STRICT** (human MUST review code before merge)
|
|
4
|
-
|
|
5
|
-
## Code Quality
|
|
6
|
-
- [ ] All code passes linting (no warnings or errors)
|
|
7
|
-
- [ ] All code passes static type checking (language-appropriate strict mode)
|
|
8
|
-
- [ ] No type escape hatches introduced (e.g. `any` in TS, `Any` in Python, `interface{}` in Go)
|
|
9
|
-
- [ ] No hardcoded secrets, API keys, or credentials
|
|
10
|
-
- [ ] Error handling on all async operations
|
|
11
|
-
- [ ] Follows existing codebase patterns and conventions
|
|
12
|
-
|
|
13
|
-
## Testing
|
|
14
|
-
- [ ] All stories have corresponding tests
|
|
15
|
-
- [ ] Tests pass (0 failures)
|
|
16
|
-
- [ ] Test coverage meets project minimum threshold
|
|
17
|
-
- [ ] Integration tests cover API endpoints
|
|
18
|
-
- [ ] Edge cases and error paths are tested
|
|
19
|
-
- [ ] No flaky tests introduced
|
|
20
|
-
|
|
21
|
-
## Acceptance Criteria
|
|
22
|
-
- [ ] Every acceptance criterion from every sprint story is verified
|
|
23
|
-
- [ ] Deviations from acceptance criteria are documented and justified
|
|
24
|
-
|
|
25
|
-
## Architecture Compliance
|
|
26
|
-
- [ ] Code follows the architecture patterns from `docs/architecture.md`
|
|
27
|
-
- [ ] API contracts match the spec (endpoints, payloads, status codes)
|
|
28
|
-
- [ ] Data models match the schema design
|
|
29
|
-
- [ ] File ownership boundaries respected (no cross-boundary edits)
|
|
30
|
-
|
|
31
|
-
## Security
|
|
32
|
-
- [ ] No new security vulnerabilities introduced (OWASP Top 10)
|
|
33
|
-
- [ ] Input validation on all user-facing endpoints
|
|
34
|
-
- [ ] Authentication and authorization enforced where required
|
|
35
|
-
- [ ] Sensitive data encrypted and handled properly
|
|
36
|
-
|
|
37
|
-
## Approval
|
|
38
|
-
**Reviewer:** _______________
|
|
39
|
-
**Date:** _______________
|
|
40
|
-
**Decision:** APPROVED / NEEDS REVISION
|
|
41
|
-
**Feedback:**
|
|
@@ -1,30 +0,0 @@
|
|
|
1
|
-
# Story Review Checklist
|
|
2
|
-
|
|
3
|
-
Gate mode: **FLEXIBLE** (auto-advance, async review)
|
|
4
|
-
|
|
5
|
-
## Epic Structure (`docs/epics/*.md`)
|
|
6
|
-
- [ ] Epics number between 6-12 (enough granularity, not too fragmented)
|
|
7
|
-
- [ ] No overlap between epics — each requirement maps to exactly one epic
|
|
8
|
-
- [ ] Epic dependencies form a DAG (no circular dependencies)
|
|
9
|
-
- [ ] Each epic has clear scope boundaries (in/out)
|
|
10
|
-
- [ ] Architecture context is EMBEDDED in each epic, not just referenced
|
|
11
|
-
- [ ] Complexity estimates are realistic
|
|
12
|
-
|
|
13
|
-
## Story Quality (`docs/stories/*.md`)
|
|
14
|
-
- [ ] Each story is self-contained — a developer can implement from the story file alone
|
|
15
|
-
- [ ] PRD context is EMBEDDED (copied), not just referenced
|
|
16
|
-
- [ ] Architecture context is EMBEDDED (data models, API contracts, patterns)
|
|
17
|
-
- [ ] UX context is EMBEDDED for frontend stories
|
|
18
|
-
- [ ] Acceptance criteria use Given/When/Then format
|
|
19
|
-
- [ ] Every acceptance criterion is testable
|
|
20
|
-
- [ ] Test requirements are specified (unit, integration, e2e)
|
|
21
|
-
- [ ] File ownership is assigned (which directories the story touches)
|
|
22
|
-
- [ ] Dependencies on other stories are declared
|
|
23
|
-
- [ ] Complexity estimate (S/M/L/XL) is assigned
|
|
24
|
-
- [ ] No story is XL — if so, it should be split
|
|
25
|
-
|
|
26
|
-
## Coverage
|
|
27
|
-
- [ ] All P0 PRD requirements are covered by stories
|
|
28
|
-
- [ ] All P1 PRD requirements are covered by stories
|
|
29
|
-
- [ ] All architecture components have at least one implementing story
|
|
30
|
-
- [ ] Story dependency chains allow reasonable sprint planning
|
|
@@ -1,32 +0,0 @@
|
|
|
1
|
-
# Test & Coverage Review Checklist
|
|
2
|
-
|
|
3
|
-
Use this checklist to review artifacts produced during a test audit lifecycle.
|
|
4
|
-
|
|
5
|
-
## Coverage Report (`docs/audits/TST-{NNN}/coverage-report.md`)
|
|
6
|
-
|
|
7
|
-
- [ ] **Coverage data sourced:** Coverage numbers come from actual test runner output, not estimates
|
|
8
|
-
- [ ] **Critical gaps identified:** Uncovered code paths in high-risk areas (auth, payments, data handling) are flagged
|
|
9
|
-
- [ ] **Risk-based ranking:** Gaps are ranked by production impact, not just by coverage percentage
|
|
10
|
-
- [ ] **Integration boundaries mapped:** Cross-module interaction points are checked against architecture components
|
|
11
|
-
- [ ] **Pattern analysis present:** Testing consistency (assertion styles, mocks, naming) is assessed
|
|
12
|
-
|
|
13
|
-
## Flaky Report (`docs/audits/TST-{NNN}/flaky-report.md`)
|
|
14
|
-
|
|
15
|
-
- [ ] **Root causes identified:** Each flaky test has a specific root cause, not just "flaky"
|
|
16
|
-
- [ ] **Evidence provided:** Flakiness is demonstrated with evidence (dual-run results or code pattern analysis)
|
|
17
|
-
- [ ] **No retry-only fixes:** Suggested fixes address root causes, not just add retries
|
|
18
|
-
- [ ] **Systemic issues flagged:** Patterns causing multiple flaky tests are identified
|
|
19
|
-
- [ ] **Quick wins separated:** Low-effort fixes are clearly distinguished from larger refactors
|
|
20
|
-
|
|
21
|
-
## Stories (`docs/audits/TST-{NNN}/stories/`)
|
|
22
|
-
|
|
23
|
-
- [ ] **Priority order:** Critical gap fixes and quick-win flake fixes come first
|
|
24
|
-
- [ ] **Scoped improvements:** Each story handles one logical test improvement
|
|
25
|
-
- [ ] **Follows conventions:** Suggested tests follow the project's existing test patterns
|
|
26
|
-
- [ ] **Actionable:** Each story has specific file:line references and concrete test approaches
|
|
27
|
-
|
|
28
|
-
## Overall
|
|
29
|
-
|
|
30
|
-
- [ ] **Consistency:** Coverage report and flaky report don't contradict each other
|
|
31
|
-
- [ ] **Completeness:** All critical gaps and systemic flake issues have corresponding stories
|
|
32
|
-
- [ ] **Practicality:** Recommendations are achievable (not "rewrite all tests from scratch")
|
|
@@ -1,34 +0,0 @@
|
|
|
1
|
-
# Workspace Feature Review Checklist
|
|
2
|
-
|
|
3
|
-
Gate mode: **strict** (cross-repo features require approval before implementation)
|
|
4
|
-
|
|
5
|
-
## Workspace Feature Brief
|
|
6
|
-
- [ ] All affected repositories are identified with clear justification
|
|
7
|
-
- [ ] Repositories not affected are explicitly excluded with reasoning
|
|
8
|
-
- [ ] New interfaces are documented with type (REST/Type/Event) and participating repos
|
|
9
|
-
- [ ] Modified interfaces reference existing contract versions
|
|
10
|
-
- [ ] Wave ordering respects the dependency graph — no repo sprints before its dependencies
|
|
11
|
-
- [ ] Repos in the same dependency tier are grouped for parallel execution
|
|
12
|
-
- [ ] Risks and considerations are identified for cross-repo coordination
|
|
13
|
-
|
|
14
|
-
## Interface Contracts
|
|
15
|
-
- [ ] Every new cross-repo interface has a formal contract
|
|
16
|
-
- [ ] Every contract has full request/response schemas for all endpoints
|
|
17
|
-
- [ ] Error responses are defined, not just success cases
|
|
18
|
-
- [ ] Shared types specify exactly one owning repository
|
|
19
|
-
- [ ] Event contracts specify producer and consumer(s) with payload schemas
|
|
20
|
-
- [ ] Contract versions follow semver (breaking changes = major bump)
|
|
21
|
-
- [ ] Contracts are self-contained — implementable without reading other repo code
|
|
22
|
-
|
|
23
|
-
## Cross-Repo Implementation Plan
|
|
24
|
-
- [ ] Per-repo work breakdown covers all affected repositories
|
|
25
|
-
- [ ] Each repo's stories reference specific contract items
|
|
26
|
-
- [ ] Sprint wave ordering matches the dependency ordering in the brief
|
|
27
|
-
- [ ] Integration validation criteria are defined for each wave boundary
|
|
28
|
-
- [ ] Rollback plan exists for contract validation failures
|
|
29
|
-
|
|
30
|
-
## Overall
|
|
31
|
-
- [ ] No circular dependencies in the wave ordering
|
|
32
|
-
- [ ] Workspace memory conventions are consistent across repos
|
|
33
|
-
- [ ] All contract files are valid YAML
|
|
34
|
-
- [ ] Feature numbering follows WKSP-XXXX pattern
|
|
@@ -1,37 +0,0 @@
|
|
|
1
|
-
# SNIPER Project
|
|
2
|
-
|
|
3
|
-
## Framework
|
|
4
|
-
This project uses SNIPER (Spawn, Navigate, Implement, Parallelize, Evaluate, Release).
|
|
5
|
-
See `.sniper/config.yaml` for project settings.
|
|
6
|
-
|
|
7
|
-
## Quick Reference
|
|
8
|
-
- Framework workflows: `.sniper/workflows/`
|
|
9
|
-
- Persona layers: `.sniper/personas/`
|
|
10
|
-
- Team definitions: `.sniper/teams/`
|
|
11
|
-
- Artifact templates: `.sniper/templates/`
|
|
12
|
-
- Quality gates: `.sniper/checklists/`
|
|
13
|
-
- Project artifacts: `docs/`
|
|
14
|
-
- Domain context: `.sniper/domain-packs/{pack-name}/`
|
|
15
|
-
|
|
16
|
-
## Commands
|
|
17
|
-
- `/sniper-init` — Initialize SNIPER in a new project
|
|
18
|
-
- `/sniper-discover` — Phase 1: Discovery & Analysis (parallel team)
|
|
19
|
-
- `/sniper-plan` — Phase 2: Planning & Architecture (parallel team)
|
|
20
|
-
- `/sniper-solve` — Phase 3: Epic Sharding & Story Creation (sequential)
|
|
21
|
-
- `/sniper-sprint` — Phase 4: Implementation Sprint (parallel team)
|
|
22
|
-
- `/sniper-review` — Run review gate for current phase
|
|
23
|
-
- `/sniper-compose` — Create a spawn prompt from persona layers
|
|
24
|
-
- `/sniper-status` — Show lifecycle status and artifact state
|
|
25
|
-
|
|
26
|
-
## Agent Teams Rules
|
|
27
|
-
When spawning teammates, always:
|
|
28
|
-
1. Read the relevant team YAML from `.sniper/teams/`
|
|
29
|
-
2. Compose spawn prompts using `/sniper-compose` with the layers specified in the YAML
|
|
30
|
-
3. Assign file ownership boundaries from `config.yaml` ownership rules
|
|
31
|
-
4. Create tasks with dependencies from the team YAML
|
|
32
|
-
5. Enter delegate mode (Shift+Tab) — the lead coordinates, it does not code
|
|
33
|
-
6. Require plan approval for tasks marked `plan_approval: true`
|
|
34
|
-
7. When a phase completes, run `/sniper-review` before advancing
|
|
35
|
-
|
|
36
|
-
## Code Standards
|
|
37
|
-
See `.sniper/config.yaml` → stack section for language/framework specifics.
|