Skip to content

Commit fa7eee8

Browse files
committed
fix: correct extractor line counts and duplicate section numbering
- Update extractor line counts from 2,299 to actual 3,023 (verified via wc -l) - Fix individual per-file counts in both ROADMAP.md and architecture.md - Remove duplicate section 3.8 numbering for strikethrough/subsumed item
1 parent 5fe0a82 commit fa7eee8

File tree

2 files changed

+14
-16
lines changed

2 files changed

+14
-16
lines changed

docs/roadmap/ROADMAP.md

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -439,8 +439,8 @@ Persist and query selected AST node types for pattern-based codebase exploration
439439

440440
Split per-language extractors from monolithic `parser.js` into dedicated modules.
441441

442-
- ✅ New `src/extractors/` directory with 11 files (2,299 lines total)
443-
- ✅ One file per language: `javascript.js` (750), `csharp.js` (248), `php.js` (243), `java.js` (230), `rust.js` (225), `ruby.js` (188), `go.js` (172), `python.js` (150), `hcl.js` (73)
442+
- ✅ New `src/extractors/` directory with 11 files (3,023 lines total)
443+
- ✅ One file per language: `javascript.js` (892), `csharp.js` (311), `php.js` (322), `java.js` (290), `rust.js` (295), `ruby.js` (277), `go.js` (237), `python.js` (284), `hcl.js` (95)
444444
- ✅ Shared utilities in `helpers.js` (`nodeEndLine()`, `findChild()`)
445445
- ✅ Barrel export via `index.js`
446446
- ✅ Consistent return schema: `{ definitions, calls, imports, classes, exports }`
@@ -705,9 +705,7 @@ Export only `*Data()` functions (the command execute functions). Never export CL
705705

706706
**Affected files:** `src/index.js`, `package.json`
707707

708-
### ~~3.8~~ -- ~~Decompose complexity.js~~ Subsumed by 3.1
709-
710-
> The standalone complexity decomposition from the previous revision is now part of the unified AST analysis framework (3.1). The `complexity.js` per-language rules become `ast-analysis/rules/complexity/{lang}.js` alongside CFG and dataflow rules.
708+
> **Removed: Decompose complexity.js** — Subsumed by 3.1. The standalone complexity decomposition from the previous revision is now part of the unified AST analysis framework (3.1). The `complexity.js` per-language rules become `ast-analysis/rules/complexity/{lang}.js` alongside CFG and dataflow rules.
711709
712710
### 3.8 -- Domain Error Hierarchy
713711

generated/architecture.md

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
| Test files | 59 | 70 | +11 |
3131
| Node kinds | 10 | 13 | +3 (parameter, property, constant) |
3232
| Edge kinds | 6 | 9 | +3 (contains, parameter_of, receiver) |
33-
| Extractor modules | 0 (inline in parser.js) | 11 files, 2,299 lines | New directory |
33+
| Extractor modules | 0 (inline in parser.js) | 11 files, 3,023 lines | New directory |
3434

3535
**Key patterns observed in this burst:**
3636

@@ -209,19 +209,19 @@ Both complexity and CFG analysis walk the same AST trees with language-specific
209209

210210
**Previous state:** parser.js at 404 lines with inline extractors.
211211

212-
**Current state:** `src/extractors/` directory with 11 files totaling 2,299 lines:
212+
**Current state:** `src/extractors/` directory with 11 files totaling 3,023 lines:
213213

214214
| File | Lines | Language |
215215
|------|-------|----------|
216-
| `javascript.js` | 750 | JS/TS/TSX |
217-
| `csharp.js` | 248 | C# |
218-
| `php.js` | 243 | PHP |
219-
| `java.js` | 230 | Java |
220-
| `rust.js` | 225 | Rust |
221-
| `ruby.js` | 188 | Ruby |
222-
| `go.js` | 172 | Go |
223-
| `python.js` | 150 | Python |
224-
| `hcl.js` | 73 | Terraform/HCL |
216+
| `javascript.js` | 892 | JS/TS/TSX |
217+
| `csharp.js` | 311 | C# |
218+
| `php.js` | 322 | PHP |
219+
| `java.js` | 290 | Java |
220+
| `rust.js` | 295 | Rust |
221+
| `ruby.js` | 277 | Ruby |
222+
| `go.js` | 237 | Go |
223+
| `python.js` | 284 | Python |
224+
| `hcl.js` | 95 | Terraform/HCL |
225225
| `helpers.js` | 11 | Shared utilities |
226226
| `index.js` | 9 | Barrel export |
227227

0 commit comments

Comments
 (0)