Skip to content

Commit a19cf35

Browse files
zaebeeclaude
andcommitted
docs(architecture): prefix FQNs with cgis. in self-portrait (#252 review)
gemini: the god-class / PageRank / coupling tables dropped the cgis. prefix, contradicting the doc's own "ingest src/ so FQNs carry cgis." instruction and the audit target. Qualified FQNs now match the real command output. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
1 parent a75d706 commit a19cf35

1 file changed

Lines changed: 14 additions & 14 deletions

File tree

docs/architecture/SELF_PORTRAIT.md

Lines changed: 14 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -26,12 +26,12 @@ prefix — the domain bindings in `docs/ontology/patterns.yaml` expect it.
2626

2727
| Class | Members | Note |
2828
|---|---:|---|
29-
| `storage.sqlite_store.SQLiteStore` | 37 | data-access layer — many CRUD methods by design (sanctioned baseline) |
30-
| `query.drift.drift.DriftScorer` | 30 | drift scoring — constraint/weight/param operations (sanctioned baseline) |
31-
| `guardian.collector.ContextCollector` | 11 | review context assembly |
32-
| `extractors.typescript_extractor.TypeScriptExtractor` | 10 | AST visitor |
33-
| `query.render.metrics.DuckDBAnalyzer` | 9 | analytical layer |
34-
| `resolver.symbols.SymbolResolver` | 9 | symbol indexing |
29+
| `cgis.storage.sqlite_store.SQLiteStore` | 37 | data-access layer — many CRUD methods by design (sanctioned baseline) |
30+
| `cgis.query.drift.drift.DriftScorer` | 30 | drift scoring — constraint/weight/param operations (sanctioned baseline) |
31+
| `cgis.guardian.collector.ContextCollector` | 11 | review context assembly |
32+
| `cgis.extractors.typescript_extractor.TypeScriptExtractor` | 10 | AST visitor |
33+
| `cgis.query.render.metrics.DuckDBAnalyzer` | 9 | analytical layer |
34+
| `cgis.resolver.symbols.SymbolResolver` | 9 | symbol indexing |
3535

3636
The two large classes are the documented `_KNOWN_GOD_OBJECTS` baseline
3737
(`tests/self_parsing/test_architecture.py`) — the slice-2 restructure introduced
@@ -41,10 +41,10 @@ no new ones.
4141

4242
| Node | PageRank | In / Out | Reading |
4343
|---|---:|---|---|
44-
| `extractors._python_ast.get_identifier` | 0.0455 | 17 / 2 | the real critical path — both the most-coupled and the most-important node; if it breaks, extraction breaks |
45-
| `core.models.Edge` | 0.0128 | 18 / 0 | foundation — a stable sink everything depends on |
46-
| `core.models.Node` | 0.0105 | 10 / 0 | foundation |
47-
| `storage.sqlite_store.SQLiteStore` | 0.0077 | 24 / 0 | foundation |
44+
| `cgis.extractors._python_ast.get_identifier` | 0.0455 | 17 / 2 | the real critical path — both the most-coupled and the most-important node; if it breaks, extraction breaks |
45+
| `cgis.core.models.Edge` | 0.0128 | 18 / 0 | foundation — a stable sink everything depends on |
46+
| `cgis.core.models.Node` | 0.0105 | 10 / 0 | foundation |
47+
| `cgis.storage.sqlite_store.SQLiteStore` | 0.0077 | 24 / 0 | foundation |
4848

4949
The high-in / zero-out nodes (`Edge`, `Node`, `SQLiteStore`) are stable
5050
abstractions on the right side of the main sequence — the base the rest builds on.
@@ -53,10 +53,10 @@ abstractions on the right side of the main sequence — the base the rest builds
5353

5454
- **Hubs (fan-in):** `get_identifier` (17), `resolve_fqn` (8 — the #145 suffix
5555
resolver, reused for slice-1's root-agnostic edges and widely depended on).
56-
- **Orchestrators (fan-out):** `guardian.chunked.run_chunked_review` (18),
57-
`query.analysis.suggest_service.suggest_packages` (16),
58-
`query.drift.drift_service.analyze_drift` (15),
59-
`query.drift.drift.DriftScorer.score` (12) — top-level coordinators, high
56+
- **Orchestrators (fan-out):** `cgis.guardian.chunked.run_chunked_review` (18),
57+
`cgis.query.analysis.suggest_service.suggest_packages` (16),
58+
`cgis.query.drift.drift_service.analyze_drift` (15),
59+
`cgis.query.drift.drift.DriftScorer.score` (12) — top-level coordinators, high
6060
out-degree by nature.
6161

6262
### 📐 Shape — architectural drift

0 commit comments

Comments
 (0)