refactor(adf): dismantle monolithic types.ts into domain-owned modules#22
Merged
SmartBrandStrategies merged 1 commit intomainfrom Mar 3, 2026
Merged
refactor(adf): dismantle monolithic types.ts into domain-owned modules#22SmartBrandStrategies merged 1 commit intomainfrom
SmartBrandStrategies merged 1 commit intomainfrom
Conversation
Split types.ts (258 LOC) into 6 focused domain files under types/: - ast.ts: AdfDocument, AdfSection, AdfContent, AdfMapEntry, AdfMetricEntry - decorations.ts: STANDARD_DECORATIONS, CANONICAL_KEY_ORDER - patch.ts: PatchOperation union and 7 op interfaces - manifest.ts: Manifest, ManifestModule, SyncEntry, CadenceEntry, MetricSource - bundle.ts: BundleResult - validation.ts: ConstraintStatus, ConstraintResult, WeightSummary, EvidenceResult, AdfLockfile, AdfSyncStatus Original types.ts replaced with barrel re-export for backward compatibility. Zero import changes needed across all consumers. Closes #10 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
stackbilt-admin
pushed a commit
that referenced
this pull request
Apr 9, 2026
Both packages are now powering downstream integrations in cc-taskrunner (Stackbilt-dev/cc-taskrunner#21, #22). Added "Downstream integrations" sections to both READMEs pointing at the taskrunner as a real-world example of wiring these primitives into a governance workflow. blast: - Documents cc-taskrunner 1.5.0's 4-level severity ladder with gate behavior table - Explains the auto_safe downgrade on critical severity - Points at compute_blast_radius() in taskrunner.sh surface: - Documents cc-taskrunner 1.4.0's mission brief fingerprint injection - Mentions the 80-line cap and graceful no-op behavior - Points at build_fingerprint() in taskrunner.sh Both sections help readers understand how these zero-dep analysis packages compose with real autonomous-agent workflows. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
types.ts(258 LOC) into 6 focused domain files undertypes/:ast.ts— AdfDocument, AdfSection, AdfContent, AdfMapEntry, AdfMetricEntrydecorations.ts— STANDARD_DECORATIONS, CANONICAL_KEY_ORDERpatch.ts— PatchOperation union and 7 op interfacesmanifest.ts— Manifest, ManifestModule, SyncEntry, CadenceEntry, MetricSourcebundle.ts— BundleResultvalidation.ts— ConstraintStatus, ConstraintResult, WeightSummary, EvidenceResult, AdfLockfile, AdfSyncStatustypes.tspreserves all existing imports unchangedCloses #10
Test plan
pnpm run buildclean (TypeScript resolves all re-exports)pnpm run test— 251 tests pass (zero changes to any test file)🤖 Generated with Claude Code