@ontrails/warden 1.0.0-beta.14 → 1.0.0-beta.16
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 +149 -0
- package/README.md +77 -34
- package/bin/warden.ts +22 -0
- package/package.json +26 -6
- package/src/ast.ts +28 -0
- package/src/cli.ts +1169 -116
- package/src/command.ts +927 -0
- package/src/config.ts +184 -0
- package/src/drift.ts +76 -34
- package/src/formatters.ts +54 -7
- package/src/guide.ts +234 -0
- package/src/index.ts +151 -38
- 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 +2884 -124
- package/src/rules/circular-refs.ts +154 -0
- package/src/rules/{context-no-trailhead-types.ts → context-no-surface-types.ts} +72 -12
- package/src/rules/contour-exists.ts +251 -0
- package/src/rules/contour-ids.ts +15 -0
- package/src/rules/cross-declarations.ts +277 -69
- package/src/rules/dead-internal-trail.ts +141 -0
- package/src/rules/draft-file-marking.ts +51 -3
- package/src/rules/draft-visible-debt.ts +40 -15
- 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 +1042 -122
- package/src/rules/incomplete-accessor-for-standard-op.ts +272 -0
- package/src/rules/incomplete-crud.ts +580 -0
- package/src/rules/index.ts +156 -19
- package/src/rules/intent-propagation.ts +127 -0
- package/src/rules/layer-field-name-drift.ts +96 -0
- package/src/rules/metadata.ts +508 -0
- package/src/rules/missing-reconcile.ts +98 -0
- package/src/rules/missing-visibility.ts +110 -0
- package/src/rules/no-dev-permit-in-source.ts +99 -0
- package/src/rules/no-direct-implementation-call.ts +1 -1
- package/src/rules/no-legacy-layer-imports.ts +193 -0
- package/src/rules/no-native-error-result.ts +111 -0
- package/src/rules/no-sync-result-assumption.ts +1131 -96
- package/src/rules/no-throw-in-detour-recover.ts +225 -0
- package/src/rules/no-throw-in-implementation.ts +6 -4
- 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-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 +113 -0
- package/src/rules/resolved-import-boundary.ts +146 -0
- package/src/rules/{provision-declarations.ts → resource-declarations.ts} +208 -138
- package/src/rules/{provision-exists.ts → resource-exists.ts} +46 -51
- package/src/rules/resource-id-grammar.ts +65 -0
- package/src/rules/scan.ts +0 -26
- package/src/rules/scheduled-destroy-intent.ts +44 -0
- package/src/rules/signal-graph-coaching.ts +191 -0
- package/src/rules/specs.ts +5 -1
- package/src/rules/static-resource-accessor-preference.ts +657 -0
- package/src/rules/types.ts +187 -7
- 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 +81 -0
- package/src/trails/circular-refs.trail.ts +29 -0
- package/src/trails/{context-no-trailhead-types.trail.ts → context-no-surface-types.trail.ts} +4 -4
- package/src/trails/contour-exists.trail.ts +21 -0
- package/src/trails/dead-internal-trail.trail.ts +26 -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/incomplete-accessor-for-standard-op.trail.ts +76 -0
- package/src/trails/incomplete-crud.trail.ts +39 -0
- package/src/trails/index.ts +56 -8
- package/src/trails/intent-propagation.trail.ts +30 -0
- package/src/trails/layer-field-name-drift.trail.ts +39 -0
- package/src/trails/missing-reconcile.trail.ts +33 -0
- package/src/trails/missing-visibility.trail.ts +22 -0
- package/src/trails/no-dev-permit-in-source.trail.ts +16 -0
- package/src/trails/no-legacy-layer-imports.trail.ts +35 -0
- package/src/trails/no-native-error-result.trail.ts +18 -0
- package/src/trails/no-throw-in-detour-recover.trail.ts +24 -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/permit-governance.trail.ts +51 -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/{provision-declarations.trail.ts → resource-declarations.trail.ts} +6 -6
- package/src/trails/{provision-exists.trail.ts → resource-exists.trail.ts} +7 -7
- package/src/trails/resource-id-grammar.trail.ts +39 -0
- package/src/trails/run.ts +133 -24
- package/src/trails/scheduled-destroy-intent.trail.ts +56 -0
- package/src/trails/schema.ts +122 -4
- package/src/trails/signal-graph-coaching.trail.ts +74 -0
- package/src/trails/static-resource-accessor-preference.trail.ts +25 -0
- package/src/trails/unmaterialized-activation-source.trail.ts +69 -0
- package/src/trails/unreachable-detour-shadowing.trail.ts +45 -0
- package/src/trails/valid-detour-contract.trail.ts +71 -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 +133 -14
- 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 -247
- package/dist/cli.js.map +0 -1
- package/dist/draft.d.ts +0 -5
- package/dist/draft.d.ts.map +0 -1
- package/dist/draft.js +0 -16
- package/dist/draft.js.map +0 -1
- package/dist/drift.d.ts +0 -29
- package/dist/drift.d.ts.map +0 -1
- package/dist/drift.js +0 -61
- package/dist/drift.js.map +0 -1
- package/dist/formatters.d.ts +0 -30
- package/dist/formatters.d.ts.map +0 -1
- package/dist/formatters.js +0 -98
- package/dist/formatters.js.map +0 -1
- package/dist/index.d.ts +0 -37
- package/dist/index.d.ts.map +0 -1
- package/dist/index.js +0 -38
- package/dist/index.js.map +0 -1
- package/dist/rules/ast.d.ts +0 -82
- package/dist/rules/ast.d.ts.map +0 -1
- package/dist/rules/ast.js +0 -363
- 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/context-no-trailhead-types.d.ts +0 -12
- package/dist/rules/context-no-trailhead-types.d.ts.map +0 -1
- package/dist/rules/context-no-trailhead-types.js +0 -96
- package/dist/rules/context-no-trailhead-types.js.map +0 -1
- package/dist/rules/cross-declarations.d.ts +0 -13
- package/dist/rules/cross-declarations.d.ts.map +0 -1
- package/dist/rules/cross-declarations.js +0 -264
- package/dist/rules/cross-declarations.js.map +0 -1
- package/dist/rules/draft-file-marking.d.ts +0 -6
- package/dist/rules/draft-file-marking.d.ts.map +0 -1
- package/dist/rules/draft-file-marking.js +0 -65
- package/dist/rules/draft-file-marking.js.map +0 -1
- package/dist/rules/draft-visible-debt.d.ts +0 -12
- package/dist/rules/draft-visible-debt.d.ts.map +0 -1
- package/dist/rules/draft-visible-debt.js +0 -45
- package/dist/rules/draft-visible-debt.js.map +0 -1
- package/dist/rules/follow-declarations.d.ts +0 -13
- package/dist/rules/follow-declarations.d.ts.map +0 -1
- package/dist/rules/follow-declarations.js +0 -264
- package/dist/rules/follow-declarations.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 -277
- package/dist/rules/implementation-returns-result.js.map +0 -1
- package/dist/rules/index.d.ts +0 -20
- package/dist/rules/index.d.ts.map +0 -1
- package/dist/rules/index.js +0 -49
- 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 -47
- 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/provision-declarations.d.ts +0 -14
- package/dist/rules/provision-declarations.d.ts.map +0 -1
- package/dist/rules/provision-declarations.js +0 -344
- package/dist/rules/provision-declarations.js.map +0 -1
- package/dist/rules/provision-exists.d.ts +0 -6
- package/dist/rules/provision-exists.d.ts.map +0 -1
- package/dist/rules/provision-exists.js +0 -90
- package/dist/rules/provision-exists.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/service-declarations.d.ts +0 -16
- package/dist/rules/service-declarations.d.ts.map +0 -1
- package/dist/rules/service-declarations.js +0 -346
- package/dist/rules/service-declarations.js.map +0 -1
- package/dist/rules/service-exists.d.ts +0 -8
- package/dist/rules/service-exists.d.ts.map +0 -1
- package/dist/rules/service-exists.js +0 -91
- package/dist/rules/service-exists.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 -54
- 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 -117
- package/dist/rules/valid-detour-refs.js.map +0 -1
- package/dist/trails/context-no-surface-types.trail.d.ts +0 -13
- package/dist/trails/context-no-surface-types.trail.d.ts.map +0 -1
- package/dist/trails/context-no-surface-types.trail.js +0 -21
- package/dist/trails/context-no-surface-types.trail.js.map +0 -1
- package/dist/trails/context-no-trailhead-types.trail.d.ts +0 -13
- package/dist/trails/context-no-trailhead-types.trail.d.ts.map +0 -1
- package/dist/trails/context-no-trailhead-types.trail.js +0 -21
- package/dist/trails/context-no-trailhead-types.trail.js.map +0 -1
- package/dist/trails/cross-declarations.trail.d.ts +0 -13
- package/dist/trails/cross-declarations.trail.d.ts.map +0 -1
- package/dist/trails/cross-declarations.trail.js +0 -22
- package/dist/trails/cross-declarations.trail.js.map +0 -1
- package/dist/trails/follow-declarations.trail.d.ts +0 -13
- package/dist/trails/follow-declarations.trail.d.ts.map +0 -1
- package/dist/trails/follow-declarations.trail.js +0 -22
- package/dist/trails/follow-declarations.trail.js.map +0 -1
- package/dist/trails/implementation-returns-result.trail.d.ts +0 -13
- package/dist/trails/implementation-returns-result.trail.d.ts.map +0 -1
- package/dist/trails/implementation-returns-result.trail.js +0 -20
- package/dist/trails/implementation-returns-result.trail.js.map +0 -1
- package/dist/trails/index.d.ts +0 -16
- package/dist/trails/index.d.ts.map +0 -1
- package/dist/trails/index.js +0 -15
- package/dist/trails/index.js.map +0 -1
- package/dist/trails/no-direct-impl-in-route.trail.d.ts +0 -13
- package/dist/trails/no-direct-impl-in-route.trail.d.ts.map +0 -1
- package/dist/trails/no-direct-impl-in-route.trail.js +0 -22
- package/dist/trails/no-direct-impl-in-route.trail.js.map +0 -1
- package/dist/trails/no-direct-implementation-call.trail.d.ts +0 -13
- package/dist/trails/no-direct-implementation-call.trail.d.ts.map +0 -1
- package/dist/trails/no-direct-implementation-call.trail.js +0 -16
- package/dist/trails/no-direct-implementation-call.trail.js.map +0 -1
- package/dist/trails/no-sync-result-assumption.trail.d.ts +0 -13
- package/dist/trails/no-sync-result-assumption.trail.d.ts.map +0 -1
- package/dist/trails/no-sync-result-assumption.trail.js +0 -19
- package/dist/trails/no-sync-result-assumption.trail.js.map +0 -1
- package/dist/trails/no-throw-in-detour-target.trail.d.ts +0 -15
- package/dist/trails/no-throw-in-detour-target.trail.d.ts.map +0 -1
- package/dist/trails/no-throw-in-detour-target.trail.js +0 -20
- package/dist/trails/no-throw-in-detour-target.trail.js.map +0 -1
- package/dist/trails/no-throw-in-implementation.trail.d.ts +0 -13
- package/dist/trails/no-throw-in-implementation.trail.d.ts.map +0 -1
- package/dist/trails/no-throw-in-implementation.trail.js +0 -20
- package/dist/trails/no-throw-in-implementation.trail.js.map +0 -1
- package/dist/trails/prefer-schema-inference.trail.d.ts +0 -13
- package/dist/trails/prefer-schema-inference.trail.d.ts.map +0 -1
- package/dist/trails/prefer-schema-inference.trail.js +0 -21
- package/dist/trails/prefer-schema-inference.trail.js.map +0 -1
- package/dist/trails/provision-declarations.trail.d.ts +0 -13
- package/dist/trails/provision-declarations.trail.d.ts.map +0 -1
- package/dist/trails/provision-declarations.trail.js +0 -25
- package/dist/trails/provision-declarations.trail.js.map +0 -1
- package/dist/trails/provision-exists.trail.d.ts +0 -15
- package/dist/trails/provision-exists.trail.d.ts.map +0 -1
- package/dist/trails/provision-exists.trail.js +0 -27
- package/dist/trails/provision-exists.trail.js.map +0 -1
- package/dist/trails/run.d.ts +0 -17
- package/dist/trails/run.d.ts.map +0 -1
- package/dist/trails/run.js +0 -39
- package/dist/trails/run.js.map +0 -1
- package/dist/trails/schema.d.ts +0 -53
- package/dist/trails/schema.d.ts.map +0 -1
- package/dist/trails/schema.js +0 -42
- package/dist/trails/schema.js.map +0 -1
- package/dist/trails/service-declarations.trail.d.ts +0 -26
- package/dist/trails/service-declarations.trail.d.ts.map +0 -1
- package/dist/trails/service-declarations.trail.js +0 -27
- package/dist/trails/service-declarations.trail.js.map +0 -1
- package/dist/trails/service-exists.trail.d.ts +0 -32
- package/dist/trails/service-exists.trail.d.ts.map +0 -1
- package/dist/trails/service-exists.trail.js +0 -29
- package/dist/trails/service-exists.trail.js.map +0 -1
- package/dist/trails/topo.d.ts +0 -3
- package/dist/trails/topo.d.ts.map +0 -1
- package/dist/trails/topo.js +0 -5
- package/dist/trails/topo.js.map +0 -1
- package/dist/trails/valid-describe-refs.trail.d.ts +0 -15
- package/dist/trails/valid-describe-refs.trail.d.ts.map +0 -1
- package/dist/trails/valid-describe-refs.trail.js +0 -18
- package/dist/trails/valid-describe-refs.trail.js.map +0 -1
- package/dist/trails/valid-detour-refs.trail.d.ts +0 -15
- package/dist/trails/valid-detour-refs.trail.d.ts.map +0 -1
- package/dist/trails/valid-detour-refs.trail.js +0 -24
- package/dist/trails/valid-detour-refs.trail.js.map +0 -1
- package/dist/trails/wrap-rule.d.ts +0 -29
- package/dist/trails/wrap-rule.d.ts.map +0 -1
- package/dist/trails/wrap-rule.js +0 -47
- package/dist/trails/wrap-rule.js.map +0 -1
- package/src/__tests__/cli.test.ts +0 -311
- package/src/__tests__/cross-declarations.test.ts +0 -303
- package/src/__tests__/drift.test.ts +0 -144
- package/src/__tests__/formatters.test.ts +0 -157
- package/src/__tests__/implementation-returns-result.test.ts +0 -129
- 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__/provision-declarations.test.ts +0 -318
- package/src/__tests__/provision-exists.test.ts +0 -122
- package/src/__tests__/rules.test.ts +0 -227
- package/src/__tests__/trails.test.ts +0 -19
- package/src/__tests__/valid-describe-refs.test.ts +0 -60
- package/src/__tests__/wrap-rule.test.ts +0 -41
- package/src/rules/no-direct-impl-in-route.ts +0 -81
- package/src/rules/no-throw-in-detour-target.ts +0 -150
- package/src/rules/valid-detour-refs.ts +0 -189
- package/src/trails/no-direct-impl-in-route.trail.ts +0 -22
- package/src/trails/no-throw-in-detour-target.trail.ts +0 -20
- package/src/trails/valid-detour-refs.trail.ts +0 -24
- package/tsconfig.json +0 -9
- package/tsconfig.tsbuildinfo +0 -1
|
@@ -1,189 +0,0 @@
|
|
|
1
|
-
import { isDraftId } from '@ontrails/core';
|
|
2
|
-
|
|
3
|
-
import { collectTrailIds } from './specs.js';
|
|
4
|
-
import type {
|
|
5
|
-
ProjectAwareWardenRule,
|
|
6
|
-
ProjectContext,
|
|
7
|
-
WardenDiagnostic,
|
|
8
|
-
} from './types.js';
|
|
9
|
-
|
|
10
|
-
interface BraceState {
|
|
11
|
-
depth: number;
|
|
12
|
-
found: boolean;
|
|
13
|
-
}
|
|
14
|
-
|
|
15
|
-
const trackBraces = (line: string, state: BraceState): void => {
|
|
16
|
-
for (const ch of line) {
|
|
17
|
-
if (ch === '{') {
|
|
18
|
-
state.depth += 1;
|
|
19
|
-
state.found = true;
|
|
20
|
-
}
|
|
21
|
-
if (ch === '}') {
|
|
22
|
-
state.depth -= 1;
|
|
23
|
-
}
|
|
24
|
-
}
|
|
25
|
-
};
|
|
26
|
-
|
|
27
|
-
const collectArrayText = (lines: readonly string[], start: number): string => {
|
|
28
|
-
let text = '';
|
|
29
|
-
for (let k = start; k < lines.length && k < start + 20; k += 1) {
|
|
30
|
-
const line = lines[k];
|
|
31
|
-
if (!line) {
|
|
32
|
-
continue;
|
|
33
|
-
}
|
|
34
|
-
text += `${line}\n`;
|
|
35
|
-
if (text.includes(']')) {
|
|
36
|
-
break;
|
|
37
|
-
}
|
|
38
|
-
}
|
|
39
|
-
return text;
|
|
40
|
-
};
|
|
41
|
-
|
|
42
|
-
const findMissingDetourTargets = (
|
|
43
|
-
text: string,
|
|
44
|
-
knownIds: ReadonlySet<string>
|
|
45
|
-
): string[] => {
|
|
46
|
-
const missing: string[] = [];
|
|
47
|
-
for (const m of text.matchAll(/target\s*:\s*["'`]([^"'`]+)["'`]/g)) {
|
|
48
|
-
const [, id] = m;
|
|
49
|
-
if (id && !knownIds.has(id) && !isDraftId(id)) {
|
|
50
|
-
missing.push(id);
|
|
51
|
-
}
|
|
52
|
-
}
|
|
53
|
-
return missing;
|
|
54
|
-
};
|
|
55
|
-
|
|
56
|
-
const findMissingPlainDetours = (
|
|
57
|
-
text: string,
|
|
58
|
-
knownIds: ReadonlySet<string>
|
|
59
|
-
): string[] => {
|
|
60
|
-
const missing: string[] = [];
|
|
61
|
-
const cleaned = text.replaceAll(/target\s*:\s*["'`][^"'`]+["'`]/g, '');
|
|
62
|
-
for (const m of cleaned.matchAll(/["'`]([^"'`]+)["'`]/g)) {
|
|
63
|
-
const [, id] = m;
|
|
64
|
-
if (id && id.includes('.') && !knownIds.has(id) && !isDraftId(id)) {
|
|
65
|
-
missing.push(id);
|
|
66
|
-
}
|
|
67
|
-
}
|
|
68
|
-
return missing;
|
|
69
|
-
};
|
|
70
|
-
|
|
71
|
-
const findAllMissingDetours = (
|
|
72
|
-
text: string,
|
|
73
|
-
knownIds: ReadonlySet<string>
|
|
74
|
-
): string[] => [
|
|
75
|
-
...findMissingDetourTargets(text, knownIds),
|
|
76
|
-
...findMissingPlainDetours(text, knownIds),
|
|
77
|
-
];
|
|
78
|
-
|
|
79
|
-
const addMissingDetourDiagnostics = (
|
|
80
|
-
specLine: string,
|
|
81
|
-
j: number,
|
|
82
|
-
lines: readonly string[],
|
|
83
|
-
trailId: string,
|
|
84
|
-
lineNum: number,
|
|
85
|
-
filePath: string,
|
|
86
|
-
knownIds: ReadonlySet<string>,
|
|
87
|
-
diagnostics: WardenDiagnostic[]
|
|
88
|
-
): void => {
|
|
89
|
-
if (!/\bdetours\s*:/.test(specLine)) {
|
|
90
|
-
return;
|
|
91
|
-
}
|
|
92
|
-
for (const targetId of findAllMissingDetours(
|
|
93
|
-
collectArrayText(lines, j),
|
|
94
|
-
knownIds
|
|
95
|
-
)) {
|
|
96
|
-
diagnostics.push({
|
|
97
|
-
filePath,
|
|
98
|
-
line: lineNum,
|
|
99
|
-
message: `Trail "${trailId}" has detour targeting "${targetId}" which is not defined.`,
|
|
100
|
-
rule: 'valid-detour-refs',
|
|
101
|
-
severity: 'error',
|
|
102
|
-
});
|
|
103
|
-
}
|
|
104
|
-
};
|
|
105
|
-
|
|
106
|
-
const scanTrailDetours = (
|
|
107
|
-
lines: readonly string[],
|
|
108
|
-
startIndex: number,
|
|
109
|
-
trailId: string,
|
|
110
|
-
filePath: string,
|
|
111
|
-
knownIds: ReadonlySet<string>,
|
|
112
|
-
diagnostics: WardenDiagnostic[]
|
|
113
|
-
): void => {
|
|
114
|
-
const braceState: BraceState = { depth: 0, found: false };
|
|
115
|
-
for (let j = startIndex; j < lines.length && j < startIndex + 200; j += 1) {
|
|
116
|
-
const specLine = lines[j];
|
|
117
|
-
if (!specLine) {
|
|
118
|
-
continue;
|
|
119
|
-
}
|
|
120
|
-
trackBraces(specLine, braceState);
|
|
121
|
-
addMissingDetourDiagnostics(
|
|
122
|
-
specLine,
|
|
123
|
-
j,
|
|
124
|
-
lines,
|
|
125
|
-
trailId,
|
|
126
|
-
startIndex + 1,
|
|
127
|
-
filePath,
|
|
128
|
-
knownIds,
|
|
129
|
-
diagnostics
|
|
130
|
-
);
|
|
131
|
-
if (braceState.found && braceState.depth <= 0) {
|
|
132
|
-
break;
|
|
133
|
-
}
|
|
134
|
-
}
|
|
135
|
-
};
|
|
136
|
-
|
|
137
|
-
const processLine = (
|
|
138
|
-
line: string,
|
|
139
|
-
i: number,
|
|
140
|
-
lines: readonly string[],
|
|
141
|
-
filePath: string,
|
|
142
|
-
knownIds: ReadonlySet<string>,
|
|
143
|
-
diagnostics: WardenDiagnostic[]
|
|
144
|
-
): void => {
|
|
145
|
-
const trailMatch = line.match(/\btrail\s*\(\s*["'`]([^"'`]+)["'`]/);
|
|
146
|
-
if (!trailMatch) {
|
|
147
|
-
return;
|
|
148
|
-
}
|
|
149
|
-
const [, trailId] = trailMatch;
|
|
150
|
-
if (!trailId) {
|
|
151
|
-
return;
|
|
152
|
-
}
|
|
153
|
-
scanTrailDetours(lines, i, trailId, filePath, knownIds, diagnostics);
|
|
154
|
-
};
|
|
155
|
-
|
|
156
|
-
const checkDetourRefs = (
|
|
157
|
-
sourceCode: string,
|
|
158
|
-
filePath: string,
|
|
159
|
-
knownIds: ReadonlySet<string>
|
|
160
|
-
): readonly WardenDiagnostic[] => {
|
|
161
|
-
const diagnostics: WardenDiagnostic[] = [];
|
|
162
|
-
const lines = sourceCode.split('\n');
|
|
163
|
-
for (let i = 0; i < lines.length; i += 1) {
|
|
164
|
-
const line = lines[i];
|
|
165
|
-
if (line) {
|
|
166
|
-
processLine(line, i, lines, filePath, knownIds, diagnostics);
|
|
167
|
-
}
|
|
168
|
-
}
|
|
169
|
-
return diagnostics;
|
|
170
|
-
};
|
|
171
|
-
|
|
172
|
-
/**
|
|
173
|
-
* Checks that all trail IDs referenced in `detours` declarations exist.
|
|
174
|
-
*/
|
|
175
|
-
export const validDetourRefs: ProjectAwareWardenRule = {
|
|
176
|
-
check(sourceCode: string, filePath: string): readonly WardenDiagnostic[] {
|
|
177
|
-
return checkDetourRefs(sourceCode, filePath, collectTrailIds(sourceCode));
|
|
178
|
-
},
|
|
179
|
-
checkWithContext(
|
|
180
|
-
sourceCode: string,
|
|
181
|
-
filePath: string,
|
|
182
|
-
context: ProjectContext
|
|
183
|
-
): readonly WardenDiagnostic[] {
|
|
184
|
-
return checkDetourRefs(sourceCode, filePath, context.knownTrailIds);
|
|
185
|
-
},
|
|
186
|
-
description: 'Ensure all detour target trail IDs reference defined trails.',
|
|
187
|
-
name: 'valid-detour-refs',
|
|
188
|
-
severity: 'error',
|
|
189
|
-
};
|
|
@@ -1,22 +0,0 @@
|
|
|
1
|
-
import { noDirectImplInRoute } from '../rules/no-direct-impl-in-route.js';
|
|
2
|
-
import { wrapRule } from './wrap-rule.js';
|
|
3
|
-
|
|
4
|
-
export const noDirectImplInRouteTrail = wrapRule({
|
|
5
|
-
examples: [
|
|
6
|
-
{
|
|
7
|
-
expected: { diagnostics: [] },
|
|
8
|
-
input: {
|
|
9
|
-
filePath: 'clean.ts',
|
|
10
|
-
sourceCode: `trail("entity.onboard", {
|
|
11
|
-
crosses: ["entity.create"],
|
|
12
|
-
blaze: async (input, ctx) => {
|
|
13
|
-
const result = await ctx.cross("entity.create", input);
|
|
14
|
-
return Result.ok(result);
|
|
15
|
-
}
|
|
16
|
-
})`,
|
|
17
|
-
},
|
|
18
|
-
name: 'Trail with crossings using ctx.cross()',
|
|
19
|
-
},
|
|
20
|
-
],
|
|
21
|
-
rule: noDirectImplInRoute,
|
|
22
|
-
});
|
|
@@ -1,20 +0,0 @@
|
|
|
1
|
-
import { noThrowInDetourTarget } from '../rules/no-throw-in-detour-target.js';
|
|
2
|
-
import { wrapRule } from './wrap-rule.js';
|
|
3
|
-
|
|
4
|
-
export const noThrowInDetourTargetTrail = wrapRule({
|
|
5
|
-
examples: [
|
|
6
|
-
{
|
|
7
|
-
expected: { diagnostics: [] },
|
|
8
|
-
input: {
|
|
9
|
-
filePath: 'clean.ts',
|
|
10
|
-
sourceCode: `trail("entity.fallback", {
|
|
11
|
-
blaze: async (input, ctx) => {
|
|
12
|
-
return Result.ok({ recovered: true });
|
|
13
|
-
}
|
|
14
|
-
})`,
|
|
15
|
-
},
|
|
16
|
-
name: 'Detour target without throw',
|
|
17
|
-
},
|
|
18
|
-
],
|
|
19
|
-
rule: noThrowInDetourTarget,
|
|
20
|
-
});
|
|
@@ -1,24 +0,0 @@
|
|
|
1
|
-
import { validDetourRefs } from '../rules/valid-detour-refs.js';
|
|
2
|
-
import { wrapRule } from './wrap-rule.js';
|
|
3
|
-
|
|
4
|
-
export const validDetourRefsTrail = wrapRule({
|
|
5
|
-
examples: [
|
|
6
|
-
{
|
|
7
|
-
expected: { diagnostics: [] },
|
|
8
|
-
input: {
|
|
9
|
-
filePath: 'clean.ts',
|
|
10
|
-
knownTrailIds: ['entity.fallback', 'entity.show'],
|
|
11
|
-
sourceCode: `trail("entity.fallback", {
|
|
12
|
-
blaze: async (input, ctx) => Result.ok(data)
|
|
13
|
-
})
|
|
14
|
-
|
|
15
|
-
trail("entity.show", {
|
|
16
|
-
detours: [{ target: "entity.fallback" }],
|
|
17
|
-
blaze: async (input, ctx) => Result.ok(data)
|
|
18
|
-
})`,
|
|
19
|
-
},
|
|
20
|
-
name: 'Valid detour target reference',
|
|
21
|
-
},
|
|
22
|
-
],
|
|
23
|
-
rule: validDetourRefs,
|
|
24
|
-
});
|
package/tsconfig.json
DELETED
package/tsconfig.tsbuildinfo
DELETED
|
@@ -1 +0,0 @@
|
|
|
1
|
-
{"root":["./src/cli.ts","./src/draft.ts","./src/drift.ts","./src/formatters.ts","./src/index.ts","./src/rules/ast.ts","./src/rules/context-no-trailhead-types.ts","./src/rules/cross-declarations.ts","./src/rules/draft-file-marking.ts","./src/rules/draft-visible-debt.ts","./src/rules/implementation-returns-result.ts","./src/rules/index.ts","./src/rules/no-direct-impl-in-route.ts","./src/rules/no-direct-implementation-call.ts","./src/rules/no-sync-result-assumption.ts","./src/rules/no-throw-in-detour-target.ts","./src/rules/no-throw-in-implementation.ts","./src/rules/prefer-schema-inference.ts","./src/rules/provision-declarations.ts","./src/rules/provision-exists.ts","./src/rules/scan.ts","./src/rules/specs.ts","./src/rules/structure.ts","./src/rules/types.ts","./src/rules/valid-describe-refs.ts","./src/rules/valid-detour-refs.ts","./src/trails/context-no-trailhead-types.trail.ts","./src/trails/cross-declarations.trail.ts","./src/trails/implementation-returns-result.trail.ts","./src/trails/index.ts","./src/trails/no-direct-impl-in-route.trail.ts","./src/trails/no-direct-implementation-call.trail.ts","./src/trails/no-sync-result-assumption.trail.ts","./src/trails/no-throw-in-detour-target.trail.ts","./src/trails/no-throw-in-implementation.trail.ts","./src/trails/prefer-schema-inference.trail.ts","./src/trails/provision-declarations.trail.ts","./src/trails/provision-exists.trail.ts","./src/trails/run.ts","./src/trails/schema.ts","./src/trails/topo.ts","./src/trails/valid-describe-refs.trail.ts","./src/trails/valid-detour-refs.trail.ts","./src/trails/wrap-rule.ts"],"version":"5.9.3"}
|