@ontrails/warden 1.0.0-beta.4 → 1.0.0-beta.42
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 +837 -13
- package/README.md +133 -26
- package/bin/warden.ts +51 -0
- package/package.json +27 -6
- package/src/adapter-check.ts +136 -0
- package/src/cli.ts +1570 -105
- package/src/command.ts +986 -0
- package/src/config.ts +193 -0
- package/src/draft.ts +22 -0
- package/src/drift.ts +233 -23
- package/src/fix.ts +126 -0
- package/src/formatters.ts +78 -13
- package/src/guide.ts +245 -0
- package/src/index.ts +247 -15
- package/src/project-context.ts +446 -0
- package/src/project-rules.ts +290 -0
- package/src/resolve.ts +531 -0
- package/src/rules/activation-orphan.ts +97 -0
- package/src/rules/captured-kernel.ts +375 -0
- package/src/rules/circular-refs.ts +150 -0
- package/src/rules/cli-command-route-coherence.ts +177 -0
- package/src/rules/composes-declarations.ts +839 -0
- package/src/rules/context-no-surface-types.ts +79 -15
- package/src/rules/dead-internal-trail.ts +161 -0
- package/src/rules/dead-public-trail.ts +258 -0
- package/src/rules/draft-file-marking.ts +155 -0
- package/src/rules/draft-visible-debt.ts +82 -0
- package/src/rules/duplicate-exported-symbol.ts +211 -0
- package/src/rules/duplicate-public-contract.ts +137 -0
- package/src/rules/entity-exists.ts +254 -0
- package/src/rules/entity-ids.ts +15 -0
- package/src/rules/error-mapping-completeness.ts +290 -0
- package/src/rules/example-valid.ts +395 -0
- package/src/rules/fires-declarations.ts +740 -0
- package/src/rules/governed-symbol-residue.ts +438 -0
- package/src/rules/implementation-returns-result.ts +1409 -166
- package/src/rules/incomplete-accessor-for-standard-op.ts +272 -0
- package/src/rules/incomplete-crud.ts +583 -0
- package/src/rules/index.ts +277 -10
- package/src/rules/intent-propagation.ts +125 -0
- package/src/rules/layer-field-name-drift.ts +102 -0
- package/src/rules/library-projection-coherence.ts +100 -0
- package/src/rules/metadata.ts +871 -0
- package/src/rules/missing-reconcile.ts +97 -0
- package/src/rules/missing-visibility.ts +111 -0
- package/src/rules/no-destructured-compose.ts +196 -0
- package/src/rules/no-dev-permit-in-source.ts +99 -0
- package/src/rules/no-direct-implementation-call.ts +12 -7
- package/src/rules/no-legacy-cli-alias-export.ts +247 -0
- package/src/rules/no-legacy-layer-imports.ts +211 -0
- package/src/rules/no-native-error-result.ts +118 -0
- package/src/rules/no-redundant-result-error-wrap.ts +384 -0
- package/src/rules/no-retired-cross-vocabulary.ts +204 -0
- package/src/rules/no-sync-result-assumption.ts +1141 -98
- package/src/rules/no-throw-in-detour-recover.ts +225 -0
- package/src/rules/no-throw-in-implementation.ts +15 -8
- package/src/rules/no-top-level-surface.ts +371 -0
- package/src/rules/on-references-exist.ts +194 -0
- package/src/rules/orphaned-signal.ts +149 -0
- package/src/rules/owner-projection-parity.ts +143 -0
- package/src/rules/permit-governance.ts +25 -0
- package/src/rules/public-export-example-coverage.ts +573 -0
- package/src/rules/public-internal-deep-imports.ts +456 -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 +188 -0
- package/src/rules/reference-exists.ts +97 -0
- package/src/rules/registry-names.ts +167 -0
- package/src/rules/resolved-import-boundary.ts +146 -0
- package/src/rules/resource-declarations.ts +697 -0
- package/src/rules/resource-exists.ts +181 -0
- package/src/rules/resource-id-grammar.ts +65 -0
- package/src/rules/resource-mock-coverage.ts +115 -0
- package/src/rules/retired-vocabulary.ts +633 -0
- package/src/rules/scan.ts +38 -25
- package/src/rules/scheduled-destroy-intent.ts +44 -0
- package/src/rules/signal-graph-coaching.ts +220 -0
- package/src/rules/source/composition.ts +165 -0
- package/src/rules/source/drafts.ts +164 -0
- package/src/rules/source/entities.ts +618 -0
- package/src/rules/source/pragmas.ts +45 -0
- package/src/rules/source/resources.ts +64 -0
- package/src/rules/source/signals.ts +397 -0
- package/src/rules/source/stores.ts +310 -0
- package/src/rules/specs.ts +9 -5
- package/src/rules/static-resource-accessor-preference.ts +653 -0
- package/src/rules/surface-overlay-coherence.ts +262 -0
- package/src/rules/surface-trailhead-coherence.ts +366 -0
- package/src/rules/trail-fork-coaching.ts +625 -0
- package/src/rules/trail-versioning-source.ts +1076 -0
- package/src/rules/trail-versioning-topo.ts +172 -0
- package/src/rules/trailhead-override-divergence.ts +356 -0
- package/src/rules/types.ts +354 -8
- package/src/rules/unmaterialized-activation-source.ts +85 -0
- package/src/rules/unreachable-detour-shadowing.ts +339 -0
- package/src/rules/valid-describe-refs.ts +162 -32
- package/src/rules/valid-detour-contract.ts +78 -0
- package/src/rules/warden-export-symmetry.ts +540 -0
- package/src/rules/warden-rules-use-ast.ts +1109 -0
- package/src/rules/webhook-route-collision.ts +306 -0
- package/src/trails/activation-orphan.trail.ts +84 -0
- package/src/trails/captured-kernel.trail.ts +108 -0
- package/src/trails/circular-refs.trail.ts +29 -0
- package/src/trails/cli-command-route-coherence.trail.ts +47 -0
- package/src/trails/composes-declarations.trail.ts +22 -0
- package/src/trails/context-no-surface-types.trail.ts +21 -0
- package/src/trails/dead-internal-trail.trail.ts +26 -0
- package/src/trails/dead-public-trail.trail.ts +31 -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/duplicate-exported-symbol.trail.ts +48 -0
- package/src/trails/duplicate-public-contract.trail.ts +47 -0
- package/src/trails/entity-exists.trail.ts +21 -0
- package/src/trails/error-mapping-completeness.trail.ts +30 -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-implementation.trail.ts +31 -0
- package/src/trails/governed-symbol-residue.trail.ts +24 -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 +89 -0
- package/src/trails/intent-propagation.trail.ts +30 -0
- package/src/trails/layer-field-name-drift.trail.ts +39 -0
- package/src/trails/library-projection-coherence.trail.ts +43 -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-cli-alias-export.trail.ts +41 -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 +160 -0
- package/src/trails/scheduled-destroy-intent.trail.ts +56 -0
- package/src/trails/schema.ts +237 -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-overlay-coherence.trail.ts +24 -0
- package/src/trails/surface-trailhead-coherence.trail.ts +25 -0
- package/src/trails/topo.ts +6 -0
- package/src/trails/trail-fork-coaching.trail.ts +42 -0
- package/src/trails/trailhead-override-divergence.trail.ts +47 -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 +64 -0
- package/src/trails/webhook-route-collision.trail.ts +50 -0
- package/src/trails/wrap-rule.ts +224 -0
- package/src/workspaces.ts +199 -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 -218
- 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 -161
- 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 -277
- package/dist/rules/implementation-returns-result.js.map +0 -1
- package/dist/rules/index.d.ts +0 -15
- package/dist/rules/index.d.ts.map +0 -1
- package/dist/rules/index.js +0 -34
- 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/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 -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__/rules.test.ts +0 -227
- package/src/__tests__/valid-describe-refs.test.ts +0 -60
- package/src/rules/ast.ts +0 -213
- 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 -187
- package/tsconfig.json +0 -9
- package/tsconfig.tsbuildinfo +0 -1
|
@@ -0,0 +1,290 @@
|
|
|
1
|
+
import { statSync } from 'node:fs';
|
|
2
|
+
import { basename, join } from 'node:path';
|
|
3
|
+
import { pathToFileURL } from 'node:url';
|
|
4
|
+
|
|
5
|
+
import type {
|
|
6
|
+
ProjectAwareWardenRule,
|
|
7
|
+
TopoAwareWardenRule,
|
|
8
|
+
WardenDiagnostic,
|
|
9
|
+
WardenRuleMetadata,
|
|
10
|
+
WardenRule,
|
|
11
|
+
} from './rules/types.js';
|
|
12
|
+
|
|
13
|
+
const PROJECT_WARDEN_RULES_FILE = '.trails/rules.ts';
|
|
14
|
+
const PROJECT_WARDEN_RULES_DIR = '.trails/rules';
|
|
15
|
+
const LEGACY_PROJECT_WARDEN_RULES_DIR = 'trails/warden/rules';
|
|
16
|
+
|
|
17
|
+
export interface ProjectWardenRules {
|
|
18
|
+
readonly diagnostics: readonly WardenDiagnostic[];
|
|
19
|
+
readonly sourceRules: readonly WardenRule[];
|
|
20
|
+
readonly topoRules: readonly TopoAwareWardenRule[];
|
|
21
|
+
}
|
|
22
|
+
|
|
23
|
+
interface RuleModule {
|
|
24
|
+
readonly default?: unknown;
|
|
25
|
+
readonly rule?: unknown;
|
|
26
|
+
readonly rules?: unknown;
|
|
27
|
+
readonly sourceRule?: unknown;
|
|
28
|
+
readonly sourceRules?: unknown;
|
|
29
|
+
readonly topoRule?: unknown;
|
|
30
|
+
readonly topoRules?: unknown;
|
|
31
|
+
}
|
|
32
|
+
|
|
33
|
+
interface ProjectRuleModulePaths {
|
|
34
|
+
readonly diagnostics: readonly WardenDiagnostic[];
|
|
35
|
+
readonly files: readonly string[];
|
|
36
|
+
}
|
|
37
|
+
|
|
38
|
+
interface LoadedRuleFile extends ProjectWardenRules {
|
|
39
|
+
readonly filePath: string;
|
|
40
|
+
}
|
|
41
|
+
|
|
42
|
+
const diagnostic = (message: string, filePath: string): WardenDiagnostic => ({
|
|
43
|
+
filePath,
|
|
44
|
+
line: 1,
|
|
45
|
+
message,
|
|
46
|
+
rule: 'project-warden-rules',
|
|
47
|
+
severity: 'error',
|
|
48
|
+
});
|
|
49
|
+
|
|
50
|
+
const asArray = (value: unknown): readonly unknown[] => {
|
|
51
|
+
if (value === undefined) {
|
|
52
|
+
return [];
|
|
53
|
+
}
|
|
54
|
+
return Array.isArray(value) ? value : [value];
|
|
55
|
+
};
|
|
56
|
+
|
|
57
|
+
const isSourceRule = (value: unknown): value is WardenRule => {
|
|
58
|
+
if (typeof value !== 'object' || value === null) {
|
|
59
|
+
return false;
|
|
60
|
+
}
|
|
61
|
+
const maybe = value as Partial<WardenRule>;
|
|
62
|
+
return (
|
|
63
|
+
typeof maybe.name === 'string' &&
|
|
64
|
+
typeof maybe.description === 'string' &&
|
|
65
|
+
(maybe.severity === 'error' || maybe.severity === 'warn') &&
|
|
66
|
+
typeof maybe.check === 'function'
|
|
67
|
+
);
|
|
68
|
+
};
|
|
69
|
+
|
|
70
|
+
const isTopoRule = (value: unknown): value is TopoAwareWardenRule => {
|
|
71
|
+
if (typeof value !== 'object' || value === null) {
|
|
72
|
+
return false;
|
|
73
|
+
}
|
|
74
|
+
const maybe = value as Partial<TopoAwareWardenRule>;
|
|
75
|
+
return (
|
|
76
|
+
typeof maybe.name === 'string' &&
|
|
77
|
+
typeof maybe.description === 'string' &&
|
|
78
|
+
(maybe.severity === 'error' || maybe.severity === 'warn') &&
|
|
79
|
+
typeof maybe.checkTopo === 'function'
|
|
80
|
+
);
|
|
81
|
+
};
|
|
82
|
+
|
|
83
|
+
const defaultProjectRuleDepth = (
|
|
84
|
+
tier: WardenRuleMetadata['tier']
|
|
85
|
+
): WardenRuleMetadata['depth'] => {
|
|
86
|
+
if (tier === 'topo-aware') {
|
|
87
|
+
return 'topo';
|
|
88
|
+
}
|
|
89
|
+
if (tier === 'project-static') {
|
|
90
|
+
return 'project';
|
|
91
|
+
}
|
|
92
|
+
return 'source';
|
|
93
|
+
};
|
|
94
|
+
|
|
95
|
+
const defaultProjectRuleMetadata = (
|
|
96
|
+
rule: { readonly description: string },
|
|
97
|
+
tier: WardenRuleMetadata['tier']
|
|
98
|
+
): WardenRuleMetadata => ({
|
|
99
|
+
concern: 'general',
|
|
100
|
+
depth: defaultProjectRuleDepth(tier),
|
|
101
|
+
invariant: rule.description,
|
|
102
|
+
lifecycle: { state: 'durable' },
|
|
103
|
+
scope: 'repo-local',
|
|
104
|
+
tier,
|
|
105
|
+
});
|
|
106
|
+
|
|
107
|
+
const isProjectAwareSourceRule = (
|
|
108
|
+
rule: WardenRule
|
|
109
|
+
): rule is ProjectAwareWardenRule =>
|
|
110
|
+
'checkWithContext' in rule && typeof rule.checkWithContext === 'function';
|
|
111
|
+
|
|
112
|
+
const withSourceRuleMetadata = (rule: WardenRule): WardenRule => ({
|
|
113
|
+
...rule,
|
|
114
|
+
metadata:
|
|
115
|
+
rule.metadata ??
|
|
116
|
+
defaultProjectRuleMetadata(
|
|
117
|
+
rule,
|
|
118
|
+
isProjectAwareSourceRule(rule) ? 'project-static' : 'source-static'
|
|
119
|
+
),
|
|
120
|
+
});
|
|
121
|
+
|
|
122
|
+
const withTopoRuleMetadata = (
|
|
123
|
+
rule: TopoAwareWardenRule
|
|
124
|
+
): TopoAwareWardenRule => ({
|
|
125
|
+
...rule,
|
|
126
|
+
metadata: rule.metadata ?? defaultProjectRuleMetadata(rule, 'topo-aware'),
|
|
127
|
+
});
|
|
128
|
+
|
|
129
|
+
const ruleCandidatesFromModule = (module: RuleModule): readonly unknown[] => [
|
|
130
|
+
...asArray(module.default),
|
|
131
|
+
...asArray(module.rule),
|
|
132
|
+
...asArray(module.rules),
|
|
133
|
+
...asArray(module.sourceRule),
|
|
134
|
+
...asArray(module.sourceRules),
|
|
135
|
+
];
|
|
136
|
+
|
|
137
|
+
const topoRuleCandidatesFromModule = (
|
|
138
|
+
module: RuleModule
|
|
139
|
+
): readonly unknown[] => [
|
|
140
|
+
...asArray(module.topoRule),
|
|
141
|
+
...asArray(module.topoRules),
|
|
142
|
+
];
|
|
143
|
+
|
|
144
|
+
const isFile = (path: string): boolean => {
|
|
145
|
+
try {
|
|
146
|
+
return statSync(path).isFile();
|
|
147
|
+
} catch {
|
|
148
|
+
return false;
|
|
149
|
+
}
|
|
150
|
+
};
|
|
151
|
+
|
|
152
|
+
const isDirectory = (path: string): boolean => {
|
|
153
|
+
try {
|
|
154
|
+
return statSync(path).isDirectory();
|
|
155
|
+
} catch {
|
|
156
|
+
return false;
|
|
157
|
+
}
|
|
158
|
+
};
|
|
159
|
+
|
|
160
|
+
const collectProjectRuleFiles = (rootDir: string): readonly string[] => {
|
|
161
|
+
const files: string[] = [];
|
|
162
|
+
const rulesFile = join(rootDir, PROJECT_WARDEN_RULES_FILE);
|
|
163
|
+
const rulesDir = join(rootDir, PROJECT_WARDEN_RULES_DIR);
|
|
164
|
+
|
|
165
|
+
if (isFile(rulesFile)) {
|
|
166
|
+
files.push(rulesFile);
|
|
167
|
+
}
|
|
168
|
+
|
|
169
|
+
if (!isDirectory(rulesDir)) {
|
|
170
|
+
return files;
|
|
171
|
+
}
|
|
172
|
+
|
|
173
|
+
const glob = new Bun.Glob('*.ts');
|
|
174
|
+
files.push(
|
|
175
|
+
...[...glob.scanSync({ cwd: rulesDir, onlyFiles: true })]
|
|
176
|
+
.filter(
|
|
177
|
+
(entry) =>
|
|
178
|
+
!entry.endsWith('.test.ts') && !basename(entry).startsWith('_')
|
|
179
|
+
)
|
|
180
|
+
.map((entry) => join(rulesDir, entry))
|
|
181
|
+
);
|
|
182
|
+
return files.toSorted((left, right) => left.localeCompare(right));
|
|
183
|
+
};
|
|
184
|
+
|
|
185
|
+
const collectProjectRuleModulePaths = (
|
|
186
|
+
rootDir: string
|
|
187
|
+
): ProjectRuleModulePaths => {
|
|
188
|
+
const diagnostics: WardenDiagnostic[] = [];
|
|
189
|
+
const legacyRulesDir = join(rootDir, LEGACY_PROJECT_WARDEN_RULES_DIR);
|
|
190
|
+
|
|
191
|
+
if (isDirectory(legacyRulesDir)) {
|
|
192
|
+
diagnostics.push(
|
|
193
|
+
diagnostic(
|
|
194
|
+
'Project Warden rules moved from trails/warden/rules to .trails/rules.ts or direct .trails/rules/*.ts files.',
|
|
195
|
+
legacyRulesDir
|
|
196
|
+
)
|
|
197
|
+
);
|
|
198
|
+
}
|
|
199
|
+
|
|
200
|
+
return {
|
|
201
|
+
diagnostics,
|
|
202
|
+
files: collectProjectRuleFiles(rootDir),
|
|
203
|
+
};
|
|
204
|
+
};
|
|
205
|
+
|
|
206
|
+
const loadRuleFile = async (filePath: string): Promise<LoadedRuleFile> => {
|
|
207
|
+
let module: RuleModule;
|
|
208
|
+
try {
|
|
209
|
+
module = (await import(pathToFileURL(filePath).href)) as RuleModule;
|
|
210
|
+
} catch (error) {
|
|
211
|
+
return {
|
|
212
|
+
diagnostics: [
|
|
213
|
+
diagnostic(
|
|
214
|
+
error instanceof Error
|
|
215
|
+
? `Failed to load project Warden rule module: ${error.message}`
|
|
216
|
+
: 'Failed to load project Warden rule module.',
|
|
217
|
+
filePath
|
|
218
|
+
),
|
|
219
|
+
],
|
|
220
|
+
filePath,
|
|
221
|
+
sourceRules: [],
|
|
222
|
+
topoRules: [],
|
|
223
|
+
};
|
|
224
|
+
}
|
|
225
|
+
|
|
226
|
+
const sourceRules = ruleCandidatesFromModule(module)
|
|
227
|
+
.filter(isSourceRule)
|
|
228
|
+
.map(withSourceRuleMetadata);
|
|
229
|
+
const topoRules = topoRuleCandidatesFromModule(module)
|
|
230
|
+
.filter(isTopoRule)
|
|
231
|
+
.map(withTopoRuleMetadata);
|
|
232
|
+
if (sourceRules.length === 0 && topoRules.length === 0) {
|
|
233
|
+
return {
|
|
234
|
+
diagnostics: [
|
|
235
|
+
diagnostic(
|
|
236
|
+
'Project Warden rule module must export a WardenRule or TopoAwareWardenRule.',
|
|
237
|
+
filePath
|
|
238
|
+
),
|
|
239
|
+
],
|
|
240
|
+
filePath,
|
|
241
|
+
sourceRules: [],
|
|
242
|
+
topoRules: [],
|
|
243
|
+
};
|
|
244
|
+
}
|
|
245
|
+
|
|
246
|
+
return { diagnostics: [], filePath, sourceRules, topoRules };
|
|
247
|
+
};
|
|
248
|
+
|
|
249
|
+
const duplicateRuleDiagnostics = (
|
|
250
|
+
loaded: readonly LoadedRuleFile[]
|
|
251
|
+
): readonly WardenDiagnostic[] => {
|
|
252
|
+
const seen = new Map<string, string>();
|
|
253
|
+
const diagnostics: WardenDiagnostic[] = [];
|
|
254
|
+
|
|
255
|
+
for (const result of loaded) {
|
|
256
|
+
for (const rule of [...result.sourceRules, ...result.topoRules]) {
|
|
257
|
+
const previousFilePath = seen.get(rule.name);
|
|
258
|
+
if (previousFilePath === undefined) {
|
|
259
|
+
seen.set(rule.name, result.filePath);
|
|
260
|
+
continue;
|
|
261
|
+
}
|
|
262
|
+
diagnostics.push(
|
|
263
|
+
diagnostic(
|
|
264
|
+
`Duplicate project Warden rule id "${rule.name}" also exported from "${previousFilePath}". Project-local rule ids must be unique.`,
|
|
265
|
+
result.filePath
|
|
266
|
+
)
|
|
267
|
+
);
|
|
268
|
+
}
|
|
269
|
+
}
|
|
270
|
+
|
|
271
|
+
return diagnostics;
|
|
272
|
+
};
|
|
273
|
+
|
|
274
|
+
export const loadProjectWardenRules = async (
|
|
275
|
+
rootDir: string
|
|
276
|
+
): Promise<ProjectWardenRules> => {
|
|
277
|
+
const collected = collectProjectRuleModulePaths(rootDir);
|
|
278
|
+
const { files } = collected;
|
|
279
|
+
const loaded = await Promise.all(files.map(loadRuleFile));
|
|
280
|
+
|
|
281
|
+
return {
|
|
282
|
+
diagnostics: [
|
|
283
|
+
...collected.diagnostics,
|
|
284
|
+
...loaded.flatMap((result) => result.diagnostics),
|
|
285
|
+
...duplicateRuleDiagnostics(loaded),
|
|
286
|
+
],
|
|
287
|
+
sourceRules: loaded.flatMap((result) => result.sourceRules),
|
|
288
|
+
topoRules: loaded.flatMap((result) => result.topoRules),
|
|
289
|
+
};
|
|
290
|
+
};
|