@codyswann/lisa 2.283.0 → 2.284.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/dist/core/upstream-evidence-manifest.d.ts.map +1 -1
- package/dist/core/upstream-evidence-manifest.js +16 -2
- package/dist/core/upstream-evidence-manifest.js.map +1 -1
- package/package.json +1 -1
- package/plugins/lisa/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa/.codex-plugin/skills/lisa-task-decomposition/SKILL.md +26 -0
- package/plugins/lisa/rules/eager/dependency-trust-classes.md +43 -0
- package/plugins/lisa/rules/reference/dependency-trust-classes.md +243 -0
- package/plugins/lisa/skills/lisa-task-decomposition/SKILL.md +26 -0
- package/plugins/lisa-agy/plugin.json +1 -1
- package/plugins/lisa-agy/skills/lisa-task-decomposition/SKILL.md +26 -0
- package/plugins/lisa-cdk/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-agy/plugin.json +1 -1
- package/plugins/lisa-cdk-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cdk-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-copilot/rules/eager/dependency-trust-classes.md +43 -0
- package/plugins/lisa-copilot/rules/reference/dependency-trust-classes.md +243 -0
- package/plugins/lisa-copilot/skills/lisa-task-decomposition/SKILL.md +26 -0
- package/plugins/lisa-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-cursor/rules/dependency-trust-classes-reference.mdc +248 -0
- package/plugins/lisa-cursor/rules/dependency-trust-classes.mdc +48 -0
- package/plugins/lisa-cursor/skills/lisa-task-decomposition/SKILL.md +26 -0
- package/plugins/lisa-expo/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-agy/plugin.json +1 -1
- package/plugins/lisa-expo-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-expo-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-agy/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-harper-fabric-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-agy/plugin.json +1 -1
- package/plugins/lisa-nestjs-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-nestjs-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-agy/plugin.json +1 -1
- package/plugins/lisa-openclaw-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-openclaw-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-agy/plugin.json +1 -1
- package/plugins/lisa-phaser-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-phaser-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-agy/plugin.json +1 -1
- package/plugins/lisa-rails-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-rails-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-agy/plugin.json +1 -1
- package/plugins/lisa-typescript-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-typescript-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki/.codex-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-agy/plugin.json +1 -1
- package/plugins/lisa-wiki-copilot/.claude-plugin/plugin.json +1 -1
- package/plugins/lisa-wiki-cursor/.claude-plugin/plugin.json +1 -1
- package/plugins/src/base/rules/eager/dependency-trust-classes.md +43 -0
- package/plugins/src/base/rules/reference/dependency-trust-classes.md +243 -0
- package/plugins/src/base/skills/lisa-task-decomposition/SKILL.md +26 -0
- package/ui/index.html +195 -43
|
@@ -0,0 +1,243 @@
|
|
|
1
|
+
# Dependency Trust Classes
|
|
2
|
+
|
|
3
|
+
Without named classes, every third-party package sits in one implicit bucket:
|
|
4
|
+
"we installed it, so presumably it's fine." That bucket makes a payment SDK and
|
|
5
|
+
a code formatter look identical at review time, and it forces every dependency
|
|
6
|
+
conversation to re-argue trust from first principles.
|
|
7
|
+
|
|
8
|
+
Trust classes fix that. Each class states, in plain language, **why we can trust
|
|
9
|
+
that kind of dependency**, and names **the one event that would trigger human
|
|
10
|
+
review**. A dependency's class is the answer to the
|
|
11
|
+
"Why we believe it's safe (trust basis)" field in
|
|
12
|
+
`.lisa/DEPENDENCY_DECISIONS.md` — the record says what
|
|
13
|
+
this dependency is and what would catch a bad update; the class says what bar it
|
|
14
|
+
had to clear to be here at all.
|
|
15
|
+
|
|
16
|
+
Exactly one class per dependency. If two seem to fit, take the lower-trust one —
|
|
17
|
+
the requirements are strictly stronger, and being over-careful about a formatter
|
|
18
|
+
costs a review cycle while being under-careful about an auth client costs users.
|
|
19
|
+
|
|
20
|
+
## The five review inputs
|
|
21
|
+
|
|
22
|
+
Every class fixes the same five inputs. What changes between classes is how
|
|
23
|
+
demanding each one is:
|
|
24
|
+
|
|
25
|
+
- **Capability owner** — who is accountable when this breaks. A role for
|
|
26
|
+
high-trust classes; a named person for low-trust ones, because "the platform
|
|
27
|
+
team" cannot be paged at 2am.
|
|
28
|
+
- **Update cadence** — how often the entry is re-read and the version re-judged.
|
|
29
|
+
- **Detection evidence** — the named check that fails if an update breaks the
|
|
30
|
+
owned capability. Stronger classes demand evidence *we* own, not upstream's
|
|
31
|
+
test suite.
|
|
32
|
+
- **Replacement-cost threshold** — the point past which the cost itself is the
|
|
33
|
+
finding. High replacement cost is acceptable in a high-trust class and is an
|
|
34
|
+
escalation in a low-trust one.
|
|
35
|
+
- **Product/human ratification** — whether a human has to say yes before the
|
|
36
|
+
dependency is added, upgraded, or kept. This is the gate, and it is deliberate
|
|
37
|
+
that the lower-trust and higher-exposure classes carry it.
|
|
38
|
+
|
|
39
|
+
## The six classes
|
|
40
|
+
|
|
41
|
+
### 1. Mature ecosystem primitive
|
|
42
|
+
|
|
43
|
+
**Why we can trust it:** so many other projects depend on it that a bad release
|
|
44
|
+
is usually found, reported, and fixed by someone else before it reaches us. It
|
|
45
|
+
has an identifiable maintaining organization, a published release cadence, and a
|
|
46
|
+
documented process for handling security reports. Trust here comes from the size
|
|
47
|
+
of the crowd standing in front of us, not from us having read the code.
|
|
48
|
+
|
|
49
|
+
Examples in kind: a language's canonical test runner, a foundation-maintained
|
|
50
|
+
linter, a mainstream UI framework.
|
|
51
|
+
|
|
52
|
+
- **Capability owner:** a role or team is sufficient.
|
|
53
|
+
- **Update cadence:** re-read at each major upgrade, and at least every 12
|
|
54
|
+
months.
|
|
55
|
+
- **Detection evidence:** an existing CI check must fail if the owned capability
|
|
56
|
+
breaks. Naming an upstream test suite is acceptable here and only here.
|
|
57
|
+
- **Replacement-cost threshold:** high replacement cost is accepted. That is the
|
|
58
|
+
bargain — we take deep coupling in exchange for the crowd's scrutiny.
|
|
59
|
+
- **Ratification:** not required to add or to take a minor/patch upgrade.
|
|
60
|
+
|
|
61
|
+
**Human review is triggered when:** maintainership changes hands or the project
|
|
62
|
+
is archived, or a security advisory goes unfixed past the project's own
|
|
63
|
+
published response window. Either event removes the crowd, which was the whole
|
|
64
|
+
basis for the trust — reclassify rather than re-trust.
|
|
65
|
+
|
|
66
|
+
### 2. Fast-moving standard implementation
|
|
67
|
+
|
|
68
|
+
**Why we can trust it:** it is the standard implementation of a protocol, spec,
|
|
69
|
+
or platform we have deliberately decided not to own — a cloud SDK, an API
|
|
70
|
+
client, a framework tracking a moving platform. We trust the *standard*; we do
|
|
71
|
+
not trust the release we happen to be on, because the project moves faster than
|
|
72
|
+
our review cycle does. Trust here is conditional on us keeping up.
|
|
73
|
+
|
|
74
|
+
- **Capability owner:** a named team that owns staying current.
|
|
75
|
+
- **Update cadence:** re-read at every minor upgrade, and no less than
|
|
76
|
+
quarterly. A dependency in this class going quiet for a year is itself a
|
|
77
|
+
finding.
|
|
78
|
+
- **Detection evidence:** an automated test **we** own that exercises the owned
|
|
79
|
+
capability. Upstream's suite does not count — it tests their contract, not our
|
|
80
|
+
use of it.
|
|
81
|
+
- **Replacement-cost threshold:** replacement cost must stay moderate. Keep the
|
|
82
|
+
coupling behind our own boundary so a forced migration is a rewrite of the
|
|
83
|
+
boundary, not of the application.
|
|
84
|
+
- **Ratification:** required for any major upgrade that changes user-visible
|
|
85
|
+
behavior.
|
|
86
|
+
|
|
87
|
+
**Human review is triggered when:** two consecutive major versions ship that we
|
|
88
|
+
did not adopt inside the cadence window. At that point we are pinned to
|
|
89
|
+
unsupported software and the decision — catch up, replace, or accept the
|
|
90
|
+
exposure — belongs to a human.
|
|
91
|
+
|
|
92
|
+
### 3. Build/development tool
|
|
93
|
+
|
|
94
|
+
**Why we can trust it:** it never runs in production and never touches customer
|
|
95
|
+
data. If it ships a bad update, developers notice within one build, and the
|
|
96
|
+
worst case is lost engineering time rather than a user-visible failure. The
|
|
97
|
+
blast radius is what earns the trust, not the maintainer.
|
|
98
|
+
|
|
99
|
+
- **Capability owner:** a role or team is sufficient.
|
|
100
|
+
- **Update cadence:** re-read at each major upgrade, and at least every 12
|
|
101
|
+
months.
|
|
102
|
+
- **Detection evidence:** the build, lint, or test job the tool powers. It fails
|
|
103
|
+
loudly and immediately, which is the point.
|
|
104
|
+
- **Replacement-cost threshold:** high cost is accepted — swapping a build tool
|
|
105
|
+
is disruptive but never urgent.
|
|
106
|
+
- **Ratification:** not required.
|
|
107
|
+
|
|
108
|
+
**Human review is triggered when:** the tool gains reach it did not have —
|
|
109
|
+
running in production, reading secrets, or reading CI credentials. The moment
|
|
110
|
+
that happens the blast-radius argument is void: reclassify it as a
|
|
111
|
+
runtime-critical service client and apply that class's requirements immediately.
|
|
112
|
+
|
|
113
|
+
### 4. Runtime-critical service client
|
|
114
|
+
|
|
115
|
+
**Why we can trust it:** honestly, we mostly don't. This class covers anything
|
|
116
|
+
that reaches users, their data, or money in production — payment SDKs, auth
|
|
117
|
+
libraries, database drivers, service clients on the request path. Even a mature,
|
|
118
|
+
well-run project lands in this class if a bad update reaches production, because
|
|
119
|
+
maturity does not shrink blast radius. Trust here is bought with our own
|
|
120
|
+
evidence and a human's signature, not the vendor's reputation.
|
|
121
|
+
|
|
122
|
+
- **Capability owner:** a named accountable person, not a team.
|
|
123
|
+
- **Update cadence:** re-judged at every version change. Versions are pinned
|
|
124
|
+
exactly, so upgrades are always deliberate.
|
|
125
|
+
- **Detection evidence:** an integration or end-to-end test that exercises the
|
|
126
|
+
real capability, **plus** a production monitor that would notice failure in
|
|
127
|
+
the wild. "Nothing would catch it" is not an acceptable answer in this class —
|
|
128
|
+
it is a blocker.
|
|
129
|
+
- **Replacement-cost threshold:** replacement cost must be written down, never
|
|
130
|
+
`_Not yet decided_`. If nobody can estimate it, we do not know how trapped we
|
|
131
|
+
are, and that unknown is the finding.
|
|
132
|
+
- **Ratification:** **product/human ratification is required** before adding the
|
|
133
|
+
dependency and before any major upgrade.
|
|
134
|
+
|
|
135
|
+
**Human review is triggered when:** any version changes, any security advisory
|
|
136
|
+
is published, or the vendor has a service incident. Every one of those is a
|
|
137
|
+
human decision in this class — that is what makes it the strictest.
|
|
138
|
+
|
|
139
|
+
### 5. Thin wrapper suitable for in-house ownership
|
|
140
|
+
|
|
141
|
+
**Why we can trust it:** it is small enough that we could read all of it in an
|
|
142
|
+
afternoon and own it outright. Safety is not really the question; whether it is
|
|
143
|
+
earning its keep is. A dependency we could write in a day still costs us a
|
|
144
|
+
supply-chain entry, a transitive tree, and an upgrade obligation forever.
|
|
145
|
+
|
|
146
|
+
- **Capability owner:** the team that would inherit the code if we in-housed it.
|
|
147
|
+
If no team will claim it, do not add it.
|
|
148
|
+
- **Update cadence:** re-read at every upgrade — there should be few.
|
|
149
|
+
- **Detection evidence:** our own unit tests covering the wrapped behavior. They
|
|
150
|
+
are cheap here precisely because the surface is small, and they are what makes
|
|
151
|
+
in-housing a one-day job later instead of a rewrite.
|
|
152
|
+
- **Replacement-cost threshold:** roughly one day. Under it, the default is to
|
|
153
|
+
in-house the code rather than take the dependency.
|
|
154
|
+
- **Ratification:** **required to keep it** when replacement cost is under the
|
|
155
|
+
threshold. Note the inversion: the human signs off on the *exception* — taking
|
|
156
|
+
a dependency we could trivially own — not on the removal.
|
|
157
|
+
|
|
158
|
+
**Human review is triggered when:** the package goes unmaintained, or its
|
|
159
|
+
transitive dependency tree grows larger than the code it wraps. Either way we
|
|
160
|
+
are now carrying more supply chain than capability, and in-housing should be
|
|
161
|
+
reconsidered on purpose.
|
|
162
|
+
|
|
163
|
+
### 6. Temporary/experimental dependency
|
|
164
|
+
|
|
165
|
+
**Why we can trust it:** we don't, and we say so. This class exists so a spike,
|
|
166
|
+
a prototype, or a stopgap can move fast without silently becoming permanent. It
|
|
167
|
+
is admitted on a clock: a written expiry date and a named exit — replace,
|
|
168
|
+
in-house, promote to another class, or remove.
|
|
169
|
+
|
|
170
|
+
- **Capability owner:** the person who introduced it, by name. Ownership does
|
|
171
|
+
not transfer to a team while the dependency is in this class.
|
|
172
|
+
- **Update cadence:** reviewed at the stated expiry date, which is at most one
|
|
173
|
+
quarter out. No expiry date means it does not belong in this class.
|
|
174
|
+
- **Detection evidence:** "nothing would catch it" is tolerated **only** while
|
|
175
|
+
exposure stays out of production. The moment it reaches production, the
|
|
176
|
+
runtime-critical requirements apply.
|
|
177
|
+
- **Replacement-cost threshold:** must be low by construction. Anything
|
|
178
|
+
expensive to remove is not temporary — it is a permanent dependency wearing a
|
|
179
|
+
temporary label, and it should be classified honestly on day one.
|
|
180
|
+
- **Ratification:** **required to extend past the expiry date** or to promote it
|
|
181
|
+
into any other class. An expiry that slides without a human saying yes is how
|
|
182
|
+
a prototype becomes production.
|
|
183
|
+
|
|
184
|
+
**Human review is triggered when:** the expiry date passes, or the dependency
|
|
185
|
+
starts running in production or touching user data — whichever happens first.
|
|
186
|
+
|
|
187
|
+
## Ratification, summarized
|
|
188
|
+
|
|
189
|
+
Ratification is not a formality; it is the point where a human owns the residual
|
|
190
|
+
risk. Required for:
|
|
191
|
+
|
|
192
|
+
- **Runtime-critical service client** — to add, and for every major upgrade.
|
|
193
|
+
- **Temporary/experimental dependency** — to extend past expiry or to promote.
|
|
194
|
+
- **Thin wrapper suitable for in-house ownership** — to keep it rather than
|
|
195
|
+
in-house it.
|
|
196
|
+
- **Fast-moving standard implementation** — for a major upgrade that changes
|
|
197
|
+
user-visible behavior.
|
|
198
|
+
|
|
199
|
+
Not required for mature ecosystem primitives or build/development tools on
|
|
200
|
+
routine adds and upgrades. Those two classes are where the crowd and the blast
|
|
201
|
+
radius do the work a human would otherwise have to do.
|
|
202
|
+
|
|
203
|
+
## Naming a class at planning time
|
|
204
|
+
|
|
205
|
+
A work item that proposes adding a new material dependency must, before it is
|
|
206
|
+
buildable, name the dependency's trust class and state that it will update the
|
|
207
|
+
`.lisa/DEPENDENCY_DECISIONS.md` entry in the same change. This is enforced at
|
|
208
|
+
decomposition time by the `lisa-task-decomposition` skill, which rejects a work
|
|
209
|
+
unit proposing a material dependency with no named class.
|
|
210
|
+
|
|
211
|
+
The class is what makes the rest of the ticket reviewable: it tells the reviewer
|
|
212
|
+
which evidence to demand and whether a human has to ratify before the work
|
|
213
|
+
starts, rather than discovering both at the end.
|
|
214
|
+
|
|
215
|
+
Naming a class is not a formality either. If nobody can pick one, that is the
|
|
216
|
+
finding — an unclassifiable dependency is usually one nobody has thought about,
|
|
217
|
+
and it should be resolved before the work item is accepted rather than after the
|
|
218
|
+
package is installed.
|
|
219
|
+
|
|
220
|
+
## Reclassification
|
|
221
|
+
|
|
222
|
+
Trust classes describe the dependency as it is used today, not as it was
|
|
223
|
+
introduced. When exposure changes — a build tool starts running in production, a
|
|
224
|
+
prototype ships to users, a thin wrapper grows a large transitive tree — move it
|
|
225
|
+
to the correct class and apply the new requirements immediately. Update the
|
|
226
|
+
`Last reviewed` date on the record entry when you do.
|
|
227
|
+
|
|
228
|
+
Reclassifying downward (to a lower-trust class) is never a demotion of the
|
|
229
|
+
dependency. It is an admission that its blast radius grew, which is a fact about
|
|
230
|
+
us, not about the maintainers.
|
|
231
|
+
|
|
232
|
+
## Agent parity
|
|
233
|
+
|
|
234
|
+
Trust classes are a governed markdown rule and a decomposition-time expectation,
|
|
235
|
+
not a runtime behavior, so Claude Code, Codex, Cursor, OpenCode, Antigravity,
|
|
236
|
+
and Copilot all reach them identically through the shared rules mirror and the
|
|
237
|
+
same `lisa-task-decomposition` skill.
|
|
238
|
+
|
|
239
|
+
The documented gap, uniform across all six runtimes: no agent runtime enforces
|
|
240
|
+
that a dependency carries a correct class. Nothing fails a build when a package
|
|
241
|
+
is installed with no class named, and nothing detects that a build tool quietly
|
|
242
|
+
started running in production. Classification is carried by this rule, the
|
|
243
|
+
decomposition skill, and review — not by a hook or a lint gate.
|
|
@@ -74,6 +74,31 @@ Each task must have a verification method. Choose the most appropriate:
|
|
|
74
74
|
- Prefer independent tasks that can run in parallel where possible
|
|
75
75
|
- Flag external dependencies (other teams, services, permissions, data) that may block progress
|
|
76
76
|
|
|
77
|
+
### 4.5. Classify Any New Material Dependency
|
|
78
|
+
|
|
79
|
+
Step 4 maps dependencies *between tasks*. This step covers third-party
|
|
80
|
+
dependencies a task proposes to **add**.
|
|
81
|
+
|
|
82
|
+
A dependency is **material** if its failure, disappearance, or bad update would
|
|
83
|
+
break something a user can see, or would cost real time to replace. If a work
|
|
84
|
+
unit proposes adding one, the work unit must, before it is buildable:
|
|
85
|
+
|
|
86
|
+
1. **Name its trust class** — exactly one of: mature ecosystem primitive,
|
|
87
|
+
fast-moving standard implementation, build/development tool, runtime-critical
|
|
88
|
+
service client, thin wrapper suitable for in-house ownership, or
|
|
89
|
+
temporary/experimental dependency. See the `dependency-trust-classes` rule
|
|
90
|
+
for what each class means and why it is trusted.
|
|
91
|
+
2. **State the class's required evidence** — the detection evidence that class
|
|
92
|
+
demands, and whether product/human ratification is required before the work
|
|
93
|
+
starts. Runtime-critical service clients and expiry extensions on temporary
|
|
94
|
+
dependencies always require ratification; a work unit that needs it and does
|
|
95
|
+
not say so is not ready.
|
|
96
|
+
3. **Include updating `.lisa/DEPENDENCY_DECISIONS.md`** in its acceptance
|
|
97
|
+
criteria, so the record entry lands in the same change as the dependency.
|
|
98
|
+
|
|
99
|
+
If nobody can pick a class, that is the finding — resolve it before accepting
|
|
100
|
+
the work unit, not after the package is installed.
|
|
101
|
+
|
|
77
102
|
### 5. Determine Execution Order
|
|
78
103
|
|
|
79
104
|
- Place foundational tasks first (types, schemas, interfaces, shared utilities)
|
|
@@ -121,6 +146,7 @@ Map each task to the skills needed to complete it. This enables delegation to sp
|
|
|
121
146
|
- Every task must have at least one acceptance criterion that can be empirically verified
|
|
122
147
|
- Do not create tasks that cannot be verified -- if you cannot define how to prove it is done, the task is not well-scoped
|
|
123
148
|
- Every Task / Bug / Sub-task / Improvement is scoped to exactly one repo -- if the work spans repos, split into per-repo work units under a shared parent Story (see step 1.5)
|
|
149
|
+
- Any task proposing a new material dependency names its trust class, states that class's required evidence and whether human ratification is needed, and updates `.lisa/DEPENDENCY_DECISIONS.md` in the same change (see step 4.5) -- a proposed material dependency with no named class is not ready to build
|
|
124
150
|
- Keep tasks ordered so that no task references work that has not been completed by a prior task
|
|
125
151
|
- Flag any task that requires access, permissions, or external input not yet available
|
|
126
152
|
- Prefer more small tasks over fewer large tasks -- smaller tasks are easier to verify and less risky to fail
|
|
@@ -0,0 +1,248 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Dependency Trust Classes"
|
|
3
|
+
alwaysApply: false
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Dependency Trust Classes
|
|
7
|
+
|
|
8
|
+
Without named classes, every third-party package sits in one implicit bucket:
|
|
9
|
+
"we installed it, so presumably it's fine." That bucket makes a payment SDK and
|
|
10
|
+
a code formatter look identical at review time, and it forces every dependency
|
|
11
|
+
conversation to re-argue trust from first principles.
|
|
12
|
+
|
|
13
|
+
Trust classes fix that. Each class states, in plain language, **why we can trust
|
|
14
|
+
that kind of dependency**, and names **the one event that would trigger human
|
|
15
|
+
review**. A dependency's class is the answer to the
|
|
16
|
+
"Why we believe it's safe (trust basis)" field in
|
|
17
|
+
`.lisa/DEPENDENCY_DECISIONS.md` — the record says what
|
|
18
|
+
this dependency is and what would catch a bad update; the class says what bar it
|
|
19
|
+
had to clear to be here at all.
|
|
20
|
+
|
|
21
|
+
Exactly one class per dependency. If two seem to fit, take the lower-trust one —
|
|
22
|
+
the requirements are strictly stronger, and being over-careful about a formatter
|
|
23
|
+
costs a review cycle while being under-careful about an auth client costs users.
|
|
24
|
+
|
|
25
|
+
## The five review inputs
|
|
26
|
+
|
|
27
|
+
Every class fixes the same five inputs. What changes between classes is how
|
|
28
|
+
demanding each one is:
|
|
29
|
+
|
|
30
|
+
- **Capability owner** — who is accountable when this breaks. A role for
|
|
31
|
+
high-trust classes; a named person for low-trust ones, because "the platform
|
|
32
|
+
team" cannot be paged at 2am.
|
|
33
|
+
- **Update cadence** — how often the entry is re-read and the version re-judged.
|
|
34
|
+
- **Detection evidence** — the named check that fails if an update breaks the
|
|
35
|
+
owned capability. Stronger classes demand evidence *we* own, not upstream's
|
|
36
|
+
test suite.
|
|
37
|
+
- **Replacement-cost threshold** — the point past which the cost itself is the
|
|
38
|
+
finding. High replacement cost is acceptable in a high-trust class and is an
|
|
39
|
+
escalation in a low-trust one.
|
|
40
|
+
- **Product/human ratification** — whether a human has to say yes before the
|
|
41
|
+
dependency is added, upgraded, or kept. This is the gate, and it is deliberate
|
|
42
|
+
that the lower-trust and higher-exposure classes carry it.
|
|
43
|
+
|
|
44
|
+
## The six classes
|
|
45
|
+
|
|
46
|
+
### 1. Mature ecosystem primitive
|
|
47
|
+
|
|
48
|
+
**Why we can trust it:** so many other projects depend on it that a bad release
|
|
49
|
+
is usually found, reported, and fixed by someone else before it reaches us. It
|
|
50
|
+
has an identifiable maintaining organization, a published release cadence, and a
|
|
51
|
+
documented process for handling security reports. Trust here comes from the size
|
|
52
|
+
of the crowd standing in front of us, not from us having read the code.
|
|
53
|
+
|
|
54
|
+
Examples in kind: a language's canonical test runner, a foundation-maintained
|
|
55
|
+
linter, a mainstream UI framework.
|
|
56
|
+
|
|
57
|
+
- **Capability owner:** a role or team is sufficient.
|
|
58
|
+
- **Update cadence:** re-read at each major upgrade, and at least every 12
|
|
59
|
+
months.
|
|
60
|
+
- **Detection evidence:** an existing CI check must fail if the owned capability
|
|
61
|
+
breaks. Naming an upstream test suite is acceptable here and only here.
|
|
62
|
+
- **Replacement-cost threshold:** high replacement cost is accepted. That is the
|
|
63
|
+
bargain — we take deep coupling in exchange for the crowd's scrutiny.
|
|
64
|
+
- **Ratification:** not required to add or to take a minor/patch upgrade.
|
|
65
|
+
|
|
66
|
+
**Human review is triggered when:** maintainership changes hands or the project
|
|
67
|
+
is archived, or a security advisory goes unfixed past the project's own
|
|
68
|
+
published response window. Either event removes the crowd, which was the whole
|
|
69
|
+
basis for the trust — reclassify rather than re-trust.
|
|
70
|
+
|
|
71
|
+
### 2. Fast-moving standard implementation
|
|
72
|
+
|
|
73
|
+
**Why we can trust it:** it is the standard implementation of a protocol, spec,
|
|
74
|
+
or platform we have deliberately decided not to own — a cloud SDK, an API
|
|
75
|
+
client, a framework tracking a moving platform. We trust the *standard*; we do
|
|
76
|
+
not trust the release we happen to be on, because the project moves faster than
|
|
77
|
+
our review cycle does. Trust here is conditional on us keeping up.
|
|
78
|
+
|
|
79
|
+
- **Capability owner:** a named team that owns staying current.
|
|
80
|
+
- **Update cadence:** re-read at every minor upgrade, and no less than
|
|
81
|
+
quarterly. A dependency in this class going quiet for a year is itself a
|
|
82
|
+
finding.
|
|
83
|
+
- **Detection evidence:** an automated test **we** own that exercises the owned
|
|
84
|
+
capability. Upstream's suite does not count — it tests their contract, not our
|
|
85
|
+
use of it.
|
|
86
|
+
- **Replacement-cost threshold:** replacement cost must stay moderate. Keep the
|
|
87
|
+
coupling behind our own boundary so a forced migration is a rewrite of the
|
|
88
|
+
boundary, not of the application.
|
|
89
|
+
- **Ratification:** required for any major upgrade that changes user-visible
|
|
90
|
+
behavior.
|
|
91
|
+
|
|
92
|
+
**Human review is triggered when:** two consecutive major versions ship that we
|
|
93
|
+
did not adopt inside the cadence window. At that point we are pinned to
|
|
94
|
+
unsupported software and the decision — catch up, replace, or accept the
|
|
95
|
+
exposure — belongs to a human.
|
|
96
|
+
|
|
97
|
+
### 3. Build/development tool
|
|
98
|
+
|
|
99
|
+
**Why we can trust it:** it never runs in production and never touches customer
|
|
100
|
+
data. If it ships a bad update, developers notice within one build, and the
|
|
101
|
+
worst case is lost engineering time rather than a user-visible failure. The
|
|
102
|
+
blast radius is what earns the trust, not the maintainer.
|
|
103
|
+
|
|
104
|
+
- **Capability owner:** a role or team is sufficient.
|
|
105
|
+
- **Update cadence:** re-read at each major upgrade, and at least every 12
|
|
106
|
+
months.
|
|
107
|
+
- **Detection evidence:** the build, lint, or test job the tool powers. It fails
|
|
108
|
+
loudly and immediately, which is the point.
|
|
109
|
+
- **Replacement-cost threshold:** high cost is accepted — swapping a build tool
|
|
110
|
+
is disruptive but never urgent.
|
|
111
|
+
- **Ratification:** not required.
|
|
112
|
+
|
|
113
|
+
**Human review is triggered when:** the tool gains reach it did not have —
|
|
114
|
+
running in production, reading secrets, or reading CI credentials. The moment
|
|
115
|
+
that happens the blast-radius argument is void: reclassify it as a
|
|
116
|
+
runtime-critical service client and apply that class's requirements immediately.
|
|
117
|
+
|
|
118
|
+
### 4. Runtime-critical service client
|
|
119
|
+
|
|
120
|
+
**Why we can trust it:** honestly, we mostly don't. This class covers anything
|
|
121
|
+
that reaches users, their data, or money in production — payment SDKs, auth
|
|
122
|
+
libraries, database drivers, service clients on the request path. Even a mature,
|
|
123
|
+
well-run project lands in this class if a bad update reaches production, because
|
|
124
|
+
maturity does not shrink blast radius. Trust here is bought with our own
|
|
125
|
+
evidence and a human's signature, not the vendor's reputation.
|
|
126
|
+
|
|
127
|
+
- **Capability owner:** a named accountable person, not a team.
|
|
128
|
+
- **Update cadence:** re-judged at every version change. Versions are pinned
|
|
129
|
+
exactly, so upgrades are always deliberate.
|
|
130
|
+
- **Detection evidence:** an integration or end-to-end test that exercises the
|
|
131
|
+
real capability, **plus** a production monitor that would notice failure in
|
|
132
|
+
the wild. "Nothing would catch it" is not an acceptable answer in this class —
|
|
133
|
+
it is a blocker.
|
|
134
|
+
- **Replacement-cost threshold:** replacement cost must be written down, never
|
|
135
|
+
`_Not yet decided_`. If nobody can estimate it, we do not know how trapped we
|
|
136
|
+
are, and that unknown is the finding.
|
|
137
|
+
- **Ratification:** **product/human ratification is required** before adding the
|
|
138
|
+
dependency and before any major upgrade.
|
|
139
|
+
|
|
140
|
+
**Human review is triggered when:** any version changes, any security advisory
|
|
141
|
+
is published, or the vendor has a service incident. Every one of those is a
|
|
142
|
+
human decision in this class — that is what makes it the strictest.
|
|
143
|
+
|
|
144
|
+
### 5. Thin wrapper suitable for in-house ownership
|
|
145
|
+
|
|
146
|
+
**Why we can trust it:** it is small enough that we could read all of it in an
|
|
147
|
+
afternoon and own it outright. Safety is not really the question; whether it is
|
|
148
|
+
earning its keep is. A dependency we could write in a day still costs us a
|
|
149
|
+
supply-chain entry, a transitive tree, and an upgrade obligation forever.
|
|
150
|
+
|
|
151
|
+
- **Capability owner:** the team that would inherit the code if we in-housed it.
|
|
152
|
+
If no team will claim it, do not add it.
|
|
153
|
+
- **Update cadence:** re-read at every upgrade — there should be few.
|
|
154
|
+
- **Detection evidence:** our own unit tests covering the wrapped behavior. They
|
|
155
|
+
are cheap here precisely because the surface is small, and they are what makes
|
|
156
|
+
in-housing a one-day job later instead of a rewrite.
|
|
157
|
+
- **Replacement-cost threshold:** roughly one day. Under it, the default is to
|
|
158
|
+
in-house the code rather than take the dependency.
|
|
159
|
+
- **Ratification:** **required to keep it** when replacement cost is under the
|
|
160
|
+
threshold. Note the inversion: the human signs off on the *exception* — taking
|
|
161
|
+
a dependency we could trivially own — not on the removal.
|
|
162
|
+
|
|
163
|
+
**Human review is triggered when:** the package goes unmaintained, or its
|
|
164
|
+
transitive dependency tree grows larger than the code it wraps. Either way we
|
|
165
|
+
are now carrying more supply chain than capability, and in-housing should be
|
|
166
|
+
reconsidered on purpose.
|
|
167
|
+
|
|
168
|
+
### 6. Temporary/experimental dependency
|
|
169
|
+
|
|
170
|
+
**Why we can trust it:** we don't, and we say so. This class exists so a spike,
|
|
171
|
+
a prototype, or a stopgap can move fast without silently becoming permanent. It
|
|
172
|
+
is admitted on a clock: a written expiry date and a named exit — replace,
|
|
173
|
+
in-house, promote to another class, or remove.
|
|
174
|
+
|
|
175
|
+
- **Capability owner:** the person who introduced it, by name. Ownership does
|
|
176
|
+
not transfer to a team while the dependency is in this class.
|
|
177
|
+
- **Update cadence:** reviewed at the stated expiry date, which is at most one
|
|
178
|
+
quarter out. No expiry date means it does not belong in this class.
|
|
179
|
+
- **Detection evidence:** "nothing would catch it" is tolerated **only** while
|
|
180
|
+
exposure stays out of production. The moment it reaches production, the
|
|
181
|
+
runtime-critical requirements apply.
|
|
182
|
+
- **Replacement-cost threshold:** must be low by construction. Anything
|
|
183
|
+
expensive to remove is not temporary — it is a permanent dependency wearing a
|
|
184
|
+
temporary label, and it should be classified honestly on day one.
|
|
185
|
+
- **Ratification:** **required to extend past the expiry date** or to promote it
|
|
186
|
+
into any other class. An expiry that slides without a human saying yes is how
|
|
187
|
+
a prototype becomes production.
|
|
188
|
+
|
|
189
|
+
**Human review is triggered when:** the expiry date passes, or the dependency
|
|
190
|
+
starts running in production or touching user data — whichever happens first.
|
|
191
|
+
|
|
192
|
+
## Ratification, summarized
|
|
193
|
+
|
|
194
|
+
Ratification is not a formality; it is the point where a human owns the residual
|
|
195
|
+
risk. Required for:
|
|
196
|
+
|
|
197
|
+
- **Runtime-critical service client** — to add, and for every major upgrade.
|
|
198
|
+
- **Temporary/experimental dependency** — to extend past expiry or to promote.
|
|
199
|
+
- **Thin wrapper suitable for in-house ownership** — to keep it rather than
|
|
200
|
+
in-house it.
|
|
201
|
+
- **Fast-moving standard implementation** — for a major upgrade that changes
|
|
202
|
+
user-visible behavior.
|
|
203
|
+
|
|
204
|
+
Not required for mature ecosystem primitives or build/development tools on
|
|
205
|
+
routine adds and upgrades. Those two classes are where the crowd and the blast
|
|
206
|
+
radius do the work a human would otherwise have to do.
|
|
207
|
+
|
|
208
|
+
## Naming a class at planning time
|
|
209
|
+
|
|
210
|
+
A work item that proposes adding a new material dependency must, before it is
|
|
211
|
+
buildable, name the dependency's trust class and state that it will update the
|
|
212
|
+
`.lisa/DEPENDENCY_DECISIONS.md` entry in the same change. This is enforced at
|
|
213
|
+
decomposition time by the `lisa-task-decomposition` skill, which rejects a work
|
|
214
|
+
unit proposing a material dependency with no named class.
|
|
215
|
+
|
|
216
|
+
The class is what makes the rest of the ticket reviewable: it tells the reviewer
|
|
217
|
+
which evidence to demand and whether a human has to ratify before the work
|
|
218
|
+
starts, rather than discovering both at the end.
|
|
219
|
+
|
|
220
|
+
Naming a class is not a formality either. If nobody can pick one, that is the
|
|
221
|
+
finding — an unclassifiable dependency is usually one nobody has thought about,
|
|
222
|
+
and it should be resolved before the work item is accepted rather than after the
|
|
223
|
+
package is installed.
|
|
224
|
+
|
|
225
|
+
## Reclassification
|
|
226
|
+
|
|
227
|
+
Trust classes describe the dependency as it is used today, not as it was
|
|
228
|
+
introduced. When exposure changes — a build tool starts running in production, a
|
|
229
|
+
prototype ships to users, a thin wrapper grows a large transitive tree — move it
|
|
230
|
+
to the correct class and apply the new requirements immediately. Update the
|
|
231
|
+
`Last reviewed` date on the record entry when you do.
|
|
232
|
+
|
|
233
|
+
Reclassifying downward (to a lower-trust class) is never a demotion of the
|
|
234
|
+
dependency. It is an admission that its blast radius grew, which is a fact about
|
|
235
|
+
us, not about the maintainers.
|
|
236
|
+
|
|
237
|
+
## Agent parity
|
|
238
|
+
|
|
239
|
+
Trust classes are a governed markdown rule and a decomposition-time expectation,
|
|
240
|
+
not a runtime behavior, so Claude Code, Codex, Cursor, OpenCode, Antigravity,
|
|
241
|
+
and Copilot all reach them identically through the shared rules mirror and the
|
|
242
|
+
same `lisa-task-decomposition` skill.
|
|
243
|
+
|
|
244
|
+
The documented gap, uniform across all six runtimes: no agent runtime enforces
|
|
245
|
+
that a dependency carries a correct class. Nothing fails a build when a package
|
|
246
|
+
is installed with no class named, and nothing detects that a build tool quietly
|
|
247
|
+
started running in production. Classification is carried by this rule, the
|
|
248
|
+
decomposition skill, and review — not by a hook or a lint gate.
|
|
@@ -0,0 +1,48 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: "Dependency Trust Classes (load-bearing)"
|
|
3
|
+
alwaysApply: true
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
# Dependency Trust Classes (load-bearing)
|
|
7
|
+
|
|
8
|
+
Every material dependency belongs to exactly one **trust class**. The class is
|
|
9
|
+
the answer to the decision record's "Why we believe it's safe (trust basis)"
|
|
10
|
+
field in `.lisa/DEPENDENCY_DECISIONS.md` — it says why we can trust a *kind* of
|
|
11
|
+
dependency, so each entry does not re-argue trust from scratch.
|
|
12
|
+
|
|
13
|
+
The six classes, from most to least self-evidently trustworthy:
|
|
14
|
+
|
|
15
|
+
1. **Mature ecosystem primitive** — so widely used that other people hit the
|
|
16
|
+
breakage first.
|
|
17
|
+
2. **Fast-moving standard implementation** — the standard implementation of
|
|
18
|
+
something we refuse to own, but it changes faster than we do.
|
|
19
|
+
3. **Build/development tool** — never runs in production; a bad update costs
|
|
20
|
+
developer time, not users.
|
|
21
|
+
4. **Runtime-critical service client** — reaches users, data, or money directly,
|
|
22
|
+
so it is the least trusted regardless of how mature it is.
|
|
23
|
+
5. **Thin wrapper suitable for in-house ownership** — small enough that we could
|
|
24
|
+
own the code, so the question is whether it earns its keep, not whether it is
|
|
25
|
+
safe.
|
|
26
|
+
6. **Temporary/experimental dependency** — not trusted at all; admitted on a
|
|
27
|
+
clock with a written expiry date and a named exit.
|
|
28
|
+
|
|
29
|
+
Each class fixes five review inputs: capability owner, update cadence, detection
|
|
30
|
+
evidence, replacement-cost threshold, and whether product/human ratification is
|
|
31
|
+
required. Higher exposure and lower trust buy stronger evidence and mandatory
|
|
32
|
+
ratification, not weaker: runtime-critical service clients always need human
|
|
33
|
+
ratification to add or major-upgrade; temporary dependencies need it to live
|
|
34
|
+
past their expiry date; thin wrappers need it to be kept rather than in-housed.
|
|
35
|
+
|
|
36
|
+
Every class also names one **human-review trigger** — the observable event that
|
|
37
|
+
takes the decision away from the agent. A class definition that cannot say what
|
|
38
|
+
would trigger human review is not finished.
|
|
39
|
+
|
|
40
|
+
A work item that proposes adding a new material dependency must name the
|
|
41
|
+
dependency's trust class and update its `.lisa/DEPENDENCY_DECISIONS.md` entry in
|
|
42
|
+
the same change. A proposal with no named class is not ready to build.
|
|
43
|
+
|
|
44
|
+
Reclassify — do not quietly re-trust — when a dependency's exposure changes: a
|
|
45
|
+
build tool that gains runtime reach or reads CI credentials becomes a
|
|
46
|
+
runtime-critical service client, and its stronger requirements apply immediately.
|
|
47
|
+
|
|
48
|
+
Full prose: [reference/dependency-trust-classes.md](dependency-trust-classes-reference.mdc).
|
|
@@ -74,6 +74,31 @@ Each task must have a verification method. Choose the most appropriate:
|
|
|
74
74
|
- Prefer independent tasks that can run in parallel where possible
|
|
75
75
|
- Flag external dependencies (other teams, services, permissions, data) that may block progress
|
|
76
76
|
|
|
77
|
+
### 4.5. Classify Any New Material Dependency
|
|
78
|
+
|
|
79
|
+
Step 4 maps dependencies *between tasks*. This step covers third-party
|
|
80
|
+
dependencies a task proposes to **add**.
|
|
81
|
+
|
|
82
|
+
A dependency is **material** if its failure, disappearance, or bad update would
|
|
83
|
+
break something a user can see, or would cost real time to replace. If a work
|
|
84
|
+
unit proposes adding one, the work unit must, before it is buildable:
|
|
85
|
+
|
|
86
|
+
1. **Name its trust class** — exactly one of: mature ecosystem primitive,
|
|
87
|
+
fast-moving standard implementation, build/development tool, runtime-critical
|
|
88
|
+
service client, thin wrapper suitable for in-house ownership, or
|
|
89
|
+
temporary/experimental dependency. See the `dependency-trust-classes` rule
|
|
90
|
+
for what each class means and why it is trusted.
|
|
91
|
+
2. **State the class's required evidence** — the detection evidence that class
|
|
92
|
+
demands, and whether product/human ratification is required before the work
|
|
93
|
+
starts. Runtime-critical service clients and expiry extensions on temporary
|
|
94
|
+
dependencies always require ratification; a work unit that needs it and does
|
|
95
|
+
not say so is not ready.
|
|
96
|
+
3. **Include updating `.lisa/DEPENDENCY_DECISIONS.md`** in its acceptance
|
|
97
|
+
criteria, so the record entry lands in the same change as the dependency.
|
|
98
|
+
|
|
99
|
+
If nobody can pick a class, that is the finding — resolve it before accepting
|
|
100
|
+
the work unit, not after the package is installed.
|
|
101
|
+
|
|
77
102
|
### 5. Determine Execution Order
|
|
78
103
|
|
|
79
104
|
- Place foundational tasks first (types, schemas, interfaces, shared utilities)
|
|
@@ -121,6 +146,7 @@ Map each task to the skills needed to complete it. This enables delegation to sp
|
|
|
121
146
|
- Every task must have at least one acceptance criterion that can be empirically verified
|
|
122
147
|
- Do not create tasks that cannot be verified -- if you cannot define how to prove it is done, the task is not well-scoped
|
|
123
148
|
- Every Task / Bug / Sub-task / Improvement is scoped to exactly one repo -- if the work spans repos, split into per-repo work units under a shared parent Story (see step 1.5)
|
|
149
|
+
- Any task proposing a new material dependency names its trust class, states that class's required evidence and whether human ratification is needed, and updates `.lisa/DEPENDENCY_DECISIONS.md` in the same change (see step 4.5) -- a proposed material dependency with no named class is not ready to build
|
|
124
150
|
- Keep tasks ordered so that no task references work that has not been completed by a prior task
|
|
125
151
|
- Flag any task that requires access, permissions, or external input not yet available
|
|
126
152
|
- Prefer more small tasks over fewer large tasks -- smaller tasks are easier to verify and less risky to fail
|