Skip to content

refactor: codec pipeline restructuring and progressive disclosure infrastructure #40

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
darko-mijic merged 7 commits into from
Apr 1, 2026
Merged

refactor: codec pipeline restructuring and progressive disclosure infrastructure #40

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
74b09bd
fix: remove dead taxonomy tags, fix category extraction bugs, filter …
darko-mijic Mar 29, 2026
0e03bc4
Prepare refactoring materials
darko-mijic Apr 1, 2026
102d1a6
refactor: codec pipeline restructuring and progressive disclosure inf…
darko-mijic Apr 1, 2026
f8a2dad
fix: address codex review findings — deep merge recursion and stale f…
darko-mijic Apr 1, 2026
e59f443
fix: address second codex review — partial tagExampleOverrides and JS…
darko-mijic Apr 1, 2026
8c3772d
fix: address adversarial review — clean breaks, no backward compat
darko-mijic Apr 1, 2026
b63215f
fix: address PR #40 review — type-safe product areas, import ordering…
darko-mijic Apr 1, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
14 changes: 7 additions & 7 deletions _claude-md/testing/test-implementation.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,13 +2,13 @@

Issues discovered during step definition implementation:

| Issue | Description | Fix |
| --------------------------------- | ------------------------------------------------------------ | ------------------------------------------------------------------------------------------------- |
| Pattern not in `bySource.gherkin` | TraceabilityCodec shows "No Timeline Patterns" | Set `filePath: '...feature'` in `createTestPattern()` - source categorization uses file extension |
| Business value not found | REMAINING-WORK.md business value is in `additionalFiles` | Check detail files via `doc.additionalFiles` not main document sections |
| Codec output mismatch | Spec says "Next Actionable table" but codec uses list format | Debug actual output with `console.log(JSON.stringify(doc.sections))` then align test expectations |
| `behaviorFileVerified` undefined | Patterns created without explicit verification status | Add `behaviorFileVerified: true/false` to `createTestPattern()` when testing traceability |
| Discovery tags missing | SessionFindingsCodec shows "No Findings" | Pass `discoveredGaps`, `discoveredImprovements`, `discoveredLearnings` to factory |
| Issue | Description | Fix |
| ------------------------------------- | ------------------------------------------------------------ | ------------------------------------------------------------------------------------------------- |
| Pattern not in `bySourceType.gherkin` | TraceabilityCodec shows "No Timeline Patterns" | Set `filePath: '...feature'` in `createTestPattern()` - source categorization uses file extension |
| Business value not found | REMAINING-WORK.md business value is in `additionalFiles` | Check detail files via `doc.additionalFiles` not main document sections |
| Codec output mismatch | Spec says "Next Actionable table" but codec uses list format | Debug actual output with `console.log(JSON.stringify(doc.sections))` then align test expectations |
| `behaviorFileVerified` undefined | Patterns created without explicit verification status | Add `behaviorFileVerified: true/false` to `createTestPattern()` when testing traceability |
| Discovery tags missing | SessionFindingsCodec shows "No Findings" | Pass `discoveredGaps`, `discoveredImprovements`, `discoveredLearnings` to factory |

### Coding & Linting Standards

Expand Down
556 changes: 556 additions & 0 deletions docs-inbox/architectural-review-progressive-disclosure-and-codecs.md
View file
Open in desktop

Large diffs are not rendered by default.

565 changes: 565 additions & 0 deletions docs-inbox/codebase-exploration-findings.md
View file
Open in desktop

Large diffs are not rendered by default.

603 changes: 603 additions & 0 deletions docs-inbox/refactoring-execution-guide.md
View file
Open in desktop

Large diffs are not rendered by default.

34 changes: 20 additions & 14 deletions docs-live/ARCHITECTURE.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,14 +7,14 @@

## Overview

This diagram shows 65 key components with explicit architectural roles across 10 bounded contexts.
This diagram shows 66 key components with explicit architectural roles across 10 bounded contexts.

| Metric | Count |
| ------------------ | ----- |
| Diagram Components | 65 |
| Diagram Components | 66 |
| Bounded Contexts | 10 |
| Component Roles | 5 |
| Total Annotated | 168 |
| Total Annotated | 173 |

---

Expand Down Expand Up @@ -69,11 +69,11 @@ graph TB
ContentDeduplicator["ContentDeduplicator[infrastructure]"]
CodecBasedGenerator["CodecBasedGenerator[service]"]
FileCache["FileCache[infrastructure]"]
DesignReviewGenerator["DesignReviewGenerator[service]"]
DecisionDocGenerator["DecisionDocGenerator[service]"]
TransformDataset["TransformDataset[service]"]
SequenceTransformUtils["SequenceTransformUtils[service]"]
RelationshipResolver["RelationshipResolver[service]"]
DesignReviewGenerator["DesignReviewGenerator[service]"]
DecisionDocGenerator["DecisionDocGenerator[service]"]
end
subgraph lint["Lint BC"]
LintRules["LintRules[service]"]
Expand All @@ -89,6 +89,7 @@ graph TB
DesignReviewCodec["DesignReviewCodec[projection]"]
DecisionDocCodec["DecisionDocCodec[projection]"]
CompositeCodec["CompositeCodec[projection]"]
CodecRegistryBarrel["CodecRegistryBarrel[service]"]
ArchitectureCodec["ArchitectureCodec[projection]"]
end
subgraph scanner["Scanner BC"]
Expand All @@ -112,6 +113,7 @@ graph TB
Convention_Annotation_Example___DD_3_Decision["Convention Annotation Example — DD-3 Decision[decider]"]
end
DoDValidator --> DualSourceExtractor
GherkinScanner --> GherkinASTParser
MCPToolRegistry --> ProcessStateAPI
MCPToolRegistry --> MCPPipelineSession
MCPServerImpl --> MCPPipelineSession
Expand All @@ -124,16 +126,13 @@ graph TB
MCPModule --> MCPFileWatcher
MCPModule --> MCPToolRegistry
LintEngine --> LintRules
SourceMapper -.-> DecisionDocCodec
SourceMapper -.-> GherkinASTParser
Documentation_Generation_Orchestrator --> Pattern_Scanner
GherkinExtractor --> GherkinASTParser
DualSourceExtractor --> GherkinExtractor
DualSourceExtractor --> GherkinScanner
Document_Extractor --> Pattern_Scanner
ConfigResolver --> ArchitectFactory
ArchitectFactory --> RegexBuilders
ConfigLoader --> ArchitectFactory
SourceMapper -.-> DecisionDocCodec
SourceMapper -.-> GherkinASTParser
Documentation_Generation_Orchestrator --> Pattern_Scanner
ReplMode --> ProcessStateAPI
ProcessAPICLIImpl --> ProcessStateAPI
ProcessAPICLIImpl --> MasterDataset
Expand All @@ -142,6 +141,9 @@ graph TB
ProcessAPICLIImpl --> OutputPipelineImpl
OutputPipelineImpl --> PatternSummarizerImpl
MCPServerBin --> MCPServerImpl
ConfigResolver --> ArchitectFactory
ArchitectFactory --> RegexBuilders
ConfigLoader --> ArchitectFactory
PatternSummarizerImpl --> ProcessStateAPI
ScopeValidatorImpl --> ProcessStateAPI
ScopeValidatorImpl --> MasterDataset
Expand All @@ -159,18 +161,17 @@ graph TB
ContextAssemblerImpl --> FuzzyMatcherImpl
ArchQueriesImpl --> ProcessStateAPI
ArchQueriesImpl --> MasterDataset
GherkinScanner --> GherkinASTParser
FSMValidator --> FSMTransitions
FSMValidator --> FSMStates
DesignReviewCodec --> MasterDataset
ArchitectureCodec --> MasterDataset
ProcessGuardDecider --> FSMValidator
TransformDataset --> MasterDataset
SequenceTransformUtils --> MasterDataset
DesignReviewGenerator --> DesignReviewCodec
DesignReviewGenerator --> MasterDataset
DecisionDocGenerator -.-> DecisionDocCodec
DecisionDocGenerator -.-> SourceMapper
TransformDataset --> MasterDataset
SequenceTransformUtils --> MasterDataset
```

---
Expand Down Expand Up @@ -259,6 +260,7 @@ All components with architecture annotations:
| ✅ Patterns Codec | renderer | projection | application | src/renderable/codecs/patterns.ts |
| ✅ Session Codec | renderer | projection | application | src/renderable/codecs/session.ts |
| ✅ Renderable Document | renderer | read-model | domain | src/renderable/schema.ts |
| 🚧 Codec Registry Barrel | renderer | service | application | src/renderable/codecs/codec-registry.ts |
| ✅ Document Generator | renderer | service | application | src/renderable/generate.ts |
| ✅ Universal Renderer | renderer | service | application | src/renderable/render.ts |
| ✅ Gherkin AST Parser | scanner | infrastructure | infrastructure | src/scanner/gherkin-ast-parser.ts |
Expand Down Expand Up @@ -328,6 +330,10 @@ All components with architecture annotations:
| ✅ Process Guard Testing | - | - | - | tests/features/validation/process-guard.feature |
| 🚧 Process Guard Types | - | - | - | src/lint/process-guard/types.ts |
| 🚧 Process State Types | - | - | - | src/api/types.ts |
| ✅ Reference Codec | - | - | - | src/renderable/codecs/reference-types.ts |
| ✅ Reference Codec | - | - | - | src/renderable/codecs/reference-diagrams.ts |
| ✅ Reference Codec | - | - | - | src/renderable/codecs/reference-builders.ts |
| ✅ Reference Codec | - | - | - | src/renderable/codecs/product-area-metadata.ts |
| 🚧 Reference Document Codec | - | - | - | src/renderable/codecs/reference.ts |
| 🚧 Reference Generator Registration | - | - | - | src/generators/built-in/reference-generators.ts |
| ✅ Renderable Document Model(RDM) | - | - | - | src/renderable/index.ts |
Expand Down
6 changes: 3 additions & 3 deletions docs-live/BUSINESS-RULES.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@

---

**Domain constraints and invariants extracted from feature specifications. 616 rules from 139 features across 7 product areas.**
**Domain constraints and invariants extracted from feature specifications. 627 rules from 140 features across 7 product areas.**

---

Expand All @@ -16,8 +16,8 @@
| [Annotation](business-rules/annotation.md) | 21 | 90 | 90 |
| [Configuration](business-rules/configuration.md) | 7 | 31 | 31 |
| [Core Types](business-rules/core-types.md) | 9 | 34 | 34 |
| [Data API](business-rules/data-api.md) | 25 | 90 | 90 |
| [Generation](business-rules/generation.md) | 61 | 302 | 302 |
| [Data API](business-rules/data-api.md) | 25 | 91 | 91 |
| [Generation](business-rules/generation.md) | 62 | 312 | 312 |
| [Process](business-rules/process.md) | 2 | 7 | 7 |
| [Validation](business-rules/validation.md) | 14 | 62 | 62 |

Expand Down
Loading
Loading