context-mode 1.0.118 → 1.0.120
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/.claude-plugin/marketplace.json +2 -2
- package/.claude-plugin/plugin.json +1 -1
- package/.openclaw-plugin/openclaw.plugin.json +1 -1
- package/.openclaw-plugin/package.json +1 -1
- package/build/adapters/openclaw/mcp-tools.js +10 -1
- package/build/adapters/pi/mcp-bridge.d.ts +28 -3
- package/build/adapters/pi/mcp-bridge.js +127 -14
- package/build/adapters/qwen-code/index.js +6 -2
- package/build/cli.js +93 -5
- package/build/opencode-plugin.js +2 -5
- package/build/server.js +104 -30
- package/build/session/purge.d.ts +27 -0
- package/build/session/purge.js +105 -3
- package/build/util/project-dir.js +9 -5
- package/cli.bundle.mjs +195 -164
- package/hooks/core/routing.mjs +13 -0
- package/openclaw.plugin.json +1 -1
- package/package.json +5 -6
- package/scripts/heal-better-sqlite3.mjs +53 -6
- package/scripts/heal-installed-plugins.mjs +104 -0
- package/scripts/postinstall.mjs +35 -1
- package/server.bundle.mjs +135 -113
- package/skills/UPSTREAM-CREDITS.md +51 -0
- package/skills/ctx-purge/SKILL.md +23 -9
- package/skills/diagnose/SKILL.md +122 -0
- package/skills/diagnose/scripts/hitl-loop.template.sh +41 -0
- package/skills/grill-me/SKILL.md +15 -0
- package/skills/grill-with-docs/ADR-FORMAT.md +47 -0
- package/skills/grill-with-docs/CONTEXT-FORMAT.md +77 -0
- package/skills/grill-with-docs/SKILL.md +93 -0
- package/skills/improve-codebase-architecture/DEEPENING.md +37 -0
- package/skills/improve-codebase-architecture/INTERFACE-DESIGN.md +44 -0
- package/skills/improve-codebase-architecture/LANGUAGE.md +53 -0
- package/skills/improve-codebase-architecture/SKILL.md +76 -0
- package/skills/tdd/SKILL.md +114 -0
- package/skills/tdd/deep-modules.md +33 -0
- package/skills/tdd/interface-design.md +31 -0
- package/skills/tdd/mocking.md +59 -0
- package/skills/tdd/refactoring.md +10 -0
- package/skills/tdd/tests.md +61 -0
- package/start.mjs +25 -1
- package/build/cache-heal.d.ts +0 -48
- package/build/cache-heal.js +0 -150
- package/build/routing-block.d.ts +0 -8
- package/build/routing-block.js +0 -86
- package/build/tool-naming.d.ts +0 -4
- package/build/tool-naming.js +0 -24
|
@@ -0,0 +1,76 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: improve-codebase-architecture
|
|
3
|
+
description: Find deepening opportunities in a codebase, informed by the domain language in CONTEXT.md and the decisions in docs/adr/. Use when the user wants to improve architecture, find refactoring opportunities, consolidate tightly-coupled modules, or make a codebase more testable and AI-navigable.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Improve Codebase Architecture
|
|
7
|
+
|
|
8
|
+
Surface architectural friction and propose **deepening opportunities** — refactors that turn shallow modules into deep ones. The aim is testability and AI-navigability.
|
|
9
|
+
|
|
10
|
+
## Glossary
|
|
11
|
+
|
|
12
|
+
Use these terms exactly in every suggestion. Consistent language is the point — don't drift into "component," "service," "API," or "boundary." Full definitions in [LANGUAGE.md](LANGUAGE.md).
|
|
13
|
+
|
|
14
|
+
- **Module** — anything with an interface and an implementation (function, class, package, slice).
|
|
15
|
+
- **Interface** — everything a caller must know to use the module: types, invariants, error modes, ordering, config. Not just the type signature.
|
|
16
|
+
- **Implementation** — the code inside.
|
|
17
|
+
- **Depth** — leverage at the interface: a lot of behaviour behind a small interface. **Deep** = high leverage. **Shallow** = interface nearly as complex as the implementation.
|
|
18
|
+
- **Seam** — where an interface lives; a place behaviour can be altered without editing in place. (Use this, not "boundary.")
|
|
19
|
+
- **Adapter** — a concrete thing satisfying an interface at a seam.
|
|
20
|
+
- **Leverage** — what callers get from depth.
|
|
21
|
+
- **Locality** — what maintainers get from depth: change, bugs, knowledge concentrated in one place.
|
|
22
|
+
|
|
23
|
+
Key principles (see [LANGUAGE.md](LANGUAGE.md) for the full list):
|
|
24
|
+
|
|
25
|
+
- **Deletion test**: imagine deleting the module. If complexity vanishes, it was a pass-through. If complexity reappears across N callers, it was earning its keep.
|
|
26
|
+
- **The interface is the test surface.**
|
|
27
|
+
- **One adapter = hypothetical seam. Two adapters = real seam.**
|
|
28
|
+
|
|
29
|
+
This skill is _informed_ by the project's domain model. The domain language gives names to good seams; ADRs record decisions the skill should not re-litigate.
|
|
30
|
+
|
|
31
|
+
## Process
|
|
32
|
+
|
|
33
|
+
### 1. Explore
|
|
34
|
+
|
|
35
|
+
Read the project's domain glossary and any ADRs in the area you're touching first.
|
|
36
|
+
|
|
37
|
+
Then use the Agent tool with `subagent_type=Explore` to walk the codebase. Don't follow rigid heuristics — explore organically and note where you experience friction:
|
|
38
|
+
|
|
39
|
+
- Where does understanding one concept require bouncing between many small modules?
|
|
40
|
+
- Where are modules **shallow** — interface nearly as complex as the implementation?
|
|
41
|
+
- Where have pure functions been extracted just for testability, but the real bugs hide in how they're called (no **locality**)?
|
|
42
|
+
- Where do tightly-coupled modules leak across their seams?
|
|
43
|
+
- Which parts of the codebase are untested, or hard to test through their current interface?
|
|
44
|
+
|
|
45
|
+
Apply the **deletion test** to anything you suspect is shallow: would deleting it concentrate complexity, or just move it? A "yes, concentrates" is the signal you want.
|
|
46
|
+
|
|
47
|
+
### 2. Present candidates
|
|
48
|
+
|
|
49
|
+
Present a numbered list of deepening opportunities. For each candidate:
|
|
50
|
+
|
|
51
|
+
- **Files** — which files/modules are involved
|
|
52
|
+
- **Problem** — why the current architecture is causing friction
|
|
53
|
+
- **Solution** — plain English description of what would change
|
|
54
|
+
- **Benefits** — explained in terms of locality and leverage, and also in how tests would improve
|
|
55
|
+
|
|
56
|
+
**Use CONTEXT.md vocabulary for the domain, and [LANGUAGE.md](LANGUAGE.md) vocabulary for the architecture.** If `CONTEXT.md` defines "Order," talk about "the Order intake module" — not "the FooBarHandler," and not "the Order service."
|
|
57
|
+
|
|
58
|
+
**ADR conflicts**: if a candidate contradicts an existing ADR, only surface it when the friction is real enough to warrant revisiting the ADR. Mark it clearly (e.g. _"contradicts ADR-0007 — but worth reopening because…"_). Don't list every theoretical refactor an ADR forbids.
|
|
59
|
+
|
|
60
|
+
Do NOT propose interfaces yet. Ask the user: "Which of these would you like to explore?"
|
|
61
|
+
|
|
62
|
+
### 3. Grilling loop
|
|
63
|
+
|
|
64
|
+
Once the user picks a candidate, drop into a grilling conversation. Walk the design tree with them — constraints, dependencies, the shape of the deepened module, what sits behind the seam, what tests survive.
|
|
65
|
+
|
|
66
|
+
Side effects happen inline as decisions crystallize:
|
|
67
|
+
|
|
68
|
+
- **Naming a deepened module after a concept not in `CONTEXT.md`?** Add the term to `CONTEXT.md` — same discipline as `/grill-with-docs` (see [CONTEXT-FORMAT.md](../grill-with-docs/CONTEXT-FORMAT.md)). Create the file lazily if it doesn't exist.
|
|
69
|
+
- **Sharpening a fuzzy term during the conversation?** Update `CONTEXT.md` right there.
|
|
70
|
+
- **User rejects the candidate with a load-bearing reason?** Offer an ADR, framed as: _"Want me to record this as an ADR so future architecture reviews don't re-suggest it?"_ Only offer when the reason would actually be needed by a future explorer to avoid re-suggesting the same thing — skip ephemeral reasons ("not worth it right now") and self-evident ones. See [ADR-FORMAT.md](../grill-with-docs/ADR-FORMAT.md).
|
|
71
|
+
- **Want to explore alternative interfaces for the deepened module?** See [INTERFACE-DESIGN.md](INTERFACE-DESIGN.md).
|
|
72
|
+
|
|
73
|
+
|
|
74
|
+
---
|
|
75
|
+
|
|
76
|
+
_Vendored from [mattpocock/skills](https://github.com/mattpocock/skills) @ `b843cb5` — MIT License. See [skills/UPSTREAM-CREDITS.md](../UPSTREAM-CREDITS.md) for refresh instructions._
|
|
@@ -0,0 +1,114 @@
|
|
|
1
|
+
---
|
|
2
|
+
name: tdd
|
|
3
|
+
description: Test-driven development with red-green-refactor loop. Use when user wants to build features or fix bugs using TDD, mentions "red-green-refactor", wants integration tests, or asks for test-first development.
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Test-Driven Development
|
|
7
|
+
|
|
8
|
+
## Philosophy
|
|
9
|
+
|
|
10
|
+
**Core principle**: Tests should verify behavior through public interfaces, not implementation details. Code can change entirely; tests shouldn't.
|
|
11
|
+
|
|
12
|
+
**Good tests** are integration-style: they exercise real code paths through public APIs. They describe _what_ the system does, not _how_ it does it. A good test reads like a specification - "user can checkout with valid cart" tells you exactly what capability exists. These tests survive refactors because they don't care about internal structure.
|
|
13
|
+
|
|
14
|
+
**Bad tests** are coupled to implementation. They mock internal collaborators, test private methods, or verify through external means (like querying a database directly instead of using the interface). The warning sign: your test breaks when you refactor, but behavior hasn't changed. If you rename an internal function and tests fail, those tests were testing implementation, not behavior.
|
|
15
|
+
|
|
16
|
+
See [tests.md](tests.md) for examples and [mocking.md](mocking.md) for mocking guidelines.
|
|
17
|
+
|
|
18
|
+
## Anti-Pattern: Horizontal Slices
|
|
19
|
+
|
|
20
|
+
**DO NOT write all tests first, then all implementation.** This is "horizontal slicing" - treating RED as "write all tests" and GREEN as "write all code."
|
|
21
|
+
|
|
22
|
+
This produces **crap tests**:
|
|
23
|
+
|
|
24
|
+
- Tests written in bulk test _imagined_ behavior, not _actual_ behavior
|
|
25
|
+
- You end up testing the _shape_ of things (data structures, function signatures) rather than user-facing behavior
|
|
26
|
+
- Tests become insensitive to real changes - they pass when behavior breaks, fail when behavior is fine
|
|
27
|
+
- You outrun your headlights, committing to test structure before understanding the implementation
|
|
28
|
+
|
|
29
|
+
**Correct approach**: Vertical slices via tracer bullets. One test → one implementation → repeat. Each test responds to what you learned from the previous cycle. Because you just wrote the code, you know exactly what behavior matters and how to verify it.
|
|
30
|
+
|
|
31
|
+
```
|
|
32
|
+
WRONG (horizontal):
|
|
33
|
+
RED: test1, test2, test3, test4, test5
|
|
34
|
+
GREEN: impl1, impl2, impl3, impl4, impl5
|
|
35
|
+
|
|
36
|
+
RIGHT (vertical):
|
|
37
|
+
RED→GREEN: test1→impl1
|
|
38
|
+
RED→GREEN: test2→impl2
|
|
39
|
+
RED→GREEN: test3→impl3
|
|
40
|
+
...
|
|
41
|
+
```
|
|
42
|
+
|
|
43
|
+
## Workflow
|
|
44
|
+
|
|
45
|
+
### 1. Planning
|
|
46
|
+
|
|
47
|
+
When exploring the codebase, use the project's domain glossary so that test names and interface vocabulary match the project's language, and respect ADRs in the area you're touching.
|
|
48
|
+
|
|
49
|
+
Before writing any code:
|
|
50
|
+
|
|
51
|
+
- [ ] Confirm with user what interface changes are needed
|
|
52
|
+
- [ ] Confirm with user which behaviors to test (prioritize)
|
|
53
|
+
- [ ] Identify opportunities for [deep modules](deep-modules.md) (small interface, deep implementation)
|
|
54
|
+
- [ ] Design interfaces for [testability](interface-design.md)
|
|
55
|
+
- [ ] List the behaviors to test (not implementation steps)
|
|
56
|
+
- [ ] Get user approval on the plan
|
|
57
|
+
|
|
58
|
+
Ask: "What should the public interface look like? Which behaviors are most important to test?"
|
|
59
|
+
|
|
60
|
+
**You can't test everything.** Confirm with the user exactly which behaviors matter most. Focus testing effort on critical paths and complex logic, not every possible edge case.
|
|
61
|
+
|
|
62
|
+
### 2. Tracer Bullet
|
|
63
|
+
|
|
64
|
+
Write ONE test that confirms ONE thing about the system:
|
|
65
|
+
|
|
66
|
+
```
|
|
67
|
+
RED: Write test for first behavior → test fails
|
|
68
|
+
GREEN: Write minimal code to pass → test passes
|
|
69
|
+
```
|
|
70
|
+
|
|
71
|
+
This is your tracer bullet - proves the path works end-to-end.
|
|
72
|
+
|
|
73
|
+
### 3. Incremental Loop
|
|
74
|
+
|
|
75
|
+
For each remaining behavior:
|
|
76
|
+
|
|
77
|
+
```
|
|
78
|
+
RED: Write next test → fails
|
|
79
|
+
GREEN: Minimal code to pass → passes
|
|
80
|
+
```
|
|
81
|
+
|
|
82
|
+
Rules:
|
|
83
|
+
|
|
84
|
+
- One test at a time
|
|
85
|
+
- Only enough code to pass current test
|
|
86
|
+
- Don't anticipate future tests
|
|
87
|
+
- Keep tests focused on observable behavior
|
|
88
|
+
|
|
89
|
+
### 4. Refactor
|
|
90
|
+
|
|
91
|
+
After all tests pass, look for [refactor candidates](refactoring.md):
|
|
92
|
+
|
|
93
|
+
- [ ] Extract duplication
|
|
94
|
+
- [ ] Deepen modules (move complexity behind simple interfaces)
|
|
95
|
+
- [ ] Apply SOLID principles where natural
|
|
96
|
+
- [ ] Consider what new code reveals about existing code
|
|
97
|
+
- [ ] Run tests after each refactor step
|
|
98
|
+
|
|
99
|
+
**Never refactor while RED.** Get to GREEN first.
|
|
100
|
+
|
|
101
|
+
## Checklist Per Cycle
|
|
102
|
+
|
|
103
|
+
```
|
|
104
|
+
[ ] Test describes behavior, not implementation
|
|
105
|
+
[ ] Test uses public interface only
|
|
106
|
+
[ ] Test would survive internal refactor
|
|
107
|
+
[ ] Code is minimal for this test
|
|
108
|
+
[ ] No speculative features added
|
|
109
|
+
```
|
|
110
|
+
|
|
111
|
+
|
|
112
|
+
---
|
|
113
|
+
|
|
114
|
+
_Vendored from [mattpocock/skills](https://github.com/mattpocock/skills) @ `b843cb5` — MIT License. See [skills/UPSTREAM-CREDITS.md](../UPSTREAM-CREDITS.md) for refresh instructions._
|
|
@@ -0,0 +1,33 @@
|
|
|
1
|
+
# Deep Modules
|
|
2
|
+
|
|
3
|
+
From "A Philosophy of Software Design":
|
|
4
|
+
|
|
5
|
+
**Deep module** = small interface + lots of implementation
|
|
6
|
+
|
|
7
|
+
```
|
|
8
|
+
┌─────────────────────┐
|
|
9
|
+
│ Small Interface │ ← Few methods, simple params
|
|
10
|
+
├─────────────────────┤
|
|
11
|
+
│ │
|
|
12
|
+
│ │
|
|
13
|
+
│ Deep Implementation│ ← Complex logic hidden
|
|
14
|
+
│ │
|
|
15
|
+
│ │
|
|
16
|
+
└─────────────────────┘
|
|
17
|
+
```
|
|
18
|
+
|
|
19
|
+
**Shallow module** = large interface + little implementation (avoid)
|
|
20
|
+
|
|
21
|
+
```
|
|
22
|
+
┌─────────────────────────────────┐
|
|
23
|
+
│ Large Interface │ ← Many methods, complex params
|
|
24
|
+
├─────────────────────────────────┤
|
|
25
|
+
│ Thin Implementation │ ← Just passes through
|
|
26
|
+
└─────────────────────────────────┘
|
|
27
|
+
```
|
|
28
|
+
|
|
29
|
+
When designing interfaces, ask:
|
|
30
|
+
|
|
31
|
+
- Can I reduce the number of methods?
|
|
32
|
+
- Can I simplify the parameters?
|
|
33
|
+
- Can I hide more complexity inside?
|
|
@@ -0,0 +1,31 @@
|
|
|
1
|
+
# Interface Design for Testability
|
|
2
|
+
|
|
3
|
+
Good interfaces make testing natural:
|
|
4
|
+
|
|
5
|
+
1. **Accept dependencies, don't create them**
|
|
6
|
+
|
|
7
|
+
```typescript
|
|
8
|
+
// Testable
|
|
9
|
+
function processOrder(order, paymentGateway) {}
|
|
10
|
+
|
|
11
|
+
// Hard to test
|
|
12
|
+
function processOrder(order) {
|
|
13
|
+
const gateway = new StripeGateway();
|
|
14
|
+
}
|
|
15
|
+
```
|
|
16
|
+
|
|
17
|
+
2. **Return results, don't produce side effects**
|
|
18
|
+
|
|
19
|
+
```typescript
|
|
20
|
+
// Testable
|
|
21
|
+
function calculateDiscount(cart): Discount {}
|
|
22
|
+
|
|
23
|
+
// Hard to test
|
|
24
|
+
function applyDiscount(cart): void {
|
|
25
|
+
cart.total -= discount;
|
|
26
|
+
}
|
|
27
|
+
```
|
|
28
|
+
|
|
29
|
+
3. **Small surface area**
|
|
30
|
+
- Fewer methods = fewer tests needed
|
|
31
|
+
- Fewer params = simpler test setup
|
|
@@ -0,0 +1,59 @@
|
|
|
1
|
+
# When to Mock
|
|
2
|
+
|
|
3
|
+
Mock at **system boundaries** only:
|
|
4
|
+
|
|
5
|
+
- External APIs (payment, email, etc.)
|
|
6
|
+
- Databases (sometimes - prefer test DB)
|
|
7
|
+
- Time/randomness
|
|
8
|
+
- File system (sometimes)
|
|
9
|
+
|
|
10
|
+
Don't mock:
|
|
11
|
+
|
|
12
|
+
- Your own classes/modules
|
|
13
|
+
- Internal collaborators
|
|
14
|
+
- Anything you control
|
|
15
|
+
|
|
16
|
+
## Designing for Mockability
|
|
17
|
+
|
|
18
|
+
At system boundaries, design interfaces that are easy to mock:
|
|
19
|
+
|
|
20
|
+
**1. Use dependency injection**
|
|
21
|
+
|
|
22
|
+
Pass external dependencies in rather than creating them internally:
|
|
23
|
+
|
|
24
|
+
```typescript
|
|
25
|
+
// Easy to mock
|
|
26
|
+
function processPayment(order, paymentClient) {
|
|
27
|
+
return paymentClient.charge(order.total);
|
|
28
|
+
}
|
|
29
|
+
|
|
30
|
+
// Hard to mock
|
|
31
|
+
function processPayment(order) {
|
|
32
|
+
const client = new StripeClient(process.env.STRIPE_KEY);
|
|
33
|
+
return client.charge(order.total);
|
|
34
|
+
}
|
|
35
|
+
```
|
|
36
|
+
|
|
37
|
+
**2. Prefer SDK-style interfaces over generic fetchers**
|
|
38
|
+
|
|
39
|
+
Create specific functions for each external operation instead of one generic function with conditional logic:
|
|
40
|
+
|
|
41
|
+
```typescript
|
|
42
|
+
// GOOD: Each function is independently mockable
|
|
43
|
+
const api = {
|
|
44
|
+
getUser: (id) => fetch(`/users/${id}`),
|
|
45
|
+
getOrders: (userId) => fetch(`/users/${userId}/orders`),
|
|
46
|
+
createOrder: (data) => fetch('/orders', { method: 'POST', body: data }),
|
|
47
|
+
};
|
|
48
|
+
|
|
49
|
+
// BAD: Mocking requires conditional logic inside the mock
|
|
50
|
+
const api = {
|
|
51
|
+
fetch: (endpoint, options) => fetch(endpoint, options),
|
|
52
|
+
};
|
|
53
|
+
```
|
|
54
|
+
|
|
55
|
+
The SDK approach means:
|
|
56
|
+
- Each mock returns one specific shape
|
|
57
|
+
- No conditional logic in test setup
|
|
58
|
+
- Easier to see which endpoints a test exercises
|
|
59
|
+
- Type safety per endpoint
|
|
@@ -0,0 +1,10 @@
|
|
|
1
|
+
# Refactor Candidates
|
|
2
|
+
|
|
3
|
+
After TDD cycle, look for:
|
|
4
|
+
|
|
5
|
+
- **Duplication** → Extract function/class
|
|
6
|
+
- **Long methods** → Break into private helpers (keep tests on public interface)
|
|
7
|
+
- **Shallow modules** → Combine or deepen
|
|
8
|
+
- **Feature envy** → Move logic to where data lives
|
|
9
|
+
- **Primitive obsession** → Introduce value objects
|
|
10
|
+
- **Existing code** the new code reveals as problematic
|
|
@@ -0,0 +1,61 @@
|
|
|
1
|
+
# Good and Bad Tests
|
|
2
|
+
|
|
3
|
+
## Good Tests
|
|
4
|
+
|
|
5
|
+
**Integration-style**: Test through real interfaces, not mocks of internal parts.
|
|
6
|
+
|
|
7
|
+
```typescript
|
|
8
|
+
// GOOD: Tests observable behavior
|
|
9
|
+
test("user can checkout with valid cart", async () => {
|
|
10
|
+
const cart = createCart();
|
|
11
|
+
cart.add(product);
|
|
12
|
+
const result = await checkout(cart, paymentMethod);
|
|
13
|
+
expect(result.status).toBe("confirmed");
|
|
14
|
+
});
|
|
15
|
+
```
|
|
16
|
+
|
|
17
|
+
Characteristics:
|
|
18
|
+
|
|
19
|
+
- Tests behavior users/callers care about
|
|
20
|
+
- Uses public API only
|
|
21
|
+
- Survives internal refactors
|
|
22
|
+
- Describes WHAT, not HOW
|
|
23
|
+
- One logical assertion per test
|
|
24
|
+
|
|
25
|
+
## Bad Tests
|
|
26
|
+
|
|
27
|
+
**Implementation-detail tests**: Coupled to internal structure.
|
|
28
|
+
|
|
29
|
+
```typescript
|
|
30
|
+
// BAD: Tests implementation details
|
|
31
|
+
test("checkout calls paymentService.process", async () => {
|
|
32
|
+
const mockPayment = jest.mock(paymentService);
|
|
33
|
+
await checkout(cart, payment);
|
|
34
|
+
expect(mockPayment.process).toHaveBeenCalledWith(cart.total);
|
|
35
|
+
});
|
|
36
|
+
```
|
|
37
|
+
|
|
38
|
+
Red flags:
|
|
39
|
+
|
|
40
|
+
- Mocking internal collaborators
|
|
41
|
+
- Testing private methods
|
|
42
|
+
- Asserting on call counts/order
|
|
43
|
+
- Test breaks when refactoring without behavior change
|
|
44
|
+
- Test name describes HOW not WHAT
|
|
45
|
+
- Verifying through external means instead of interface
|
|
46
|
+
|
|
47
|
+
```typescript
|
|
48
|
+
// BAD: Bypasses interface to verify
|
|
49
|
+
test("createUser saves to database", async () => {
|
|
50
|
+
await createUser({ name: "Alice" });
|
|
51
|
+
const row = await db.query("SELECT * FROM users WHERE name = ?", ["Alice"]);
|
|
52
|
+
expect(row).toBeDefined();
|
|
53
|
+
});
|
|
54
|
+
|
|
55
|
+
// GOOD: Verifies through interface
|
|
56
|
+
test("createUser makes user retrievable", async () => {
|
|
57
|
+
const user = await createUser({ name: "Alice" });
|
|
58
|
+
const retrieved = await getUser(user.id);
|
|
59
|
+
expect(retrieved.name).toBe("Alice");
|
|
60
|
+
});
|
|
61
|
+
```
|
package/start.mjs
CHANGED
|
@@ -122,7 +122,7 @@ if (cacheMatch) {
|
|
|
122
122
|
// truth) so users who fix themselves via `npm install -g context-mode`
|
|
123
123
|
// follow the exact same code path. Best-effort, never blocks MCP boot.
|
|
124
124
|
try {
|
|
125
|
-
const { healInstalledPlugins, healSettingsEnabledPlugins } =
|
|
125
|
+
const { healInstalledPlugins, healSettingsEnabledPlugins, healPluginJsonMcpServers } =
|
|
126
126
|
await import("./scripts/heal-installed-plugins.mjs");
|
|
127
127
|
const pluginKey = "context-mode@context-mode";
|
|
128
128
|
const registryPath = resolve(homedir(), ".claude", "plugins", "installed_plugins.json");
|
|
@@ -135,6 +135,30 @@ try {
|
|
|
135
135
|
// disable state is repaired before next /reload-plugins.
|
|
136
136
|
try { healSettingsEnabledPlugins({ settingsPath, pluginKey }); }
|
|
137
137
|
catch { /* best effort */ }
|
|
138
|
+
// v1.0.119 — Layer 5b (Issue #523): heal .claude-plugin/plugin.json's
|
|
139
|
+
// mcpServers["context-mode"].args[0] when /ctx-upgrade left a tmpdir-prefixed
|
|
140
|
+
// path baked in. Iterates EVERY installed cache entry's installPath so
|
|
141
|
+
// multi-version installs all self-recover. Each call is independently wrapped
|
|
142
|
+
// because one poisoned entry must not block heals on the others. Best effort.
|
|
143
|
+
try {
|
|
144
|
+
if (existsSync(registryPath)) {
|
|
145
|
+
const ip = JSON.parse(readFileSync(registryPath, "utf-8"));
|
|
146
|
+
const entries = (ip && ip.plugins && ip.plugins[pluginKey]) || [];
|
|
147
|
+
if (Array.isArray(entries)) {
|
|
148
|
+
for (const entry of entries) {
|
|
149
|
+
const installPath = entry && entry.installPath;
|
|
150
|
+
if (typeof installPath !== "string" || !installPath) continue;
|
|
151
|
+
try {
|
|
152
|
+
healPluginJsonMcpServers({
|
|
153
|
+
pluginRoot: installPath,
|
|
154
|
+
pluginCacheRoot,
|
|
155
|
+
pluginKey,
|
|
156
|
+
});
|
|
157
|
+
} catch { /* best effort — per-entry */ }
|
|
158
|
+
}
|
|
159
|
+
}
|
|
160
|
+
}
|
|
161
|
+
} catch { /* best effort */ }
|
|
138
162
|
} catch { /* best effort — never block MCP boot */ }
|
|
139
163
|
|
|
140
164
|
// ── Self-heal Layer 4: Deploy global SessionStart hook + register in settings.json ──
|
package/build/cache-heal.d.ts
DELETED
|
@@ -1,48 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Plugin cache self-heal — fixes broken CLAUDE_PLUGIN_ROOT references.
|
|
3
|
-
*
|
|
4
|
-
* Claude Code's plugin auto-update can leave installed_plugins.json pointing
|
|
5
|
-
* to a non-existent directory (anthropics/claude-code#46915). This module
|
|
6
|
-
* detects and repairs the mismatch by creating symlinks.
|
|
7
|
-
*
|
|
8
|
-
* 4-layer defense:
|
|
9
|
-
* 1. start.mjs startup — reverse heal (registry → symlink to us)
|
|
10
|
-
* 2. server.ts first tool call — mid-session heal
|
|
11
|
-
* 3. postinstall.mjs — backward symlink on new install
|
|
12
|
-
* 4. global hook auto-deploy — survives total plugin cache breakage
|
|
13
|
-
*/
|
|
14
|
-
export interface HealResult {
|
|
15
|
-
healed: boolean;
|
|
16
|
-
action?: "symlink" | "global-hook" | "none";
|
|
17
|
-
from?: string;
|
|
18
|
-
to?: string;
|
|
19
|
-
}
|
|
20
|
-
/**
|
|
21
|
-
* Core heal: if installed_plugins.json points to a non-existent directory,
|
|
22
|
-
* create a symlink from that path to our actual directory.
|
|
23
|
-
*
|
|
24
|
-
* @param currentDir - The directory we're actually running from
|
|
25
|
-
* @param installedPluginsPath - Path to installed_plugins.json (injectable for testing)
|
|
26
|
-
*/
|
|
27
|
-
export declare function healRegistryMismatch(currentDir: string, installedPluginsPath?: string): HealResult;
|
|
28
|
-
/**
|
|
29
|
-
* Deploy a global SessionStart hook that heals plugin cache mismatches.
|
|
30
|
-
* This hook lives outside the plugin directory, so it survives cache breakage.
|
|
31
|
-
*
|
|
32
|
-
* Written to ~/.claude/hooks/context-mode-cache-heal.sh
|
|
33
|
-
*/
|
|
34
|
-
export declare function deployGlobalHealHook(): HealResult;
|
|
35
|
-
/**
|
|
36
|
-
* Backward symlink: during postinstall, if the registry points to a
|
|
37
|
-
* non-existent OLD path, create a symlink from old → new (our directory).
|
|
38
|
-
* Same as healRegistryMismatch but called from postinstall context.
|
|
39
|
-
*/
|
|
40
|
-
export { healRegistryMismatch as healBackwardCompat };
|
|
41
|
-
/**
|
|
42
|
-
* Mid-session heal — call on first MCP tool invocation.
|
|
43
|
-
* Checks if registry path differs from our running directory.
|
|
44
|
-
* Creates symlink if needed. Runs only once per process.
|
|
45
|
-
*/
|
|
46
|
-
export declare function healMidSession(currentDir: string): HealResult;
|
|
47
|
-
/** Reset mid-session flag (for testing only) */
|
|
48
|
-
export declare function _resetMidSession(): void;
|
package/build/cache-heal.js
DELETED
|
@@ -1,150 +0,0 @@
|
|
|
1
|
-
/**
|
|
2
|
-
* Plugin cache self-heal — fixes broken CLAUDE_PLUGIN_ROOT references.
|
|
3
|
-
*
|
|
4
|
-
* Claude Code's plugin auto-update can leave installed_plugins.json pointing
|
|
5
|
-
* to a non-existent directory (anthropics/claude-code#46915). This module
|
|
6
|
-
* detects and repairs the mismatch by creating symlinks.
|
|
7
|
-
*
|
|
8
|
-
* 4-layer defense:
|
|
9
|
-
* 1. start.mjs startup — reverse heal (registry → symlink to us)
|
|
10
|
-
* 2. server.ts first tool call — mid-session heal
|
|
11
|
-
* 3. postinstall.mjs — backward symlink on new install
|
|
12
|
-
* 4. global hook auto-deploy — survives total plugin cache breakage
|
|
13
|
-
*/
|
|
14
|
-
import { existsSync, readFileSync, symlinkSync, mkdirSync, writeFileSync } from "node:fs";
|
|
15
|
-
import { resolve, dirname } from "node:path";
|
|
16
|
-
import { homedir } from "node:os";
|
|
17
|
-
/**
|
|
18
|
-
* Core heal: if installed_plugins.json points to a non-existent directory,
|
|
19
|
-
* create a symlink from that path to our actual directory.
|
|
20
|
-
*
|
|
21
|
-
* @param currentDir - The directory we're actually running from
|
|
22
|
-
* @param installedPluginsPath - Path to installed_plugins.json (injectable for testing)
|
|
23
|
-
*/
|
|
24
|
-
export function healRegistryMismatch(currentDir, installedPluginsPath) {
|
|
25
|
-
const ipPath = installedPluginsPath ?? resolve(homedir(), ".claude", "plugins", "installed_plugins.json");
|
|
26
|
-
if (!existsSync(ipPath))
|
|
27
|
-
return { healed: false, action: "none" };
|
|
28
|
-
if (!existsSync(currentDir))
|
|
29
|
-
return { healed: false, action: "none" };
|
|
30
|
-
let ip;
|
|
31
|
-
try {
|
|
32
|
-
ip = JSON.parse(readFileSync(ipPath, "utf-8"));
|
|
33
|
-
}
|
|
34
|
-
catch {
|
|
35
|
-
return { healed: false, action: "none" };
|
|
36
|
-
}
|
|
37
|
-
for (const [key, entries] of Object.entries(ip.plugins ?? {})) {
|
|
38
|
-
if (!key.toLowerCase().includes("context-mode"))
|
|
39
|
-
continue;
|
|
40
|
-
for (const entry of entries) {
|
|
41
|
-
const registryPath = entry.installPath;
|
|
42
|
-
if (!registryPath)
|
|
43
|
-
continue;
|
|
44
|
-
// Registry path exists — no healing needed
|
|
45
|
-
if (existsSync(registryPath))
|
|
46
|
-
continue;
|
|
47
|
-
// Registry path doesn't exist — create symlink to our directory
|
|
48
|
-
try {
|
|
49
|
-
const parent = dirname(registryPath);
|
|
50
|
-
if (!existsSync(parent))
|
|
51
|
-
mkdirSync(parent, { recursive: true });
|
|
52
|
-
if (process.platform === "win32") {
|
|
53
|
-
// Windows: use junction (no admin required)
|
|
54
|
-
symlinkSync(currentDir, registryPath, "junction");
|
|
55
|
-
}
|
|
56
|
-
else {
|
|
57
|
-
symlinkSync(currentDir, registryPath);
|
|
58
|
-
}
|
|
59
|
-
return { healed: true, action: "symlink", from: registryPath, to: currentDir };
|
|
60
|
-
}
|
|
61
|
-
catch {
|
|
62
|
-
return { healed: false, action: "none" };
|
|
63
|
-
}
|
|
64
|
-
}
|
|
65
|
-
}
|
|
66
|
-
return { healed: false, action: "none" };
|
|
67
|
-
}
|
|
68
|
-
/**
|
|
69
|
-
* Deploy a global SessionStart hook that heals plugin cache mismatches.
|
|
70
|
-
* This hook lives outside the plugin directory, so it survives cache breakage.
|
|
71
|
-
*
|
|
72
|
-
* Written to ~/.claude/hooks/context-mode-cache-heal.sh
|
|
73
|
-
*/
|
|
74
|
-
export function deployGlobalHealHook() {
|
|
75
|
-
const hooksDir = resolve(homedir(), ".claude", "hooks");
|
|
76
|
-
const hookPath = resolve(hooksDir, "context-mode-cache-heal.sh");
|
|
77
|
-
// Already deployed
|
|
78
|
-
if (existsSync(hookPath))
|
|
79
|
-
return { healed: false, action: "none" };
|
|
80
|
-
try {
|
|
81
|
-
if (!existsSync(hooksDir))
|
|
82
|
-
mkdirSync(hooksDir, { recursive: true });
|
|
83
|
-
const script = `#!/usr/bin/env bash
|
|
84
|
-
# context-mode plugin cache self-heal — auto-deployed by context-mode MCP server
|
|
85
|
-
# Fixes anthropics/claude-code#46915: auto-update breaks CLAUDE_PLUGIN_ROOT
|
|
86
|
-
# This hook runs at SessionStart (global, not plugin-level) so it works even
|
|
87
|
-
# when the plugin cache is broken.
|
|
88
|
-
|
|
89
|
-
set -euo pipefail
|
|
90
|
-
|
|
91
|
-
PLUGINS_FILE="$HOME/.claude/plugins/installed_plugins.json"
|
|
92
|
-
[[ -f "$PLUGINS_FILE" ]] || exit 0
|
|
93
|
-
|
|
94
|
-
# Find context-mode entries and heal missing directories
|
|
95
|
-
node -e '
|
|
96
|
-
const fs = require("fs");
|
|
97
|
-
const path = require("path");
|
|
98
|
-
try {
|
|
99
|
-
const ip = JSON.parse(fs.readFileSync(process.argv[1], "utf-8"));
|
|
100
|
-
for (const [key, entries] of Object.entries(ip.plugins || {})) {
|
|
101
|
-
if (!key.toLowerCase().includes("context-mode")) continue;
|
|
102
|
-
for (const entry of entries) {
|
|
103
|
-
const p = entry.installPath;
|
|
104
|
-
if (!p || fs.existsSync(p)) continue;
|
|
105
|
-
const parent = path.dirname(p);
|
|
106
|
-
if (!fs.existsSync(parent)) continue;
|
|
107
|
-
const dirs = fs.readdirSync(parent).filter(d => /^\\d+\\.\\d+/.test(d) && fs.statSync(path.join(parent, d)).isDirectory());
|
|
108
|
-
if (dirs.length === 0) continue;
|
|
109
|
-
dirs.sort((a, b) => {
|
|
110
|
-
const pa = a.split(".").map(Number), pb = b.split(".").map(Number);
|
|
111
|
-
for (let i = 0; i < 3; i++) { if ((pa[i]||0) !== (pb[i]||0)) return (pa[i]||0) - (pb[i]||0); }
|
|
112
|
-
return 0;
|
|
113
|
-
});
|
|
114
|
-
const target = path.join(parent, dirs[dirs.length - 1]);
|
|
115
|
-
try { fs.symlinkSync(target, p); } catch {}
|
|
116
|
-
}
|
|
117
|
-
}
|
|
118
|
-
} catch {}
|
|
119
|
-
' "$PLUGINS_FILE" 2>/dev/null || true
|
|
120
|
-
`;
|
|
121
|
-
writeFileSync(hookPath, script, { mode: 0o755 });
|
|
122
|
-
return { healed: true, action: "global-hook", from: hookPath };
|
|
123
|
-
}
|
|
124
|
-
catch {
|
|
125
|
-
return { healed: false, action: "none" };
|
|
126
|
-
}
|
|
127
|
-
}
|
|
128
|
-
/**
|
|
129
|
-
* Backward symlink: during postinstall, if the registry points to a
|
|
130
|
-
* non-existent OLD path, create a symlink from old → new (our directory).
|
|
131
|
-
* Same as healRegistryMismatch but called from postinstall context.
|
|
132
|
-
*/
|
|
133
|
-
export { healRegistryMismatch as healBackwardCompat };
|
|
134
|
-
/** One-shot flag for mid-session heal in server.ts */
|
|
135
|
-
let _midSessionHealed = false;
|
|
136
|
-
/**
|
|
137
|
-
* Mid-session heal — call on first MCP tool invocation.
|
|
138
|
-
* Checks if registry path differs from our running directory.
|
|
139
|
-
* Creates symlink if needed. Runs only once per process.
|
|
140
|
-
*/
|
|
141
|
-
export function healMidSession(currentDir) {
|
|
142
|
-
if (_midSessionHealed)
|
|
143
|
-
return { healed: false, action: "none" };
|
|
144
|
-
_midSessionHealed = true;
|
|
145
|
-
return healRegistryMismatch(currentDir);
|
|
146
|
-
}
|
|
147
|
-
/** Reset mid-session flag (for testing only) */
|
|
148
|
-
export function _resetMidSession() {
|
|
149
|
-
_midSessionHealed = false;
|
|
150
|
-
}
|
package/build/routing-block.d.ts
DELETED
|
@@ -1,8 +0,0 @@
|
|
|
1
|
-
import type { ToolNamer } from "./tool-naming.js";
|
|
2
|
-
export interface RoutingBlockOptions {
|
|
3
|
-
includeCommands?: boolean;
|
|
4
|
-
}
|
|
5
|
-
export declare function createRoutingBlock(t: ToolNamer, options?: RoutingBlockOptions): string;
|
|
6
|
-
export declare function createReadGuidance(t: ToolNamer): string;
|
|
7
|
-
export declare function createGrepGuidance(t: ToolNamer): string;
|
|
8
|
-
export declare function createBashGuidance(t: ToolNamer): string;
|