geneseed 4.9.3 → 4.10.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/CHANGELOG.md +31 -0
- package/README.md +14 -1
- package/SHIPPED.md +1 -1
- package/adapters/claude-code/README.md +1 -1
- package/adapters/opencode/plugins/geneseed-guard.js +20 -1
- package/docs/README.md +1 -1
- package/docs/concepts/collaboration.md +1 -1
- package/docs/concepts/enforcement-table.md +2 -2
- package/docs/concepts/foreman-mode.md +1 -1
- package/docs/concepts/hooks.md +3 -2
- package/docs/concepts/kits.md +79 -0
- package/docs/concepts/loop-branch.md +1 -1
- package/docs/concepts/loop-bricks.md +1 -1
- package/docs/concepts/loop-contracts.md +1 -1
- package/docs/concepts/loop-gates.md +1 -1
- package/docs/concepts/loop-notes.md +1 -1
- package/docs/concepts/loop-risk.md +1 -1
- package/docs/concepts/loop-templates.md +1 -1
- package/docs/concepts/loops.md +1 -1
- package/docs/concepts/lsp.md +1 -1
- package/docs/extending.md +108 -4
- package/docs/guides/install-scripted.md +1 -1
- package/docs/reference/glossary.md +1 -0
- package/docs/reference/plugin-guard.md +1 -1
- package/docs/understand/enforced-vs-asked.md +1 -1
- package/docs/understand/machine-claude.md +1 -1
- package/docs/web-ui.md +4 -0
- package/harness.config.json +1 -1
- package/js/build/args.mjs +27 -6
- package/js/build/driver.mjs +29 -8
- package/js/build/emit-claude.mjs +3 -1
- package/js/build/generate.mjs +10 -5
- package/js/build/render.mjs +94 -11
- package/js/build/source.mjs +117 -4
- package/js/cli-table.json +15 -0
- package/js/hosts/hooks.mjs +2 -1
- package/js/hosts/installs.mjs +35 -2
- package/js/hosts/loss.mjs +23 -2
- package/js/hosts/native.mjs +14 -4
- package/js/inspect/checks-authoring.mjs +6 -3
- package/js/inspect/checks-build.mjs +7 -1
- package/js/inspect/checks-counts.mjs +153 -6
- package/js/inspect/checks-repo.mjs +95 -20
- package/js/inspect/cost.mjs +1 -1
- package/js/inspect/diff.mjs +3 -1
- package/js/inspect/inventory.mjs +1 -0
- package/js/inspect/status.mjs +44 -3
- package/js/inspect/validate.mjs +3 -3
- package/js/maintain/setup.mjs +30 -6
- package/js/maintain/update.mjs +1 -1
- package/js/web/actions.mjs +30 -6
- package/js/web/api.mjs +40 -3
- package/js/web/routes.mjs +2 -1
- package/package.json +1 -1
- package/registry.json +63 -0
- package/src/AGENT.md.tmpl +2 -2
- package/src/agents/devops-engineer.md +45 -0
- package/src/agents/docker-expert.md +45 -0
- package/src/agents/kubernetes-specialist.md +45 -0
- package/src/agents/reviewer.md +2 -0
- package/src/agents/spring-boot-engineer.md +48 -0
- package/src/agents/spring-reviewer.md +49 -0
- package/src/agents/spring-security.md +50 -0
- package/src/kits/spring-boot/rules.md +10 -0
- package/src/kits/spring-boot.json +23 -0
- package/src/laws/universal.md +13 -13
- package/src/licenses/mattpocock-skills.txt +21 -0
- package/src/licenses/piomin-spring-boot.txt +190 -0
- package/src/licenses/superpowers.txt +21 -0
- package/src/licenses/vibe-wise.txt +21 -0
- package/src/skills/brainstorm/builder-check.md +2 -0
- package/src/skills/brainstorm.md +4 -0
- package/src/skills/codebase-design.md +4 -0
- package/src/skills/daydream/VENDOR.md +7 -7
- package/src/skills/debug.md +5 -1
- package/src/skills/develop.md +2 -0
- package/src/skills/geneseed-code-review.md +6 -2
- package/src/skills/git-history.md +2 -0
- package/src/skills/handoff.md +2 -0
- package/src/skills/learn-mode.md +3 -2
- package/src/skills/parallel-agents.md +2 -0
- package/src/skills/pipeline.md +2 -0
- package/src/skills/plan.md +4 -0
- package/src/skills/prototype.md +2 -0
- package/src/skills/react-view-transitions/AGENTS.md +646 -221
- package/src/skills/react-view-transitions/SKILL.md +28 -16
- package/src/skills/react-view-transitions/VENDOR.md +10 -8
- package/src/skills/react-view-transitions/references/css-recipes.md +72 -2
- package/src/skills/react-view-transitions/references/implementation.md +14 -26
- package/src/skills/react-view-transitions/references/nextjs.md +94 -37
- package/src/skills/react-view-transitions/references/patterns.md +60 -48
- package/src/skills/react-view-transitions/references/troubleshooting.md +47 -0
- package/src/skills/research.md +2 -0
- package/src/skills/review-response.md +2 -0
- package/src/skills/rule.md +2 -0
- package/src/skills/ship.md +2 -0
- package/src/skills/skill-forge.md +1 -0
- package/src/skills/spring-boot/cloud.md +103 -0
- package/src/skills/spring-boot/java-modern.md +71 -0
- package/src/skills/spring-boot/logging.md +89 -0
- package/src/skills/spring-boot/security.md +112 -0
- package/src/skills/spring-boot/web.md +112 -0
- package/src/skills/spring-boot.md +57 -0
- package/src/skills/spring-data-jpa/patterns.md +68 -0
- package/src/skills/spring-data-jpa.md +81 -0
- package/src/skills/spring-testing.md +93 -0
- package/src/skills/wayfinder.md +2 -0
- package/src/upstreams.json +59 -0
- package/themes/_TEMPLATE.json +9 -0
- package/themes/biker.json +9 -0
- package/themes/commentator.json +9 -0
- package/themes/cyberpunk.json +9 -0
- package/themes/gamer.json +9 -0
- package/themes/imperial.json +9 -0
- package/themes/joker.json +9 -0
- package/themes/marvin.json +9 -0
- package/themes/mean.json +9 -0
- package/themes/military.json +9 -0
- package/themes/neutral.json +9 -0
- package/themes/pirate.json +9 -0
- package/themes/sports.json +9 -0
- package/themes/verstappen.json +9 -0
- package/themes/wizard.json +9 -0
- package/web/dist/assets/{Activity-C8sZVIUU.js → Activity-BjyJaf8e.js} +1 -1
- package/web/dist/assets/{ActivityDetail-Cc6vWUS4.js → ActivityDetail-Bo9nRDNV.js} +1 -1
- package/web/dist/assets/{Dashboard-C5ATa3ow.js → Dashboard-BQJGU7vE.js} +2 -2
- package/web/dist/assets/{Docs-tDW7i7mo.js → Docs-DLVckkew.js} +1 -1
- package/web/dist/assets/{GreenhouseView-zk2Uzc3A.js → GreenhouseView--DML7CW3.js} +1 -1
- package/web/dist/assets/{Installs-CB_rvbBP.js → Installs-CxCiln1F.js} +2 -2
- package/web/dist/assets/{JournalView-Bfj27XHC.js → JournalView-ymgZleHi.js} +1 -1
- package/web/dist/assets/{Laws-BA3CzUf5.js → Laws-BkKnZOBI.js} +1 -1
- package/web/dist/assets/Library-C8ercIZv.js +1 -0
- package/web/dist/assets/{Loops-DaPk-ckL.js → Loops-C95zhbFl.js} +1 -1
- package/web/dist/assets/{OperatorHudView-CyiGhiHk.js → OperatorHudView-B9qwc3w3.js} +1 -1
- package/web/dist/assets/{Personal-Bwd27CmT.js → Personal-DyR72AF-.js} +1 -1
- package/web/dist/assets/{Tabs-BOD7Kaz1.js → Tabs-ChCr_HqV.js} +1 -1
- package/web/dist/assets/api-BfpZmUMI.js +1 -0
- package/web/dist/assets/{drawable-CyLe1gtW.js → drawable-CgYi0Dtg.js} +1 -1
- package/web/dist/assets/index-B32EvfDT.js +2 -0
- package/web/dist/index.html +2 -2
- package/web/dist/assets/Library-BWwiy9Vb.js +0 -1
- package/web/dist/assets/api-CiXlkbe0.js +0 -1
- package/web/dist/assets/index-DQXG_mqo.js +0 -2
|
@@ -0,0 +1,48 @@
|
|
|
1
|
+
# {{AGENT}}: spring-boot-engineer
|
|
2
|
+
|
|
3
|
+
> {{DESC_SPRING_BOOT_ENGINEER}}
|
|
4
|
+
|
|
5
|
+
{{CREDIT:piomin-spring-boot:agents/spring-boot-engineer.md}}
|
|
6
|
+
|
|
7
|
+
Implements Spring Boot 4 / Java 21+ features, following the spring-boot, spring-data-jpa and spring-testing skills.
|
|
8
|
+
|
|
9
|
+
## When to dispatch
|
|
10
|
+
- A Spring Boot feature, endpoint, repository or configuration change is scoped and code must be written.
|
|
11
|
+
|
|
12
|
+
## When NOT to dispatch
|
|
13
|
+
- The design is unclear — use [architect](architect.md) first.
|
|
14
|
+
- Tests only — use [tester](tester.md). Non-Spring code — use [developer](developer.md).
|
|
15
|
+
|
|
16
|
+
## Inputs
|
|
17
|
+
- The scoped task, the target module or worktree, and the acceptance criteria.
|
|
18
|
+
|
|
19
|
+
## Allowed tools
|
|
20
|
+
- **Read + write to the assigned worktree.** May run the project's build and test commands.
|
|
21
|
+
<!-- bash: allow -->
|
|
22
|
+
- Never commits or pushes ({{DOCTRINE:consent-before-push}}).
|
|
23
|
+
|
|
24
|
+
## Procedure
|
|
25
|
+
0. If `{{DIR_MEMORY}}/agents/<your-name>.md` exists, read it first — your durable lessons from prior dispatches ({{DOCTRINE:persist-insight}}).
|
|
26
|
+
1. Follow the project's existing layout and build tool; match its group id, packages and CI. Lombok is fine when the project already uses it.
|
|
27
|
+
2. Constructor injection; records for request and response DTOs; `@Valid` on request bodies; one `@RestControllerAdvice` for errors.
|
|
28
|
+
3. Services: `@Transactional(readOnly = true)` on the class, `@Transactional` on mutating methods. Never call a transactional or `@Async` method through `this`.
|
|
29
|
+
4. Group related settings in `@ConfigurationProperties`, not scattered `@Value`.
|
|
30
|
+
5. Outbound HTTP: `RestClient` for blocking calls, `@HttpExchange` interfaces for declarative clients; keep `WebClient` for reactive code only.
|
|
31
|
+
6. Persistence uses `jakarta.persistence`; never return entities from controllers.
|
|
32
|
+
7. Tests: positive and negative cases; `@MockitoBean` and `@MockitoSpyBean` for replaced beans; Testcontainers with `@ServiceConnection`.
|
|
33
|
+
8. Defaults worth setting: `server.shutdown=graceful` and `spring.lifecycle.timeout-per-shutdown-phase`.
|
|
34
|
+
9. Run the affected build and tests before handing off ({{LAW:verify-before-asserting}}).
|
|
35
|
+
|
|
36
|
+
## Output contract
|
|
37
|
+
- Files changed, a one-line summary, and the test command with its result.
|
|
38
|
+
- If the task cannot be done as given, report the blocker instead of guessing.
|
|
39
|
+
|
|
40
|
+
## Self-improvement
|
|
41
|
+
|
|
42
|
+
If this spec misled you — an input you needed but were not given, a boundary
|
|
43
|
+
that proved wrong, a step you could not execute — end your report with one line:
|
|
44
|
+
`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
|
|
45
|
+
friction. The caller weighs the feedback, folds a real flaw back into this file
|
|
46
|
+
with the user's assent, and records it to {{MEMORY}} only if it clears
|
|
47
|
+
{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
|
|
48
|
+
|
|
@@ -0,0 +1,49 @@
|
|
|
1
|
+
# {{AGENT}}: spring-reviewer
|
|
2
|
+
|
|
3
|
+
> {{DESC_SPRING_REVIEWER}}
|
|
4
|
+
|
|
5
|
+
{{CREDIT:piomin-spring-boot:agents/code-reviewer.md}}
|
|
6
|
+
|
|
7
|
+
Spring-specific review only. Generic correctness, naming and structure belong to [reviewer](reviewer.md); defer to it and do not repeat its checks.
|
|
8
|
+
|
|
9
|
+
## When to dispatch
|
|
10
|
+
- A change touches Spring Boot code (controllers, services, repositories, configuration, tests) and needs a framework-aware pass.
|
|
11
|
+
|
|
12
|
+
## When NOT to dispatch
|
|
13
|
+
- Non-Spring changes, or a generic review — use [reviewer](reviewer.md).
|
|
14
|
+
- Security surface — use [spring-security](spring-security.md).
|
|
15
|
+
|
|
16
|
+
## Inputs
|
|
17
|
+
- The base..head range (or the diff) and the task it was meant to satisfy.
|
|
18
|
+
|
|
19
|
+
## Allowed tools
|
|
20
|
+
- **Read-only.** May read code, run the test suite and linters, inspect history.
|
|
21
|
+
- Does not edit code; it reports. Never moves HEAD, the index or the working tree.
|
|
22
|
+
<!-- bash: allow -->
|
|
23
|
+
|
|
24
|
+
## Procedure
|
|
25
|
+
0. If `{{DIR_MEMORY}}/agents/<your-name>.md` exists, read it first — your durable lessons from prior dispatches ({{DOCTRINE:persist-insight}}).
|
|
26
|
+
1. Read the changed files and the build file to learn the Boot version.
|
|
27
|
+
2. Check, in this order:
|
|
28
|
+
- Injection: field `@Autowired`, circular dependencies.
|
|
29
|
+
- Transactions: multi-step writes without `@Transactional`; self-invocation bypassing the proxy; transactions held across remote calls.
|
|
30
|
+
- Persistence: N+1 on lazy collections (use `JOIN FETCH` or `@EntityGraph`), unbounded `findAll()`, entities returned from controllers, legacy `javax` persistence imports (the package is `jakarta.persistence`).
|
|
31
|
+
- Boot 4 API misuse: the removed Boot test-package mock and spy bean annotations (use `@MockitoBean`/`@MockitoSpyBean`), new `RestTemplate` code or blocking `WebClient` (use `RestClient`), new `TestRestTemplate` tests (prefer `RestTestClient`), `@DynamicPropertySource` where `@ServiceConnection` supports the container, `WebSecurityConfigurerAdapter`.
|
|
32
|
+
- Test slices: `@SpringBootTest` where `@WebMvcTest` or `@DataJpaTest` would do; slices missing their collaborators.
|
|
33
|
+
- Configuration: raw `@Value` for property groups, secrets in committed config.
|
|
34
|
+
- Actuator: endpoints exposed beyond need, `show-details` open to anonymous callers.
|
|
35
|
+
3. Verify by running the tests rather than assuming ({{LAW:verify-before-asserting}}).
|
|
36
|
+
|
|
37
|
+
## Output contract
|
|
38
|
+
- Findings as Conventional Comments, `label (decoration): file:line — problem — suggested fix`, most serious first; then a **Set aside:** list (or `none`) and a one-line verdict: any blocking → block, any if-minor and nothing blocking → fix-then-ship, otherwise ship.
|
|
39
|
+
- Name any check that was impossible instead of issuing a verdict on partial evidence.
|
|
40
|
+
|
|
41
|
+
## Self-improvement
|
|
42
|
+
|
|
43
|
+
If this spec misled you — an input you needed but were not given, a boundary
|
|
44
|
+
that proved wrong, a step you could not execute — end your report with one line:
|
|
45
|
+
`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
|
|
46
|
+
friction. The caller weighs the feedback, folds a real flaw back into this file
|
|
47
|
+
with the user's assent, and records it to {{MEMORY}} only if it clears
|
|
48
|
+
{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
|
|
49
|
+
|
|
@@ -0,0 +1,50 @@
|
|
|
1
|
+
# {{AGENT}}: spring-security
|
|
2
|
+
|
|
3
|
+
> {{DESC_SPRING_SECURITY}}
|
|
4
|
+
|
|
5
|
+
{{CREDIT:piomin-spring-boot:agents/security-engineer.md}}
|
|
6
|
+
|
|
7
|
+
Spring Security 7 audit only. The generic audit (injection, secrets, dependencies beyond Spring) belongs to [security](security.md); defer to it.
|
|
8
|
+
|
|
9
|
+
## When to dispatch
|
|
10
|
+
- A Spring change touches authentication, authorization, CORS, CSRF, headers, TLS or secrets handling.
|
|
11
|
+
|
|
12
|
+
## When NOT to dispatch
|
|
13
|
+
- Non-Spring security work — use [security](security.md).
|
|
14
|
+
- General Spring conventions — use [spring-reviewer](spring-reviewer.md).
|
|
15
|
+
|
|
16
|
+
## Inputs
|
|
17
|
+
- The diff or the security configuration (`SecurityFilterChain` beans, controllers, filters, `application.yml`).
|
|
18
|
+
|
|
19
|
+
## Allowed tools
|
|
20
|
+
- **Read-only.** May run dependency and secret scanners. Reports; does not patch.
|
|
21
|
+
<!-- bash: allow -->
|
|
22
|
+
|
|
23
|
+
## Procedure
|
|
24
|
+
0. If `{{DIR_MEMORY}}/agents/<your-name>.md` exists, read it first — your durable lessons from prior dispatches ({{DOCTRINE:persist-insight}}).
|
|
25
|
+
1. Read the `SecurityFilterChain` beans first, then method-security usage and config files.
|
|
26
|
+
2. Check:
|
|
27
|
+
- Filter chain: a bean, never `WebSecurityConfigurerAdapter`; `authorizeHttpRequests` ends in `anyRequest().authenticated()` or `denyAll()`; only health and login endpoints open.
|
|
28
|
+
- Resource server: `oauth2ResourceServer(o -> o.jwt(...))` with an issuer or JWK source and audience validation; no hand-rolled token filter without a reason.
|
|
29
|
+
- Method security: `@EnableMethodSecurity` present wherever `@PreAuthorize` is used.
|
|
30
|
+
- Sessions and CSRF: stateless APIs set `SessionCreationPolicy.STATELESS` and may disable CSRF; session or cookie apps keep CSRF on.
|
|
31
|
+
- CORS: explicit origins, never `*` with credentials; the chain calls `cors(...)`.
|
|
32
|
+
- Headers: defaults not disabled; HSTS on HTTPS.
|
|
33
|
+
- TLS: SSL bundles; `reload-on-update` works only for Tomcat and Netty and only from `file:` locations, so a `classpath:` bundle never reloads.
|
|
34
|
+
- Secrets: none in committed config; use env vars, mounted files or config trees.
|
|
35
|
+
- Errors: no stack traces or internals in responses.
|
|
36
|
+
3. Verify with `grep` and the test suite ({{LAW:verify-before-asserting}}).
|
|
37
|
+
|
|
38
|
+
## Output contract
|
|
39
|
+
- Findings by severity (critical, high, medium, low), each with `file:line`, the risk and a fix; then a one-line verdict.
|
|
40
|
+
- Name any check that was impossible instead of issuing a verdict on partial evidence.
|
|
41
|
+
|
|
42
|
+
## Self-improvement
|
|
43
|
+
|
|
44
|
+
If this spec misled you — an input you needed but were not given, a boundary
|
|
45
|
+
that proved wrong, a step you could not execute — end your report with one line:
|
|
46
|
+
`spec-feedback: <what failed — the one-line fix>`. Omit it when there is no
|
|
47
|
+
friction. The caller weighs the feedback, folds a real flaw back into this file
|
|
48
|
+
with the user's assent, and records it to {{MEMORY}} only if it clears
|
|
49
|
+
{{DOCTRINE:persist-insight}}'s bar — most reports carry no feedback at all.
|
|
50
|
+
|
|
@@ -0,0 +1,10 @@
|
|
|
1
|
+
### Spring Boot kit
|
|
2
|
+
|
|
3
|
+
Applies to Spring Boot 4 projects (Spring Framework 7, Java 21+). Read the project's own pom.xml or Gradle build first and check any API you reach for against the version it pins; do not assume a newer or older one.
|
|
4
|
+
|
|
5
|
+
- Inject through constructors only. No field-level `@Autowired`.
|
|
6
|
+
- Blocking HTTP calls use `RestClient`. Do not add new `RestTemplate` code.
|
|
7
|
+
- Persistence and validation imports are `jakarta.*`, never `javax.*`.
|
|
8
|
+
- Tests that need a database or broker use Testcontainers wired with `@ServiceConnection`, not hand-copied connection properties.
|
|
9
|
+
- Replace beans in tests with `@MockitoBean` and `@MockitoSpyBean`; the older Boot test-package mock and spy bean annotations are removed.
|
|
10
|
+
- Match the project's existing style, build tool and CI before introducing another.
|
|
@@ -0,0 +1,23 @@
|
|
|
1
|
+
{
|
|
2
|
+
"description": "Spring Boot 4 / Java 21+ services: build, data access, testing, security review, and the container and cluster around them.",
|
|
3
|
+
"skills": ["spring-boot", "spring-data-jpa", "spring-testing"],
|
|
4
|
+
"agents": ["spring-boot-engineer", "spring-reviewer", "spring-security", "kubernetes-specialist", "docker-expert", "devops-engineer"],
|
|
5
|
+
"rules": "kits/spring-boot/rules.md",
|
|
6
|
+
"rulesWordBudget": 300,
|
|
7
|
+
"banned": ["@MockBean", "@SpyBean", "javax.persistence", "ClientHttpRequestFactories", "Spring Security 6"],
|
|
8
|
+
"catalogue": {
|
|
9
|
+
"skills": [
|
|
10
|
+
"| [spring-boot]({{DIR_SKILLS}}/spring-boot.md) | building or changing a Spring Boot service: controllers, configuration, REST clients, error handling, observability, packaging |",
|
|
11
|
+
"| [spring-data-jpa]({{DIR_SKILLS}}/spring-data-jpa.md) | Spring Data JPA or Hibernate work: entities, repositories, queries, transactions, schema migrations, N+1 and fetch problems |",
|
|
12
|
+
"| [spring-testing]({{DIR_SKILLS}}/spring-testing.md) | writing or fixing Spring Boot tests: slices, MockMvc, Testcontainers with service connections, mocking beans |"
|
|
13
|
+
],
|
|
14
|
+
"agents": [
|
|
15
|
+
"| [spring-boot-engineer]({{DIR_AGENTS}}/spring-boot-engineer.md) | a Spring Boot feature or service must be implemented end to end in a Java codebase |",
|
|
16
|
+
"| [spring-reviewer]({{DIR_AGENTS}}/spring-reviewer.md) | a Spring Boot change needs a framework-aware review: layering, transactions, JPA, configuration, test slices |",
|
|
17
|
+
"| [spring-security]({{DIR_AGENTS}}/spring-security.md) | a Spring Security change (filter chain, method security, OAuth2/JWT, CORS/CSRF) needs a security pass |",
|
|
18
|
+
"| [kubernetes-specialist]({{DIR_AGENTS}}/kubernetes-specialist.md) | a Spring Boot service needs Kubernetes manifests, probes, resources or a rollout strategy reviewed or written |",
|
|
19
|
+
"| [docker-expert]({{DIR_AGENTS}}/docker-expert.md) | a Spring Boot service needs a Dockerfile, layered or native image, or container build reviewed or written |",
|
|
20
|
+
"| [devops-engineer]({{DIR_AGENTS}}/devops-engineer.md) | a Spring Boot service needs its build and delivery pipeline reviewed or improved, on whatever CI the project already uses |"
|
|
21
|
+
]
|
|
22
|
+
}
|
|
23
|
+
}
|
package/src/laws/universal.md
CHANGED
|
@@ -72,21 +72,21 @@ governs the goal).
|
|
|
72
72
|
Real data loss and any irreversible, financial, privilege-changing or
|
|
73
73
|
outward-facing act (publishing, sending data to a third party) need explicit
|
|
74
74
|
confirmation bound to that act, or a durable authorization covering it; one
|
|
75
|
-
yes never stretches to the next. Real loss: deleting what
|
|
76
|
-
|
|
77
|
-
|
|
78
|
-
|
|
79
|
-
|
|
80
|
-
|
|
81
|
-
|
|
75
|
+
yes never stretches to the next. Real loss: deleting what git cannot restore
|
|
76
|
+
(untracked, modified, unpushed or unmerged work), discarding uncommitted work,
|
|
77
|
+
rewriting, deleting or force-moving main, master or the default branch, or
|
|
78
|
+
pruning reflog/gc. Classify every action as Create, Read, Update, or Delete
|
|
79
|
+
and tier by reversibility: what git restores — an edit, tracked clean files,
|
|
80
|
+
force-pushing your own branch, a worktree — runs freely, as does
|
|
81
|
+
`reset --hard` over a dirty tree.
|
|
82
82
|
<!-- LEAN:else -->
|
|
83
83
|
Real data loss — deleting untracked, modified, unpushed, unmerged work,
|
|
84
|
-
discarding uncommitted work
|
|
85
|
-
default branch, pruning reflog/gc — and any
|
|
86
|
-
privilege-changing or outward-facing act need
|
|
87
|
-
that act, or a durable authorization covering
|
|
88
|
-
the next. What git restores runs freely:
|
|
89
|
-
force-pushing your own branch.
|
|
84
|
+
discarding uncommitted work (except `reset --hard`), deleting or
|
|
85
|
+
force-moving main, master or the default branch, pruning reflog/gc — and any
|
|
86
|
+
irreversible, financial, privilege-changing or outward-facing act need
|
|
87
|
+
explicit confirmation bound to that act, or a durable authorization covering
|
|
88
|
+
it; one yes never stretches to the next. What git restores runs freely:
|
|
89
|
+
tracked clean files, a worktree, force-pushing your own branch.
|
|
90
90
|
<!-- LEAN:end -->
|
|
91
91
|
|
|
92
92
|
### {{LAW:surface-failures}} Surface Failures
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2026 Matt Pocock
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
|
@@ -0,0 +1,190 @@
|
|
|
1
|
+
Apache License
|
|
2
|
+
Version 2.0, January 2004
|
|
3
|
+
http://www.apache.org/licenses/
|
|
4
|
+
|
|
5
|
+
TERMS AND CONDITIONS FOR USE, REPRODUCTION, AND DISTRIBUTION
|
|
6
|
+
|
|
7
|
+
1. Definitions.
|
|
8
|
+
|
|
9
|
+
"License" shall mean the terms and conditions for use, reproduction,
|
|
10
|
+
and distribution as defined by Sections 1 through 9 of this document.
|
|
11
|
+
|
|
12
|
+
"Licensor" shall mean the copyright owner or entity authorized by
|
|
13
|
+
the copyright owner that is granting the License.
|
|
14
|
+
|
|
15
|
+
"Legal Entity" shall mean the union of the acting entity and all
|
|
16
|
+
other entities that control, are controlled by, or are under common
|
|
17
|
+
control with that entity. For the purposes of this definition,
|
|
18
|
+
"control" means (i) the power, direct or indirect, to cause the
|
|
19
|
+
direction or management of such entity, whether by contract or
|
|
20
|
+
otherwise, or (ii) ownership of fifty percent (50%) or more of the
|
|
21
|
+
outstanding shares, or (iii) beneficial ownership of such entity.
|
|
22
|
+
|
|
23
|
+
"You" (or "Your") shall mean an individual or Legal Entity
|
|
24
|
+
exercising permissions granted by this License.
|
|
25
|
+
|
|
26
|
+
"Source" form shall mean the preferred form for making modifications,
|
|
27
|
+
including but not limited to software source code, documentation
|
|
28
|
+
source, and configuration files.
|
|
29
|
+
|
|
30
|
+
"Object" form shall mean any form resulting from mechanical
|
|
31
|
+
transformation or translation of a Source form, including but
|
|
32
|
+
not limited to compiled object code, generated documentation,
|
|
33
|
+
and conversions to other media types.
|
|
34
|
+
|
|
35
|
+
"Work" shall mean the work of authorship, whether in Source or
|
|
36
|
+
Object form, made available under the License, as indicated by a
|
|
37
|
+
copyright notice that is included in or attached to the work
|
|
38
|
+
(an example is provided in the Appendix below).
|
|
39
|
+
|
|
40
|
+
"Derivative Works" shall mean any work, whether in Source or Object
|
|
41
|
+
form, that is based on (or derived from) the Work and for which the
|
|
42
|
+
editorial revisions, annotations, elaborations, or other modifications
|
|
43
|
+
represent, as a whole, an original work of authorship. For the purposes
|
|
44
|
+
of this License, Derivative Works shall not include works that remain
|
|
45
|
+
separable from, or merely link (or bind by name) to the interfaces of,
|
|
46
|
+
the Work and Derivative Works thereof.
|
|
47
|
+
|
|
48
|
+
"Contribution" shall mean any work of authorship, including
|
|
49
|
+
the original version of the Work and any modifications or additions
|
|
50
|
+
to that Work or Derivative Works thereof, that is intentionally
|
|
51
|
+
submitted to Licensor for inclusion in the Work by the copyright owner
|
|
52
|
+
or by an individual or Legal Entity authorized to submit on behalf of
|
|
53
|
+
the copyright owner. For the purposes of this definition, "submitted"
|
|
54
|
+
means any form of electronic, verbal, or written communication sent
|
|
55
|
+
to the Licensor or its representatives, including but not limited to
|
|
56
|
+
communication on electronic mailing lists, source code control systems,
|
|
57
|
+
and issue tracking systems that are managed by, or on behalf of, the
|
|
58
|
+
Licensor for the purpose of discussing and improving the Work, but
|
|
59
|
+
excluding communication that is conspicuously marked or otherwise
|
|
60
|
+
designated in writing by the copyright owner as "Not a Contribution."
|
|
61
|
+
|
|
62
|
+
"Contributor" shall mean Licensor and any individual or Legal Entity
|
|
63
|
+
on behalf of whom a Contribution has been received by Licensor and
|
|
64
|
+
subsequently incorporated within the Work.
|
|
65
|
+
|
|
66
|
+
2. Grant of Copyright License. Subject to the terms and conditions of
|
|
67
|
+
this License, each Contributor hereby grants to You a perpetual,
|
|
68
|
+
worldwide, non-exclusive, no-charge, royalty-free, irrevocable
|
|
69
|
+
copyright license to reproduce, prepare Derivative Works of,
|
|
70
|
+
publicly display, publicly perform, sublicense, and distribute the
|
|
71
|
+
Work and such Derivative Works in Source or Object form.
|
|
72
|
+
|
|
73
|
+
3. Grant of Patent License. Subject to the terms and conditions of
|
|
74
|
+
this License, each Contributor hereby grants to You a perpetual,
|
|
75
|
+
worldwide, non-exclusive, no-charge, royalty-free, irrevocable
|
|
76
|
+
(except as stated in this section) patent license to make, have made,
|
|
77
|
+
use, offer to sell, sell, import, and otherwise transfer the Work,
|
|
78
|
+
where such license applies only to those patent claims licensable
|
|
79
|
+
by such Contributor that are necessarily infringed by their
|
|
80
|
+
Contribution(s) alone or by combination of their Contribution(s)
|
|
81
|
+
with the Work to which such Contribution(s) was submitted. If You
|
|
82
|
+
institute patent litigation against any entity (including a
|
|
83
|
+
cross-claim or counterclaim in a lawsuit) alleging that the Work
|
|
84
|
+
or a Contribution incorporated within the Work constitutes direct
|
|
85
|
+
or contributory patent infringement, then any patent licenses
|
|
86
|
+
granted to You under this License for that Work shall terminate
|
|
87
|
+
as of the date such litigation is filed.
|
|
88
|
+
|
|
89
|
+
4. Redistribution. You may reproduce and distribute copies of the
|
|
90
|
+
Work or Derivative Works thereof in any medium, with or without
|
|
91
|
+
modifications, and in Source or Object form, provided that You
|
|
92
|
+
meet the following conditions:
|
|
93
|
+
|
|
94
|
+
(a) You must give any other recipients of the Work or
|
|
95
|
+
Derivative Works a copy of this License; and
|
|
96
|
+
|
|
97
|
+
(b) You must cause any modified files to carry prominent notices
|
|
98
|
+
stating that You changed the files; and
|
|
99
|
+
|
|
100
|
+
(c) You must retain, in the Source form of any Derivative Works
|
|
101
|
+
that You distribute, all copyright, patent, trademark, and
|
|
102
|
+
attribution notices from the Source form of the Work,
|
|
103
|
+
excluding those notices that do not pertain to any part of
|
|
104
|
+
the Derivative Works; and
|
|
105
|
+
|
|
106
|
+
(d) If the Work includes a "NOTICE" text file as part of its
|
|
107
|
+
distribution, then any Derivative Works that You distribute must
|
|
108
|
+
include a readable copy of the attribution notices contained
|
|
109
|
+
within such NOTICE file, excluding those notices that do not
|
|
110
|
+
pertain to any part of the Derivative Works, in at least one
|
|
111
|
+
of the following places: within a NOTICE text file distributed
|
|
112
|
+
as part of the Derivative Works; within the Source form or
|
|
113
|
+
documentation, if provided along with the Derivative Works; or,
|
|
114
|
+
within a display generated by the Derivative Works, if and
|
|
115
|
+
wherever such third-party notices normally appear. The contents
|
|
116
|
+
of the NOTICE file are for informational purposes only and
|
|
117
|
+
do not modify the License. You may add Your own attribution
|
|
118
|
+
notices within Derivative Works that You distribute, alongside
|
|
119
|
+
or as an addendum to the NOTICE text from the Work, provided
|
|
120
|
+
that such additional attribution notices cannot be construed
|
|
121
|
+
as modifying the License.
|
|
122
|
+
|
|
123
|
+
You may add Your own copyright statement to Your modifications and
|
|
124
|
+
may provide additional or different license terms and conditions
|
|
125
|
+
for use, reproduction, or distribution of Your modifications, or
|
|
126
|
+
for any such Derivative Works as a whole, provided Your use,
|
|
127
|
+
reproduction, and distribution of the Work otherwise complies with
|
|
128
|
+
the conditions stated in this License.
|
|
129
|
+
|
|
130
|
+
5. Submission of Contributions. Unless You explicitly state otherwise,
|
|
131
|
+
any Contribution intentionally submitted for inclusion in the Work
|
|
132
|
+
by You to the Licensor shall be under the terms and conditions of
|
|
133
|
+
this License, without any additional terms or conditions.
|
|
134
|
+
Notwithstanding the above, nothing herein shall supersede or modify
|
|
135
|
+
the terms of any separate license agreement you may have executed
|
|
136
|
+
with Licensor regarding such Contributions.
|
|
137
|
+
|
|
138
|
+
6. Trademarks. This License does not grant permission to use the trade
|
|
139
|
+
names, trademarks, service marks, or product names of the Licensor,
|
|
140
|
+
except as required for reasonable and customary use in describing the
|
|
141
|
+
origin of the Work and reproducing the content of the NOTICE file.
|
|
142
|
+
|
|
143
|
+
7. Disclaimer of Warranty. Unless required by applicable law or
|
|
144
|
+
agreed to in writing, Licensor provides the Work (and each
|
|
145
|
+
Contributor provides its Contributions) on an "AS IS" BASIS,
|
|
146
|
+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
|
|
147
|
+
implied, including, without limitation, any warranties or conditions
|
|
148
|
+
of TITLE, NON-INFRINGEMENT, MERCHANTABILITY, or FITNESS FOR A
|
|
149
|
+
PARTICULAR PURPOSE. You are solely responsible for determining the
|
|
150
|
+
appropriateness of using or redistributing the Work and assume any
|
|
151
|
+
risks associated with Your exercise of permissions under this License.
|
|
152
|
+
|
|
153
|
+
8. Limitation of Liability. In no event and under no legal theory,
|
|
154
|
+
whether in tort (including negligence), contract, or otherwise,
|
|
155
|
+
unless required by applicable law (such as deliberate and grossly
|
|
156
|
+
negligent acts) or agreed to in writing, shall any Contributor be
|
|
157
|
+
liable to You for damages, including any direct, indirect, special,
|
|
158
|
+
incidental, or consequential damages of any character arising as a
|
|
159
|
+
result of this License or out of the use or inability to use the
|
|
160
|
+
Work (including but not limited to damages for loss of goodwill,
|
|
161
|
+
work stoppage, computer failure or malfunction, or any and all
|
|
162
|
+
other commercial damages or losses), even if such Contributor
|
|
163
|
+
has been advised of the possibility of such damages.
|
|
164
|
+
|
|
165
|
+
9. Accepting Warranty or Additional Liability. While redistributing
|
|
166
|
+
the Work or Derivative Works thereof, You may choose to offer,
|
|
167
|
+
and charge a fee for, acceptance of support, warranty, indemnity,
|
|
168
|
+
or other liability obligations and/or rights consistent with this
|
|
169
|
+
License. However, in accepting such obligations, You may act only
|
|
170
|
+
on Your own behalf and on Your sole responsibility, not on behalf
|
|
171
|
+
of any other Contributor, and only if You agree to indemnify,
|
|
172
|
+
defend, and hold each Contributor harmless for any liability
|
|
173
|
+
incurred by, or claims asserted against, such Contributor by reason
|
|
174
|
+
of your accepting any such warranty or additional liability.
|
|
175
|
+
|
|
176
|
+
END OF TERMS AND CONDITIONS
|
|
177
|
+
|
|
178
|
+
Copyright 2026 Piotr Mińkowski
|
|
179
|
+
|
|
180
|
+
Licensed under the Apache License, Version 2.0 (the "License");
|
|
181
|
+
you may not use this file except in compliance with the License.
|
|
182
|
+
You may obtain a copy of the License at
|
|
183
|
+
|
|
184
|
+
http://www.apache.org/licenses/LICENSE-2.0
|
|
185
|
+
|
|
186
|
+
Unless required by applicable law or agreed to in writing, software
|
|
187
|
+
distributed under the License is distributed on an "AS IS" BASIS,
|
|
188
|
+
WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
189
|
+
See the License for the specific language governing permissions and
|
|
190
|
+
limitations under the License.
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2025 Jesse Vincent
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
|
@@ -0,0 +1,21 @@
|
|
|
1
|
+
MIT License
|
|
2
|
+
|
|
3
|
+
Copyright (c) 2026 Noah Kim
|
|
4
|
+
|
|
5
|
+
Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
6
|
+
of this software and associated documentation files (the "Software"), to deal
|
|
7
|
+
in the Software without restriction, including without limitation the rights
|
|
8
|
+
to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
9
|
+
copies of the Software, and to permit persons to whom the Software is
|
|
10
|
+
furnished to do so, subject to the following conditions:
|
|
11
|
+
|
|
12
|
+
The above copyright notice and this permission notice shall be included in all
|
|
13
|
+
copies or substantial portions of the Software.
|
|
14
|
+
|
|
15
|
+
THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
16
|
+
IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
17
|
+
FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
18
|
+
AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
19
|
+
LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
20
|
+
OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
|
|
21
|
+
SOFTWARE.
|
package/src/skills/brainstorm.md
CHANGED
|
@@ -3,6 +3,10 @@
|
|
|
3
3
|
> {{DESC_BRAINSTORM}}
|
|
4
4
|
<!-- aliases: grill-me -->
|
|
5
5
|
|
|
6
|
+
{{CREDIT:superpowers:skills/brainstorming/SKILL.md}}
|
|
7
|
+
|
|
8
|
+
{{CREDIT:mattpocock-skills:skills/productivity/grilling/SKILL.md}}
|
|
9
|
+
|
|
6
10
|
**Trigger:** a new feature, behaviour change, or project arrives without a design, or the
|
|
7
11
|
user says "brainstorm", "interview me" or "grill me". Also when goal, scope or success
|
|
8
12
|
criteria are unstated ("let's design this", "clarify this first", "stress-test this plan"),
|
|
@@ -3,6 +3,10 @@
|
|
|
3
3
|
> {{DESC_CODEBASE_DESIGN}}
|
|
4
4
|
<!-- aliases: domain-modeling -->
|
|
5
5
|
|
|
6
|
+
{{CREDIT:mattpocock-skills:skills/engineering/codebase-design/SKILL.md}}
|
|
7
|
+
|
|
8
|
+
{{CREDIT:mattpocock-skills:skills/engineering/domain-modeling/SKILL.md}}
|
|
9
|
+
|
|
6
10
|
**Trigger:** the user asks "where should the seam go", how to make a module testable through
|
|
7
11
|
its interface, or for a glossary term or ADR. Also designing or reshaping a module's interface,
|
|
8
12
|
a design talk that keeps stumbling over ambiguous terms, or another {{SKILL}} needing this vocabulary.
|
|
@@ -5,14 +5,14 @@ skill. It rides along in the rendered bundle and is exempt from Geneseed's autho
|
|
|
5
5
|
gates (token, dead-link, hermeticity, skill counts) — it is listed in the
|
|
6
6
|
generator's `VENDORED_SKILL_DIRS`.
|
|
7
7
|
|
|
8
|
-
- **Upstream:**
|
|
9
|
-
- **
|
|
10
|
-
- **Author:** Gleb Kalinin
|
|
11
|
-
- **License:** MIT (declared in `.claude-plugin/plugin.json`). The text is in `LICENSE`,
|
|
12
|
-
copied from upstream's root `LICENSE` as added in 93677f479f40ab43520883e7a6c954b755ab39ef
|
|
13
|
-
(the pinned commit predates the file; its copyright line covers 2025-2026).
|
|
8
|
+
- **Upstream key:** daydream (repo, pinned commit, author and licence: `src/upstreams.json`)
|
|
9
|
+
- **Credit:** {{CREDIT:daydream:daydream}}
|
|
14
10
|
- **Inspired by:** Gwern's [LLM Daydreaming](https://gwern.net/ai-daydreaming)
|
|
15
11
|
|
|
12
|
+
The licence (MIT, declared upstream in `.claude-plugin/plugin.json`) is in `LICENSE`, copied
|
|
13
|
+
from upstream's root `LICENSE` as added in 93677f479f40ab43520883e7a6c954b755ab39ef (the
|
|
14
|
+
pinned commit predates the file; its copyright line covers 2025-2026).
|
|
15
|
+
|
|
16
16
|
## Geneseed adaptation
|
|
17
17
|
|
|
18
18
|
Vault resolution is adapted to read the harness's `geneseed-wiki.jsonc` knowledge-base
|
|
@@ -32,7 +32,7 @@ does not tell the model where a skill lives.
|
|
|
32
32
|
|
|
33
33
|
To update, re-copy the upstream folder, re-apply the geneseed-wiki.jsonc adaptation, the
|
|
34
34
|
frontmatter and the placeholder paths in `SKILL.md` / `instructions.md`, and
|
|
35
|
-
bump the commit
|
|
35
|
+
bump the `daydream` entry's `commit` in `src/upstreams.json`.
|
|
36
36
|
|
|
37
37
|
`SKILL.md`'s frontmatter also carries `disable-model-invocation: true`, which upstream lacks: the
|
|
38
38
|
skill is user-only (`/daydream`), so its description stays out of the model's always-on catalogue —
|
package/src/skills/debug.md
CHANGED
|
@@ -3,6 +3,10 @@
|
|
|
3
3
|
> {{DESC_DEBUG}}
|
|
4
4
|
<!-- aliases: ci-fix -->
|
|
5
5
|
|
|
6
|
+
{{CREDIT:superpowers:skills/systematic-debugging/SKILL.md}}
|
|
7
|
+
|
|
8
|
+
{{CREDIT:mattpocock-skills:skills/engineering/diagnosing-bugs/SKILL.md}}
|
|
9
|
+
|
|
6
10
|
**Trigger:** a test fails, code crashes or prints a stack trace, or returns the wrong result — load this FIRST. Load it before reading code or investigating, not only before proposing a fix. Also when a CI run, pipeline job or pre-merge check is red ("CI is failing", "fix the build") — then start with the CI branch.
|
|
7
11
|
|
|
8
12
|
**No fix without a root cause.** A change you cannot tie to a cause you understand is
|
|
@@ -12,7 +16,7 @@ The pressure to skip straight to a patch is highest exactly when this rule matte
|
|
|
12
16
|
faster than thrashing; do not trade it away under pressure.
|
|
13
17
|
|
|
14
18
|
## Procedure
|
|
15
|
-
1. **Build one command that goes red on *this* bug** before reading code for a theory. It asserts the user's exact symptom (not "didn't crash"), gives the same verdict every run, takes seconds and runs unattended; run it and show the output. Read the error and stack trace in full. For a flaky bug, raise the reproduction rate until it is debuggable. If you cannot build it, say so, list what you tried and ask for access or a captured artifact — no command, no hypothesis. Read `GLOSSARY.md` (or `CONTEXT.md`) and the ADRs in the area for the domain's vocabulary.
|
|
19
|
+
1. **Build one command that goes red on *this* bug** before reading code for a theory. It asserts the user's exact symptom (not "didn't crash"), gives the same verdict every run, takes seconds and runs unattended; run it and show the output. If you forced the red by mutating code or a fixture, `diff` against a pristine copy to prove the mutation landed, then revert it. Read the error and stack trace in full. For a flaky bug, raise the reproduction rate until it is debuggable. If you cannot build it, say so, list what you tried and ask for access or a captured artifact — no command, no hypothesis. Read `GLOSSARY.md` (or `CONTEXT.md`) and the ADRs in the area for the domain's vocabulary.
|
|
16
20
|
2. **Minimise, then isolate.** Cut inputs and steps one at a time until every remaining element is load-bearing. Then binary-search the cause — the code path, or for a regression the commit range with `git bisect` (the [git-history {{SKILL}}](git-history.md) carries the method). Change one variable at a time. Find the nearest working twin (a passing sibling test, the last green commit, the same call elsewhere) and list every difference, however small. For a bad value deep in the stack, trace it back caller by caller to where it was born, and fix there. In a multi-layer system (CI → build → sign, API → service → DB), instrument each boundary once to see *which* layer breaks before investigating inside it. For a performance regression, take a baseline measurement and bisect; don't log.
|
|
17
21
|
3. **Rank 3–5 falsifiable hypotheses** ("if X is the cause, changing Y removes the bug") — list them in your reply, then test the top one without waiting. Cite the evidence for each as `file:line` or the exact log line. Proceed only on one that explains ALL the evidence — else gather more evidence, don't guess.
|
|
18
22
|
4. Apply the smallest fix that addresses the root cause, not the symptom; resist fixing things the evidence doesn't implicate ({{LAW:one-intent-one-act}}). One change at a time — no bundled "while I'm here" edits.
|
package/src/skills/develop.md
CHANGED
|
@@ -2,6 +2,8 @@
|
|
|
2
2
|
|
|
3
3
|
> {{DESC_DEVELOP}}
|
|
4
4
|
|
|
5
|
+
{{CREDIT:mattpocock-skills:skills/engineering/tdd/SKILL.md}}
|
|
6
|
+
|
|
5
7
|
**Trigger:** implementing a feature, fixing a bug, or changing code behaviour in a
|
|
6
8
|
repo that has (or should have) a test suite — before writing implementation code.
|
|
7
9
|
Covers both the test-first cycle (red → green → refactor) and the cover-alongside
|
|
@@ -2,6 +2,10 @@
|
|
|
2
2
|
|
|
3
3
|
> {{DESC_CODE_REVIEW}}
|
|
4
4
|
|
|
5
|
+
{{CREDIT:mattpocock-skills:skills/engineering/code-review/SKILL.md}}
|
|
6
|
+
|
|
7
|
+
{{CREDIT:superpowers:skills/requesting-code-review/code-reviewer.md}}
|
|
8
|
+
|
|
5
9
|
**Trigger:** reviewing changes before merge, or the user asks for a review — including "roast my code": same passes, findings unsoftened. Critiquing the user's own ask or plan is [roast-me](roast-me.md).
|
|
6
10
|
|
|
7
11
|
## Procedure
|
|
@@ -22,8 +26,8 @@
|
|
|
22
26
|
kept apart, one axis's findings never mask the other's.
|
|
23
27
|
5. Pass 1 — correctness: logic errors, edge cases, error handling, race
|
|
24
28
|
conditions. Verify suspect behaviour by running tests, not by assuming.
|
|
25
|
-
6. Pass 2 — quality: after `REVIEW.md`, the repo's documented standards
|
|
26
|
-
`CONTRIBUTING.md
|
|
29
|
+
6. Pass 2 — quality: after `REVIEW.md`, the repo's documented standards override: search the repo for every file that states how code
|
|
30
|
+
is written (`CODING_STANDARDS.md`, `CONTRIBUTING.md`, style or conventions docs), not one fixed path. Then, as judgement calls and skipping what tooling enforces:
|
|
27
31
|
mysterious name, duplication, feature envy, data clumps, primitive obsession, repeated switches, shotgun surgery, divergent change, speculative generality, message chains, middle man, refused bequest, dead code, units that do too much.
|
|
28
32
|
7. Pass 3 — spec fidelity: flag anything asked for but missing, present but never
|
|
29
33
|
asked for, or implemented wrong — quoting the spec line for each. Where the spec is silent,
|
|
@@ -3,6 +3,8 @@
|
|
|
3
3
|
> {{DESC_GIT_HISTORY}}
|
|
4
4
|
<!-- aliases: git-archaeology, git-rescue -->
|
|
5
5
|
|
|
6
|
+
{{CREDIT:mattpocock-skills-frozen:skills/engineering/resolving-merge-conflicts/SKILL.md}}
|
|
7
|
+
|
|
6
8
|
**Trigger:** something must be learned *from* the git history, or it needs repair.
|
|
7
9
|
Learn: when a regression came in, who last touched a line and why, where a symbol or
|
|
8
10
|
string entered. Repair: lost commits, a rebase/merge/reset gone wrong, a detached HEAD, a
|
package/src/skills/handoff.md
CHANGED
package/src/skills/learn-mode.md
CHANGED
|
@@ -4,6 +4,8 @@
|
|
|
4
4
|
|
|
5
5
|
<!-- invocation: user -->
|
|
6
6
|
|
|
7
|
+
{{CREDIT:vibe-wise:skills/learn/behavior.md}}
|
|
8
|
+
|
|
7
9
|
**Trigger:** the user wants to *learn while building* rather than be handed finished code.
|
|
8
10
|
Phrases: "learn mode", "turn on learning", "I want to understand what we build", "let me design it,
|
|
9
11
|
you write it". Also "pause learning" / "resume learning" / "stop learn mode". To be taught a
|
|
@@ -11,8 +13,7 @@ topic outside a build, use the [teach {{SKILL}}](teach.md); to have knowledge te
|
|
|
11
13
|
[quiz {{SKILL}}](quiz.md).
|
|
12
14
|
|
|
13
15
|
The user is the engineer and owns the design; you write the code for the design they
|
|
14
|
-
chose and understand. Learning and the user's control outrank build speed.
|
|
15
|
-
from [VibeWise](https://github.com/nykooi1/vibe-wise) (MIT); this is a rewrite, not a copy.
|
|
16
|
+
chose and understand. Learning and the user's control outrank build speed.
|
|
16
17
|
|
|
17
18
|
## State — `.geneseed/learn.md`
|
|
18
19
|
|