@ontrails/warden 1.0.0-beta.2 → 1.0.0-beta.22
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 +508 -6
- package/README.md +77 -26
- package/bin/warden.ts +50 -0
- package/package.json +27 -5
- package/src/adapter-check.ts +136 -0
- package/src/ast.ts +28 -0
- package/src/cli.ts +1374 -103
- package/src/command.ts +953 -0
- package/src/config.ts +184 -0
- package/src/draft.ts +22 -0
- package/src/drift.ts +106 -22
- package/src/fix.ts +120 -0
- package/src/formatters.ts +79 -9
- package/src/guide.ts +245 -0
- package/src/index.ts +206 -14
- package/src/project-context.ts +163 -0
- package/src/resolve.ts +530 -0
- package/src/rules/activation-orphan.ts +97 -0
- package/src/rules/ast.ts +3176 -85
- package/src/rules/circular-refs.ts +154 -0
- package/src/rules/composes-declarations.ts +704 -0
- package/src/rules/context-no-surface-types.ts +68 -8
- package/src/rules/contour-exists.ts +251 -0
- package/src/rules/contour-ids.ts +15 -0
- package/src/rules/dead-internal-trail.ts +154 -0
- package/src/rules/draft-file-marking.ts +160 -0
- package/src/rules/draft-visible-debt.ts +87 -0
- package/src/rules/error-mapping-completeness.ts +288 -0
- package/src/rules/example-valid.ts +401 -0
- package/src/rules/fires-declarations.ts +758 -0
- package/src/rules/implementation-returns-result.ts +1265 -95
- package/src/rules/incomplete-accessor-for-standard-op.ts +272 -0
- package/src/rules/incomplete-crud.ts +580 -0
- package/src/rules/index.ts +219 -18
- package/src/rules/intent-propagation.ts +127 -0
- package/src/rules/layer-field-name-drift.ts +96 -0
- package/src/rules/metadata.ts +654 -0
- package/src/rules/missing-reconcile.ts +98 -0
- package/src/rules/missing-visibility.ts +110 -0
- package/src/rules/no-destructured-compose.ts +192 -0
- package/src/rules/no-dev-permit-in-source.ts +99 -0
- package/src/rules/no-direct-implementation-call.ts +7 -7
- package/src/rules/no-legacy-layer-imports.ts +211 -0
- package/src/rules/no-native-error-result.ts +111 -0
- package/src/rules/no-redundant-result-error-wrap.ts +331 -0
- package/src/rules/no-retired-cross-vocabulary.ts +194 -0
- package/src/rules/no-sync-result-assumption.ts +1134 -99
- package/src/rules/no-throw-in-detour-recover.ts +225 -0
- package/src/rules/no-throw-in-implementation.ts +10 -9
- package/src/rules/no-top-level-surface.ts +389 -0
- package/src/rules/on-references-exist.ts +194 -0
- package/src/rules/orphaned-signal.ts +150 -0
- package/src/rules/owner-projection-parity.ts +146 -0
- package/src/rules/permit-governance.ts +25 -0
- package/src/rules/public-export-example-coverage.ts +553 -0
- package/src/rules/public-internal-deep-imports.ts +517 -0
- package/src/rules/public-output-schema.ts +29 -0
- package/src/rules/public-union-output-discriminants.ts +150 -0
- package/src/rules/read-intent-fires.ts +187 -0
- package/src/rules/reference-exists.ts +98 -0
- package/src/rules/registry-names.ts +145 -0
- package/src/rules/resolved-import-boundary.ts +146 -0
- package/src/rules/resource-declarations.ts +704 -0
- package/src/rules/resource-exists.ts +179 -0
- package/src/rules/resource-id-grammar.ts +65 -0
- package/src/rules/resource-mock-coverage.ts +115 -0
- package/src/rules/scan.ts +38 -25
- package/src/rules/scheduled-destroy-intent.ts +44 -0
- package/src/rules/signal-graph-coaching.ts +191 -0
- package/src/rules/specs.ts +9 -5
- package/src/rules/static-resource-accessor-preference.ts +657 -0
- package/src/rules/surface-facet-coherence.ts +370 -0
- package/src/rules/trail-versioning-source.ts +1094 -0
- package/src/rules/trail-versioning-topo.ts +172 -0
- package/src/rules/types.ts +270 -6
- package/src/rules/unmaterialized-activation-source.ts +84 -0
- package/src/rules/unreachable-detour-shadowing.ts +344 -0
- package/src/rules/valid-describe-refs.ts +160 -32
- package/src/rules/valid-detour-contract.ts +78 -0
- package/src/rules/warden-export-symmetry.ts +533 -0
- package/src/rules/warden-rules-use-ast.ts +996 -0
- package/src/rules/webhook-route-collision.ts +243 -0
- package/src/trails/activation-orphan.trail.ts +84 -0
- package/src/trails/circular-refs.trail.ts +29 -0
- package/src/trails/composes-declarations.trail.ts +22 -0
- package/src/trails/context-no-surface-types.trail.ts +21 -0
- package/src/trails/contour-exists.trail.ts +21 -0
- package/src/trails/dead-internal-trail.trail.ts +26 -0
- package/src/trails/deprecation-without-guidance.trail.ts +21 -0
- package/src/trails/draft-file-marking.trail.ts +16 -0
- package/src/trails/draft-visible-debt.trail.ts +16 -0
- package/src/trails/error-mapping-completeness.trail.ts +29 -0
- package/src/trails/example-valid.trail.ts +25 -0
- package/src/trails/fires-declarations.trail.ts +23 -0
- package/src/trails/fork-without-preserved-blaze.trail.ts +31 -0
- package/src/trails/implementation-returns-result.trail.ts +20 -0
- package/src/trails/incomplete-accessor-for-standard-op.trail.ts +76 -0
- package/src/trails/incomplete-crud.trail.ts +39 -0
- package/src/trails/index.ts +78 -0
- package/src/trails/intent-propagation.trail.ts +30 -0
- package/src/trails/layer-field-name-drift.trail.ts +39 -0
- package/src/trails/marker-schema-unsupported.trail.ts +23 -0
- package/src/trails/missing-reconcile.trail.ts +33 -0
- package/src/trails/missing-visibility.trail.ts +22 -0
- package/src/trails/no-destructured-compose.trail.ts +44 -0
- package/src/trails/no-dev-permit-in-source.trail.ts +16 -0
- package/src/trails/no-direct-implementation-call.trail.ts +16 -0
- package/src/trails/no-legacy-layer-imports.trail.ts +41 -0
- package/src/trails/no-native-error-result.trail.ts +18 -0
- package/src/trails/no-redundant-result-error-wrap.trail.ts +55 -0
- package/src/trails/no-retired-cross-vocabulary.trail.ts +42 -0
- package/src/trails/no-sync-result-assumption.trail.ts +19 -0
- package/src/trails/no-throw-in-detour-recover.trail.ts +24 -0
- package/src/trails/no-throw-in-implementation.trail.ts +20 -0
- package/src/trails/no-top-level-surface.trail.ts +43 -0
- package/src/trails/on-references-exist.trail.ts +21 -0
- package/src/trails/orphaned-signal.trail.ts +36 -0
- package/src/trails/owner-projection-parity.trail.ts +26 -0
- package/src/trails/pending-force.trail.ts +21 -0
- package/src/trails/permit-governance.trail.ts +51 -0
- package/src/trails/prefer-schema-inference.trail.ts +21 -0
- package/src/trails/public-export-example-coverage.trail.ts +16 -0
- package/src/trails/public-internal-deep-imports.trail.ts +94 -0
- package/src/trails/public-output-schema.trail.ts +55 -0
- package/src/trails/public-union-output-discriminants.trail.ts +33 -0
- package/src/trails/read-intent-fires.trail.ts +20 -0
- package/src/trails/reference-exists.trail.ts +25 -0
- package/src/trails/resolved-import-boundary.trail.ts +109 -0
- package/src/trails/resource-declarations.trail.ts +25 -0
- package/src/trails/resource-exists.trail.ts +27 -0
- package/src/trails/resource-id-grammar.trail.ts +39 -0
- package/src/trails/resource-mock-coverage.trail.ts +40 -0
- package/src/trails/run.ts +162 -0
- package/src/trails/scheduled-destroy-intent.trail.ts +56 -0
- package/src/trails/schema.ts +194 -0
- package/src/trails/signal-graph-coaching.trail.ts +77 -0
- package/src/trails/static-resource-accessor-preference.trail.ts +25 -0
- package/src/trails/surface-facet-coherence.trail.ts +25 -0
- package/src/trails/topo.ts +6 -0
- package/src/trails/unmaterialized-activation-source.trail.ts +72 -0
- package/src/trails/unreachable-detour-shadowing.trail.ts +45 -0
- package/src/trails/valid-describe-refs.trail.ts +18 -0
- package/src/trails/valid-detour-contract.trail.ts +71 -0
- package/src/trails/version-gap.trail.ts +35 -0
- package/src/trails/version-pinned-compose.trail.ts +23 -0
- package/src/trails/version-without-examples.trail.ts +38 -0
- package/src/trails/warden-export-symmetry.trail.ts +16 -0
- package/src/trails/warden-rules-use-ast.trail.ts +45 -0
- package/src/trails/webhook-route-collision.trail.ts +50 -0
- package/src/trails/wrap-rule.ts +213 -0
- package/src/workspaces.ts +238 -0
- package/.turbo/turbo-build.log +0 -1
- package/.turbo/turbo-lint.log +0 -3
- package/.turbo/turbo-typecheck.log +0 -1
- package/dist/cli.d.ts +0 -46
- package/dist/cli.d.ts.map +0 -1
- package/dist/cli.js +0 -221
- package/dist/cli.js.map +0 -1
- package/dist/drift.d.ts +0 -26
- package/dist/drift.d.ts.map +0 -1
- package/dist/drift.js +0 -27
- package/dist/drift.js.map +0 -1
- package/dist/formatters.d.ts +0 -29
- package/dist/formatters.d.ts.map +0 -1
- package/dist/formatters.js +0 -87
- package/dist/formatters.js.map +0 -1
- package/dist/index.d.ts +0 -26
- package/dist/index.d.ts.map +0 -1
- package/dist/index.js +0 -26
- package/dist/index.js.map +0 -1
- package/dist/rules/ast.d.ts +0 -41
- package/dist/rules/ast.d.ts.map +0 -1
- package/dist/rules/ast.js +0 -163
- package/dist/rules/ast.js.map +0 -1
- package/dist/rules/context-no-surface-types.d.ts +0 -12
- package/dist/rules/context-no-surface-types.d.ts.map +0 -1
- package/dist/rules/context-no-surface-types.js +0 -96
- package/dist/rules/context-no-surface-types.js.map +0 -1
- package/dist/rules/implementation-returns-result.d.ts +0 -13
- package/dist/rules/implementation-returns-result.d.ts.map +0 -1
- package/dist/rules/implementation-returns-result.js +0 -231
- package/dist/rules/implementation-returns-result.js.map +0 -1
- package/dist/rules/index.d.ts +0 -22
- package/dist/rules/index.d.ts.map +0 -1
- package/dist/rules/index.js +0 -41
- package/dist/rules/index.js.map +0 -1
- package/dist/rules/no-direct-impl-in-route.d.ts +0 -12
- package/dist/rules/no-direct-impl-in-route.d.ts.map +0 -1
- package/dist/rules/no-direct-impl-in-route.js +0 -46
- package/dist/rules/no-direct-impl-in-route.js.map +0 -1
- package/dist/rules/no-direct-implementation-call.d.ts +0 -12
- package/dist/rules/no-direct-implementation-call.d.ts.map +0 -1
- package/dist/rules/no-direct-implementation-call.js +0 -39
- package/dist/rules/no-direct-implementation-call.js.map +0 -1
- package/dist/rules/no-sync-result-assumption.d.ts +0 -6
- package/dist/rules/no-sync-result-assumption.d.ts.map +0 -1
- package/dist/rules/no-sync-result-assumption.js +0 -98
- package/dist/rules/no-sync-result-assumption.js.map +0 -1
- package/dist/rules/no-throw-in-detour-target.d.ts +0 -12
- package/dist/rules/no-throw-in-detour-target.d.ts.map +0 -1
- package/dist/rules/no-throw-in-detour-target.js +0 -87
- package/dist/rules/no-throw-in-detour-target.js.map +0 -1
- package/dist/rules/no-throw-in-implementation.d.ts +0 -9
- package/dist/rules/no-throw-in-implementation.d.ts.map +0 -1
- package/dist/rules/no-throw-in-implementation.js +0 -34
- package/dist/rules/no-throw-in-implementation.js.map +0 -1
- package/dist/rules/prefer-schema-inference.d.ts +0 -7
- package/dist/rules/prefer-schema-inference.d.ts.map +0 -1
- package/dist/rules/prefer-schema-inference.js +0 -86
- package/dist/rules/prefer-schema-inference.js.map +0 -1
- package/dist/rules/scan.d.ts +0 -8
- package/dist/rules/scan.d.ts.map +0 -1
- package/dist/rules/scan.js +0 -32
- package/dist/rules/scan.js.map +0 -1
- package/dist/rules/specs.d.ts +0 -29
- package/dist/rules/specs.d.ts.map +0 -1
- package/dist/rules/specs.js +0 -192
- package/dist/rules/specs.js.map +0 -1
- package/dist/rules/structure.d.ts +0 -13
- package/dist/rules/structure.d.ts.map +0 -1
- package/dist/rules/structure.js +0 -142
- package/dist/rules/structure.js.map +0 -1
- package/dist/rules/types.d.ts +0 -52
- package/dist/rules/types.d.ts.map +0 -1
- package/dist/rules/types.js +0 -2
- package/dist/rules/types.js.map +0 -1
- package/dist/rules/valid-describe-refs.d.ts +0 -7
- package/dist/rules/valid-describe-refs.d.ts.map +0 -1
- package/dist/rules/valid-describe-refs.js +0 -51
- package/dist/rules/valid-describe-refs.js.map +0 -1
- package/dist/rules/valid-detour-refs.d.ts +0 -6
- package/dist/rules/valid-detour-refs.d.ts.map +0 -1
- package/dist/rules/valid-detour-refs.js +0 -116
- package/dist/rules/valid-detour-refs.js.map +0 -1
- package/src/__tests__/cli.test.ts +0 -198
- package/src/__tests__/drift.test.ts +0 -74
- package/src/__tests__/formatters.test.ts +0 -157
- package/src/__tests__/implementation-returns-result.test.ts +0 -75
- package/src/__tests__/no-direct-implementation-call.test.ts +0 -83
- package/src/__tests__/no-sync-result-assumption.test.ts +0 -85
- package/src/__tests__/no-throw-in-detour-target.test.ts +0 -78
- package/src/__tests__/prefer-schema-inference.test.ts +0 -84
- package/src/__tests__/rules.test.ts +0 -188
- package/src/__tests__/valid-describe-refs.test.ts +0 -60
- package/src/rules/no-direct-impl-in-route.ts +0 -77
- package/src/rules/no-throw-in-detour-target.ts +0 -150
- package/src/rules/valid-detour-refs.ts +0 -187
- package/tsconfig.json +0 -9
- package/tsconfig.tsbuildinfo +0 -1
package/src/guide.ts
ADDED
|
@@ -0,0 +1,245 @@
|
|
|
1
|
+
import type {
|
|
2
|
+
WardenFixCapability,
|
|
3
|
+
WardenGuidance,
|
|
4
|
+
WardenGuidanceLink,
|
|
5
|
+
WardenRuleConcern,
|
|
6
|
+
WardenRuleLifecycle,
|
|
7
|
+
WardenRuleScope,
|
|
8
|
+
WardenRuleTier,
|
|
9
|
+
WardenSeverity,
|
|
10
|
+
} from './rules/index.js';
|
|
11
|
+
import {
|
|
12
|
+
listWardenRuleMetadata,
|
|
13
|
+
wardenRules,
|
|
14
|
+
wardenTopoRules,
|
|
15
|
+
} from './rules/index.js';
|
|
16
|
+
import type { WardenDepth } from './config.js';
|
|
17
|
+
|
|
18
|
+
export const wardenGuideFormatValues = [
|
|
19
|
+
'markdown',
|
|
20
|
+
'agent-json',
|
|
21
|
+
'manifest',
|
|
22
|
+
] as const;
|
|
23
|
+
|
|
24
|
+
export type WardenGuideFormat = (typeof wardenGuideFormatValues)[number];
|
|
25
|
+
|
|
26
|
+
export interface WardenRuleGuideEntry {
|
|
27
|
+
readonly concern: WardenRuleConcern;
|
|
28
|
+
readonly depth: WardenDepth;
|
|
29
|
+
readonly description: string;
|
|
30
|
+
readonly docs: readonly WardenGuidanceLink[];
|
|
31
|
+
readonly fix?: WardenFixCapability | undefined;
|
|
32
|
+
readonly guidance?: WardenGuidance | undefined;
|
|
33
|
+
readonly id: string;
|
|
34
|
+
readonly invariant: string;
|
|
35
|
+
readonly lifecycle: WardenRuleLifecycle;
|
|
36
|
+
readonly scope: WardenRuleScope;
|
|
37
|
+
readonly severity: WardenSeverity;
|
|
38
|
+
readonly tier: WardenRuleTier;
|
|
39
|
+
}
|
|
40
|
+
|
|
41
|
+
export interface WardenGuideManifest {
|
|
42
|
+
readonly generatedFrom: {
|
|
43
|
+
readonly package: '@ontrails/warden';
|
|
44
|
+
readonly registries: readonly ['wardenRules', 'wardenTopoRules'];
|
|
45
|
+
readonly source: 'builtin-rule-metadata';
|
|
46
|
+
};
|
|
47
|
+
readonly kind: 'trails-warden-guide-manifest';
|
|
48
|
+
readonly ruleCount: number;
|
|
49
|
+
readonly rules: readonly WardenRuleGuideEntry[];
|
|
50
|
+
readonly version: 1;
|
|
51
|
+
}
|
|
52
|
+
|
|
53
|
+
interface WardenAgentRuleGuide {
|
|
54
|
+
readonly appliesAt: {
|
|
55
|
+
readonly depth: WardenDepth;
|
|
56
|
+
readonly scope: WardenRuleScope;
|
|
57
|
+
readonly tier: WardenRuleTier;
|
|
58
|
+
};
|
|
59
|
+
readonly concern: WardenRuleConcern;
|
|
60
|
+
readonly fix?: WardenFixCapability | undefined;
|
|
61
|
+
readonly guidance?: WardenGuidance | undefined;
|
|
62
|
+
readonly id: string;
|
|
63
|
+
readonly invariant: string;
|
|
64
|
+
readonly severity: WardenSeverity;
|
|
65
|
+
}
|
|
66
|
+
|
|
67
|
+
interface WardenAgentGuide {
|
|
68
|
+
readonly instructions: readonly string[];
|
|
69
|
+
readonly kind: 'trails-warden-agent-guide';
|
|
70
|
+
readonly rules: readonly WardenAgentRuleGuide[];
|
|
71
|
+
readonly version: 1;
|
|
72
|
+
}
|
|
73
|
+
|
|
74
|
+
const lookupRule = (id: string) =>
|
|
75
|
+
wardenRules.get(id) ?? wardenTopoRules.get(id);
|
|
76
|
+
|
|
77
|
+
const compareRuleEntries = (
|
|
78
|
+
[left]: readonly [string, unknown],
|
|
79
|
+
[right]: readonly [string, unknown]
|
|
80
|
+
): number => left.localeCompare(right);
|
|
81
|
+
|
|
82
|
+
export const buildWardenGuideManifest = (): WardenGuideManifest => {
|
|
83
|
+
const rules = listWardenRuleMetadata()
|
|
84
|
+
.toSorted(compareRuleEntries)
|
|
85
|
+
.map(([id, metadata]) => {
|
|
86
|
+
const rule = lookupRule(id);
|
|
87
|
+
const docs = metadata.guidance?.docs ?? [];
|
|
88
|
+
return {
|
|
89
|
+
concern: metadata.concern,
|
|
90
|
+
depth: metadata.depth,
|
|
91
|
+
description: rule?.description ?? '',
|
|
92
|
+
docs,
|
|
93
|
+
fix: metadata.fix,
|
|
94
|
+
guidance: metadata.guidance,
|
|
95
|
+
id,
|
|
96
|
+
invariant: metadata.invariant,
|
|
97
|
+
lifecycle: metadata.lifecycle,
|
|
98
|
+
scope: metadata.scope,
|
|
99
|
+
severity: rule?.severity ?? 'warn',
|
|
100
|
+
tier: metadata.tier,
|
|
101
|
+
} satisfies WardenRuleGuideEntry;
|
|
102
|
+
});
|
|
103
|
+
|
|
104
|
+
return {
|
|
105
|
+
generatedFrom: {
|
|
106
|
+
package: '@ontrails/warden',
|
|
107
|
+
registries: ['wardenRules', 'wardenTopoRules'],
|
|
108
|
+
source: 'builtin-rule-metadata',
|
|
109
|
+
},
|
|
110
|
+
kind: 'trails-warden-guide-manifest',
|
|
111
|
+
ruleCount: rules.length,
|
|
112
|
+
rules,
|
|
113
|
+
version: 1,
|
|
114
|
+
};
|
|
115
|
+
};
|
|
116
|
+
|
|
117
|
+
const formatGuideLink = (link: WardenGuidanceLink): string => {
|
|
118
|
+
if (link.path) {
|
|
119
|
+
return `[${link.label}](${link.path})`;
|
|
120
|
+
}
|
|
121
|
+
if (link.url) {
|
|
122
|
+
return `[${link.label}](${link.url})`;
|
|
123
|
+
}
|
|
124
|
+
return link.label;
|
|
125
|
+
};
|
|
126
|
+
|
|
127
|
+
const renderOptionalList = (
|
|
128
|
+
lines: string[],
|
|
129
|
+
label: string,
|
|
130
|
+
items: readonly string[] | undefined
|
|
131
|
+
): void => {
|
|
132
|
+
if (items === undefined || items.length === 0) {
|
|
133
|
+
return;
|
|
134
|
+
}
|
|
135
|
+
|
|
136
|
+
lines.push(`- ${label}:`);
|
|
137
|
+
for (const [index, item] of items.entries()) {
|
|
138
|
+
lines.push(` ${index + 1}. ${item}`);
|
|
139
|
+
}
|
|
140
|
+
};
|
|
141
|
+
|
|
142
|
+
const renderRuleMarkdown = (rule: WardenRuleGuideEntry): readonly string[] => {
|
|
143
|
+
const lines = [
|
|
144
|
+
`### \`${rule.id}\``,
|
|
145
|
+
'',
|
|
146
|
+
`- Severity: \`${rule.severity}\``,
|
|
147
|
+
`- Concern: \`${rule.concern}\``,
|
|
148
|
+
`- Depth: \`${rule.depth}\``,
|
|
149
|
+
`- Tier: \`${rule.tier}\``,
|
|
150
|
+
`- Scope: \`${rule.scope}\``,
|
|
151
|
+
`- Lifecycle: \`${rule.lifecycle.state}\``,
|
|
152
|
+
`- Invariant: ${rule.invariant}`,
|
|
153
|
+
`- Description: ${rule.description}`,
|
|
154
|
+
];
|
|
155
|
+
|
|
156
|
+
if (rule.lifecycle.retireWhen) {
|
|
157
|
+
lines.push(`- Retire when: ${rule.lifecycle.retireWhen}`);
|
|
158
|
+
}
|
|
159
|
+
|
|
160
|
+
if (rule.fix) {
|
|
161
|
+
lines.push(
|
|
162
|
+
`- Fix: \`${rule.fix.class}\` (${rule.fix.safety === 'safe' ? 'safe, applied by `warden --fix`' : 'review-required'})`
|
|
163
|
+
);
|
|
164
|
+
}
|
|
165
|
+
|
|
166
|
+
if (rule.guidance) {
|
|
167
|
+
lines.push('', `Guidance: ${rule.guidance.summary}`);
|
|
168
|
+
renderOptionalList(lines, 'Steps', rule.guidance.steps);
|
|
169
|
+
renderOptionalList(lines, 'Commands', rule.guidance.commands);
|
|
170
|
+
if (rule.docs.length > 0) {
|
|
171
|
+
lines.push(`- Docs: ${rule.docs.map(formatGuideLink).join(', ')}`);
|
|
172
|
+
}
|
|
173
|
+
if (rule.guidance.relatedRules && rule.guidance.relatedRules.length > 0) {
|
|
174
|
+
lines.push(
|
|
175
|
+
`- Related rules: ${rule.guidance.relatedRules.map((id) => `\`${id}\``).join(', ')}`
|
|
176
|
+
);
|
|
177
|
+
}
|
|
178
|
+
}
|
|
179
|
+
|
|
180
|
+
return lines;
|
|
181
|
+
};
|
|
182
|
+
|
|
183
|
+
export const formatWardenGuideMarkdown = (
|
|
184
|
+
manifest: WardenGuideManifest
|
|
185
|
+
): string => {
|
|
186
|
+
const lines = [
|
|
187
|
+
'# Trails Warden Guide',
|
|
188
|
+
'',
|
|
189
|
+
'Generated from `@ontrails/warden` built-in rule metadata and live rule registries.',
|
|
190
|
+
'',
|
|
191
|
+
'## Summary',
|
|
192
|
+
'',
|
|
193
|
+
`- Rules: ${manifest.ruleCount}`,
|
|
194
|
+
`- Guided rules: ${manifest.rules.filter((rule) => rule.guidance).length}`,
|
|
195
|
+
'',
|
|
196
|
+
'## Rules',
|
|
197
|
+
'',
|
|
198
|
+
];
|
|
199
|
+
|
|
200
|
+
for (const rule of manifest.rules) {
|
|
201
|
+
lines.push(...renderRuleMarkdown(rule), '');
|
|
202
|
+
}
|
|
203
|
+
|
|
204
|
+
return lines.join('\n').trimEnd();
|
|
205
|
+
};
|
|
206
|
+
|
|
207
|
+
export const buildWardenAgentGuide = (
|
|
208
|
+
manifest: WardenGuideManifest
|
|
209
|
+
): WardenAgentGuide => ({
|
|
210
|
+
instructions: [
|
|
211
|
+
'Treat Warden rules as enforceable Trails doctrine when working in this repository.',
|
|
212
|
+
'Prefer the rule guidance summary and ordered steps over diagnostic prose when deciding how to remediate a finding.',
|
|
213
|
+
'When guidance is absent, use the invariant, concern, tier, and scope as classification metadata rather than inventing a rule-specific fix.',
|
|
214
|
+
],
|
|
215
|
+
kind: 'trails-warden-agent-guide',
|
|
216
|
+
rules: manifest.rules.map((rule) => ({
|
|
217
|
+
appliesAt: {
|
|
218
|
+
depth: rule.depth,
|
|
219
|
+
scope: rule.scope,
|
|
220
|
+
tier: rule.tier,
|
|
221
|
+
},
|
|
222
|
+
concern: rule.concern,
|
|
223
|
+
fix: rule.fix,
|
|
224
|
+
guidance: rule.guidance,
|
|
225
|
+
id: rule.id,
|
|
226
|
+
invariant: rule.invariant,
|
|
227
|
+
severity: rule.severity,
|
|
228
|
+
})),
|
|
229
|
+
version: 1,
|
|
230
|
+
});
|
|
231
|
+
|
|
232
|
+
export const formatWardenGuide = (
|
|
233
|
+
manifest: WardenGuideManifest,
|
|
234
|
+
format: WardenGuideFormat
|
|
235
|
+
): string => {
|
|
236
|
+
if (format === 'markdown') {
|
|
237
|
+
return formatWardenGuideMarkdown(manifest);
|
|
238
|
+
}
|
|
239
|
+
|
|
240
|
+
if (format === 'agent-json') {
|
|
241
|
+
return JSON.stringify(buildWardenAgentGuide(manifest), null, 2);
|
|
242
|
+
}
|
|
243
|
+
|
|
244
|
+
return JSON.stringify(manifest, null, 2);
|
|
245
|
+
};
|
package/src/index.ts
CHANGED
|
@@ -9,32 +9,105 @@
|
|
|
9
9
|
|
|
10
10
|
// Rule types
|
|
11
11
|
export type {
|
|
12
|
+
BuiltinWardenRuleName,
|
|
12
13
|
ProjectAwareWardenRule,
|
|
13
14
|
ProjectContext,
|
|
15
|
+
TopoAwareWardenRule,
|
|
14
16
|
WardenDiagnostic,
|
|
17
|
+
WardenFix,
|
|
18
|
+
WardenFixCapability,
|
|
19
|
+
WardenFixClass,
|
|
20
|
+
WardenFixEdit,
|
|
21
|
+
WardenFixSafety,
|
|
22
|
+
WardenGuidance,
|
|
23
|
+
WardenGuidanceLink,
|
|
15
24
|
WardenRule,
|
|
25
|
+
WardenRuleConcern,
|
|
26
|
+
WardenRuleLifecycle,
|
|
27
|
+
WardenRuleLifecycleState,
|
|
28
|
+
WardenRuleMetadata,
|
|
29
|
+
WardenRuleScope,
|
|
30
|
+
WardenRuleTier,
|
|
16
31
|
WardenSeverity,
|
|
17
32
|
} from './rules/index.js';
|
|
18
33
|
|
|
19
|
-
// Individual rules
|
|
20
|
-
export { noThrowInImplementation } from './rules/no-throw-in-implementation.js';
|
|
21
|
-
export { contextNoSurfaceTypes } from './rules/context-no-surface-types.js';
|
|
22
|
-
export { validDetourRefs } from './rules/valid-detour-refs.js';
|
|
23
|
-
export { noDirectImplInRoute } from './rules/no-direct-impl-in-route.js';
|
|
24
|
-
export { noDirectImplementationCall } from './rules/no-direct-implementation-call.js';
|
|
25
|
-
export { noSyncResultAssumption } from './rules/no-sync-result-assumption.js';
|
|
26
|
-
export { implementationReturnsResult } from './rules/implementation-returns-result.js';
|
|
27
|
-
export { noThrowInDetourTarget } from './rules/no-throw-in-detour-target.js';
|
|
28
|
-
export { preferSchemaInference } from './rules/prefer-schema-inference.js';
|
|
29
|
-
export { validDescribeRefs } from './rules/valid-describe-refs.js';
|
|
30
|
-
|
|
31
34
|
// Rule registry
|
|
32
|
-
export {
|
|
35
|
+
export {
|
|
36
|
+
builtinWardenRuleMetadata,
|
|
37
|
+
getWardenRuleMetadata,
|
|
38
|
+
listWardenRuleMetadata,
|
|
39
|
+
wardenFixClasses,
|
|
40
|
+
wardenFixSafeties,
|
|
41
|
+
wardenRuleConcerns,
|
|
42
|
+
wardenRuleLifecycleStates,
|
|
43
|
+
wardenRuleScopes,
|
|
44
|
+
wardenRuleTiers,
|
|
45
|
+
wardenRules,
|
|
46
|
+
wardenTopoRules,
|
|
47
|
+
} from './rules/index.js';
|
|
48
|
+
|
|
49
|
+
// Rule-scoped cache controls for long-lived consumers (watch mode, LSPs).
|
|
50
|
+
export { clearImplementationReturnsResultCache } from './rules/implementation-returns-result.js';
|
|
51
|
+
export {
|
|
52
|
+
isWardenDevPermitTestScanTarget,
|
|
53
|
+
isWardenInfrastructureScanTarget,
|
|
54
|
+
isWardenSourceScanTarget,
|
|
55
|
+
isWardenTestScanTarget,
|
|
56
|
+
} from './rules/scan.js';
|
|
33
57
|
|
|
34
58
|
// CLI runner
|
|
35
|
-
export type {
|
|
59
|
+
export type {
|
|
60
|
+
WardenOptions,
|
|
61
|
+
WardenReport,
|
|
62
|
+
WardenRunOptions,
|
|
63
|
+
WardenTopoTarget,
|
|
64
|
+
} from './cli.js';
|
|
36
65
|
export { formatWardenReport, runWarden } from './cli.js';
|
|
37
66
|
|
|
67
|
+
// Adapter authoring checks
|
|
68
|
+
export {
|
|
69
|
+
adapterCheckRuleName,
|
|
70
|
+
runWardenAdapterChecks,
|
|
71
|
+
} from './adapter-check.js';
|
|
72
|
+
|
|
73
|
+
// CLI command surface
|
|
74
|
+
export type {
|
|
75
|
+
ParsedWardenCommand,
|
|
76
|
+
RunWardenCommandOptions,
|
|
77
|
+
WardenCommandResult,
|
|
78
|
+
} from './command.js';
|
|
79
|
+
export {
|
|
80
|
+
formatWardenCommandOutput,
|
|
81
|
+
loadWardenConfig,
|
|
82
|
+
parseWardenCommandArgs,
|
|
83
|
+
resolveWardenTopoTargets,
|
|
84
|
+
runWardenCommand,
|
|
85
|
+
} from './command.js';
|
|
86
|
+
|
|
87
|
+
// Config schema
|
|
88
|
+
export {
|
|
89
|
+
wardenConfigSchema,
|
|
90
|
+
wardenDepthValues,
|
|
91
|
+
wardenDraftsValues,
|
|
92
|
+
wardenFailOnValues,
|
|
93
|
+
wardenFormatValues,
|
|
94
|
+
wardenLockValues,
|
|
95
|
+
} from './config.js';
|
|
96
|
+
export type {
|
|
97
|
+
WardenConfig,
|
|
98
|
+
WardenConfigInput,
|
|
99
|
+
WardenDepth,
|
|
100
|
+
WardenDraftsMode,
|
|
101
|
+
WardenFailOn,
|
|
102
|
+
WardenFormat,
|
|
103
|
+
WardenLockMode,
|
|
104
|
+
EffectiveWardenConfig,
|
|
105
|
+
ResolveWardenConfigOptions,
|
|
106
|
+
WardenConfigLayer,
|
|
107
|
+
WardenConfigResolution,
|
|
108
|
+
} from './config.js';
|
|
109
|
+
export { resolveWardenConfig } from './config.js';
|
|
110
|
+
|
|
38
111
|
// CI formatters
|
|
39
112
|
export {
|
|
40
113
|
formatGitHubAnnotations,
|
|
@@ -45,3 +118,122 @@ export {
|
|
|
45
118
|
// Drift detection
|
|
46
119
|
export type { DriftResult } from './drift.js';
|
|
47
120
|
export { checkDrift } from './drift.js';
|
|
121
|
+
|
|
122
|
+
// Guide projection
|
|
123
|
+
export type {
|
|
124
|
+
WardenGuideFormat,
|
|
125
|
+
WardenGuideManifest,
|
|
126
|
+
WardenRuleGuideEntry,
|
|
127
|
+
} from './guide.js';
|
|
128
|
+
export {
|
|
129
|
+
buildWardenAgentGuide,
|
|
130
|
+
buildWardenGuideManifest,
|
|
131
|
+
formatWardenGuide,
|
|
132
|
+
formatWardenGuideMarkdown,
|
|
133
|
+
wardenGuideFormatValues,
|
|
134
|
+
} from './guide.js';
|
|
135
|
+
|
|
136
|
+
// Resolver helpers
|
|
137
|
+
export {
|
|
138
|
+
collectImportResolutionsForFile,
|
|
139
|
+
collectImportSpecifiers,
|
|
140
|
+
createWardenResolver,
|
|
141
|
+
defaultWardenResolveOptions,
|
|
142
|
+
} from './resolve.js';
|
|
143
|
+
export type {
|
|
144
|
+
WardenImportResolution,
|
|
145
|
+
WardenImportResolutionErrorKind,
|
|
146
|
+
WardenImportSpecifier,
|
|
147
|
+
WardenProjectResolver,
|
|
148
|
+
WardenResolverOptions,
|
|
149
|
+
} from './resolve.js';
|
|
150
|
+
|
|
151
|
+
// Draft helpers
|
|
152
|
+
export {
|
|
153
|
+
DRAFT_FILE_PREFIX,
|
|
154
|
+
DRAFT_FILE_SEGMENT,
|
|
155
|
+
isDraftMarkedFile,
|
|
156
|
+
stripDraftFileMarkers,
|
|
157
|
+
} from './draft.js';
|
|
158
|
+
|
|
159
|
+
// Trail layer
|
|
160
|
+
export { wardenTopo } from './trails/topo.js';
|
|
161
|
+
export { runTopoAwareWardenTrails, runWardenTrails } from './trails/run.js';
|
|
162
|
+
export {
|
|
163
|
+
activationOrphanTrail,
|
|
164
|
+
circularRefsTrail,
|
|
165
|
+
contourExistsTrail,
|
|
166
|
+
contextNoSurfaceTypesTrail,
|
|
167
|
+
composesDeclarationsTrail,
|
|
168
|
+
deadInternalTrailTrail,
|
|
169
|
+
deprecationWithoutGuidanceTrail,
|
|
170
|
+
diagnosticSchema,
|
|
171
|
+
draftFileMarkingTrail,
|
|
172
|
+
draftVisibleDebtTrail,
|
|
173
|
+
errorMappingCompletenessTrail,
|
|
174
|
+
exampleValidTrail,
|
|
175
|
+
firesDeclarationsTrail,
|
|
176
|
+
forkWithoutPreservedBlazeTrail,
|
|
177
|
+
implementationReturnsResultTrail,
|
|
178
|
+
incompleteAccessorForStandardOpTrail,
|
|
179
|
+
incompleteCrudTrail,
|
|
180
|
+
intentPropagationTrail,
|
|
181
|
+
layerFieldNameDriftTrail,
|
|
182
|
+
markerSchemaUnsupportedTrail,
|
|
183
|
+
missingVisibilityTrail,
|
|
184
|
+
missingReconcileTrail,
|
|
185
|
+
noDevPermitInSourceTrail,
|
|
186
|
+
noDestructuredComposeTrail,
|
|
187
|
+
noDirectImplementationCallTrail,
|
|
188
|
+
noLegacyLayerImportsTrail,
|
|
189
|
+
noNativeErrorResultTrail,
|
|
190
|
+
noRedundantResultErrorWrapTrail,
|
|
191
|
+
noRetiredCrossVocabularyTrail,
|
|
192
|
+
noSyncResultAssumptionTrail,
|
|
193
|
+
noThrowInDetourRecoverTrail,
|
|
194
|
+
noThrowInImplementationTrail,
|
|
195
|
+
noTopLevelSurfaceTrail,
|
|
196
|
+
onReferencesExistTrail,
|
|
197
|
+
orphanedSignalTrail,
|
|
198
|
+
ownerProjectionParityTrail,
|
|
199
|
+
pendingForceTrail,
|
|
200
|
+
permitGovernanceTrail,
|
|
201
|
+
preferSchemaInferenceTrail,
|
|
202
|
+
projectAwareRuleInput,
|
|
203
|
+
publicExportExampleCoverageTrail,
|
|
204
|
+
publicInternalDeepImportsTrail,
|
|
205
|
+
publicOutputSchemaTrail,
|
|
206
|
+
publicUnionOutputDiscriminantsTrail,
|
|
207
|
+
readIntentFiresTrail,
|
|
208
|
+
referenceExistsTrail,
|
|
209
|
+
resolvedImportBoundaryTrail,
|
|
210
|
+
ruleInput,
|
|
211
|
+
ruleOutput,
|
|
212
|
+
resourceDeclarationsTrail,
|
|
213
|
+
resourceIdGrammarTrail,
|
|
214
|
+
resourceExistsTrail,
|
|
215
|
+
resourceMockCoverageTrail,
|
|
216
|
+
scheduledDestroyIntentTrail,
|
|
217
|
+
signalGraphCoachingTrail,
|
|
218
|
+
staticResourceAccessorPreferenceTrail,
|
|
219
|
+
surfaceFacetCoherenceTrail,
|
|
220
|
+
unmaterializedActivationSourceTrail,
|
|
221
|
+
topoAwareRuleInput,
|
|
222
|
+
unreachableDetourShadowingTrail,
|
|
223
|
+
validDetourContractTrail,
|
|
224
|
+
validDescribeRefsTrail,
|
|
225
|
+
versionGapTrail,
|
|
226
|
+
versionPinnedComposeTrail,
|
|
227
|
+
versionWithoutExamplesTrail,
|
|
228
|
+
wardenExportSymmetryTrail,
|
|
229
|
+
wardenRulesUseAstTrail,
|
|
230
|
+
webhookRouteCollisionTrail,
|
|
231
|
+
wrapRule,
|
|
232
|
+
wrapTopoRule,
|
|
233
|
+
} from './trails/index.js';
|
|
234
|
+
export type {
|
|
235
|
+
ProjectAwareRuleInput,
|
|
236
|
+
RuleInput,
|
|
237
|
+
RuleOutput,
|
|
238
|
+
TopoAwareRuleInput,
|
|
239
|
+
} from './trails/index.js';
|
|
@@ -0,0 +1,163 @@
|
|
|
1
|
+
/**
|
|
2
|
+
* Project-context helpers shared by the Warden runner and resolver-backed
|
|
3
|
+
* rules.
|
|
4
|
+
*/
|
|
5
|
+
|
|
6
|
+
import { realpathSync } from 'node:fs';
|
|
7
|
+
import { resolve } from 'node:path';
|
|
8
|
+
|
|
9
|
+
import {
|
|
10
|
+
collectImportResolutionsForFile,
|
|
11
|
+
createWardenResolver,
|
|
12
|
+
normalizePath,
|
|
13
|
+
} from './resolve.js';
|
|
14
|
+
import type {
|
|
15
|
+
WardenImportResolution,
|
|
16
|
+
WardenResolverOptions,
|
|
17
|
+
} from './resolve.js';
|
|
18
|
+
import { offsetToLine } from './rules/ast.js';
|
|
19
|
+
import { collectPublicWorkspaces } from './workspaces.js';
|
|
20
|
+
import type { WardenPublicWorkspace } from './workspaces.js';
|
|
21
|
+
|
|
22
|
+
const ONTRAILS_DOCUMENTATION_SPECIFIER_PATTERN =
|
|
23
|
+
/@ontrails\/[a-z0-9-]+(?:\/[A-Za-z0-9._~-]+)+/g;
|
|
24
|
+
|
|
25
|
+
const normalizeRealPath = (path: string): string => {
|
|
26
|
+
try {
|
|
27
|
+
return normalizePath(realpathSync(path));
|
|
28
|
+
} catch {
|
|
29
|
+
return normalizePath(resolve(path));
|
|
30
|
+
}
|
|
31
|
+
};
|
|
32
|
+
|
|
33
|
+
const setResolutionsForFile = (
|
|
34
|
+
resolutionsByFile: Map<string, readonly WardenImportResolution[]>,
|
|
35
|
+
sourceFilePath: string,
|
|
36
|
+
resolutions: readonly WardenImportResolution[]
|
|
37
|
+
): void => {
|
|
38
|
+
const normalizedFilePath =
|
|
39
|
+
resolutions[0]?.importerPath ?? normalizeRealPath(sourceFilePath);
|
|
40
|
+
resolutionsByFile.set(normalizedFilePath, resolutions);
|
|
41
|
+
if (normalizedFilePath !== sourceFilePath) {
|
|
42
|
+
resolutionsByFile.set(sourceFilePath, resolutions);
|
|
43
|
+
}
|
|
44
|
+
};
|
|
45
|
+
|
|
46
|
+
export interface WardenProjectContextSourceFile {
|
|
47
|
+
readonly filePath: string;
|
|
48
|
+
readonly kind: 'documentation' | 'text' | 'typescript';
|
|
49
|
+
readonly sourceCode: string;
|
|
50
|
+
}
|
|
51
|
+
|
|
52
|
+
const collectDocumentationImportSpecifiers = (
|
|
53
|
+
sourceCode: string
|
|
54
|
+
): readonly { readonly importSource: string; readonly line: number }[] => {
|
|
55
|
+
const specifiers: { importSource: string; line: number }[] = [];
|
|
56
|
+
for (const match of sourceCode.matchAll(
|
|
57
|
+
ONTRAILS_DOCUMENTATION_SPECIFIER_PATTERN
|
|
58
|
+
)) {
|
|
59
|
+
if (match.index === undefined) {
|
|
60
|
+
continue;
|
|
61
|
+
}
|
|
62
|
+
specifiers.push({
|
|
63
|
+
importSource: match[0],
|
|
64
|
+
line: offsetToLine(sourceCode, match.index),
|
|
65
|
+
});
|
|
66
|
+
}
|
|
67
|
+
return specifiers;
|
|
68
|
+
};
|
|
69
|
+
|
|
70
|
+
const exportAliasesForWorkspaces = (
|
|
71
|
+
workspaces: ReadonlyMap<string, WardenPublicWorkspace>
|
|
72
|
+
): Record<string, string[]> => {
|
|
73
|
+
const aliases: Record<string, string[]> = {};
|
|
74
|
+
for (const workspace of workspaces.values()) {
|
|
75
|
+
for (const [specifier, target] of Object.entries(
|
|
76
|
+
workspace.exportTargets ?? {}
|
|
77
|
+
)) {
|
|
78
|
+
aliases[`${specifier}$`] = [target];
|
|
79
|
+
}
|
|
80
|
+
}
|
|
81
|
+
return aliases;
|
|
82
|
+
};
|
|
83
|
+
|
|
84
|
+
export const collectProjectImportResolutions = ({
|
|
85
|
+
resolveOptions,
|
|
86
|
+
rootDir,
|
|
87
|
+
sourceFiles,
|
|
88
|
+
}: {
|
|
89
|
+
readonly resolveOptions?: WardenResolverOptions['resolveOptions'];
|
|
90
|
+
readonly rootDir: string;
|
|
91
|
+
readonly sourceFiles: readonly WardenProjectContextSourceFile[];
|
|
92
|
+
}): ReadonlyMap<string, readonly WardenImportResolution[]> => {
|
|
93
|
+
const resolver = createWardenResolver({ resolveOptions, rootDir });
|
|
94
|
+
const resolutionsByFile = new Map<
|
|
95
|
+
string,
|
|
96
|
+
readonly WardenImportResolution[]
|
|
97
|
+
>();
|
|
98
|
+
|
|
99
|
+
for (const sourceFile of sourceFiles) {
|
|
100
|
+
if (sourceFile.kind !== 'typescript') {
|
|
101
|
+
continue;
|
|
102
|
+
}
|
|
103
|
+
const resolutions = collectImportResolutionsForFile({
|
|
104
|
+
filePath: sourceFile.filePath,
|
|
105
|
+
resolver,
|
|
106
|
+
sourceCode: sourceFile.sourceCode,
|
|
107
|
+
});
|
|
108
|
+
if (resolutions.length > 0) {
|
|
109
|
+
setResolutionsForFile(
|
|
110
|
+
resolutionsByFile,
|
|
111
|
+
sourceFile.filePath,
|
|
112
|
+
resolutions
|
|
113
|
+
);
|
|
114
|
+
}
|
|
115
|
+
}
|
|
116
|
+
|
|
117
|
+
return resolutionsByFile;
|
|
118
|
+
};
|
|
119
|
+
|
|
120
|
+
export const collectProjectDocumentationImportResolutions = ({
|
|
121
|
+
rootDir,
|
|
122
|
+
sourceFiles,
|
|
123
|
+
}: {
|
|
124
|
+
readonly rootDir: string;
|
|
125
|
+
readonly sourceFiles: readonly WardenProjectContextSourceFile[];
|
|
126
|
+
}): ReadonlyMap<string, readonly WardenImportResolution[]> => {
|
|
127
|
+
const publicWorkspaces = collectPublicWorkspaces(rootDir);
|
|
128
|
+
const resolver = createWardenResolver({
|
|
129
|
+
resolveOptions: { alias: exportAliasesForWorkspaces(publicWorkspaces) },
|
|
130
|
+
rootDir,
|
|
131
|
+
});
|
|
132
|
+
const resolutionsByFile = new Map<
|
|
133
|
+
string,
|
|
134
|
+
readonly WardenImportResolution[]
|
|
135
|
+
>();
|
|
136
|
+
|
|
137
|
+
for (const sourceFile of sourceFiles) {
|
|
138
|
+
if (sourceFile.kind !== 'documentation') {
|
|
139
|
+
continue;
|
|
140
|
+
}
|
|
141
|
+
const resolutions = collectDocumentationImportSpecifiers(
|
|
142
|
+
sourceFile.sourceCode
|
|
143
|
+
).map((specifier) =>
|
|
144
|
+
resolver.resolveImport(
|
|
145
|
+
sourceFile.filePath,
|
|
146
|
+
specifier.importSource,
|
|
147
|
+
specifier.line
|
|
148
|
+
)
|
|
149
|
+
);
|
|
150
|
+
if (resolutions.length > 0) {
|
|
151
|
+
setResolutionsForFile(
|
|
152
|
+
resolutionsByFile,
|
|
153
|
+
sourceFile.filePath,
|
|
154
|
+
resolutions
|
|
155
|
+
);
|
|
156
|
+
}
|
|
157
|
+
}
|
|
158
|
+
|
|
159
|
+
return resolutionsByFile;
|
|
160
|
+
};
|
|
161
|
+
|
|
162
|
+
export { collectPublicWorkspaces };
|
|
163
|
+
export type { WardenPublicWorkspace } from './workspaces.js';
|