@thanh01.pmt/domain-kit 0.6.0 → 0.6.1

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2025 Tony Pham
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README.md

@@ -112,7 +112,7 @@ them, instead of surfacing as a broken lesson three packages later.
  runProjectGraph   runKnowledgeGraph   runProjectGraphPipeline + runKnowledgeGraphPipeline
  Pipeline          Pipeline            (two full graph runs), then:
  STEP_1 scan/      (parseSyllabus      generateHybridGraph()
-   select/AST        first if raw        STEP_1 feature-concept links
+   select/regex      first if raw        STEP_1 feature-concept links
  STEP_2 keyword      syllabusText)       STEP_2 decomposePhases
    extract           generateKnowledge     (auditPhasePlan fail-closed)
  STEP_3 SDK API      Graph:              STEP_3 totals
@@ -136,8 +136,8 @@ them, instead of surfacing as a broken lesson three packages later.
                               v
  +------------------------------------------------------------------------------+
  | LAYER 3 - emitCurriculumFeed(graph) - pure projection, no LLM                |
- |   Called from THREE places (R3): router for the project branch               |
- |   (domainRouter.ts:171), knowledge orchestrator, hybrid orchestrator.        |
+ |   Options built in ONE shared helper (pipeline/emitFeed.ts, R3 fixed):       |
+ |   router (project), knowledge orchestrator, hybrid orchestrator.             |
  |   learning_nodes[] - dependency_edges[] - suggested_groupings[] - phases[]   |
  |   (phases[] is filled ONLY by the hybrid emitter - R1)                       |
  +------------------------------------+-----------------------------------------+
@@ -190,10 +190,10 @@ verified against. Full postmortems: deep review 2026-10-05 (app repo,
 |---|---|---|---|
 | R1 | **FIXED 2026-10-07.** Was: `hybrid` without `repoDir` silently degraded to a knowledge-graph feed (only a `warnings[]` entry). Now a FORCED hybrid throws (fail-closed); an auto-detected downgrade returns a structured `modeDowngrade { from, to, reason }`. | `src/pipeline/domainRouter.ts` | High |
 | R2 | **FIXED 2026-10-07.** Was: the hybrid branch's knowledge legs dropped `knowledgeTreePath`/`proposalsPath`, so STEP_2_5 ran in concept mode but not hybrid mode for the same subject. Now both hybrid legs pass them through. | `src/pipeline/domainRouter.ts` | High |
-| R3 | Feed emission is invoked from three different places, and emission options already diverge (`hallucinationCount` is only passed by the router). Drift risk with every emitter change. | `domainRouter.ts:171`, `pipeline/knowledgeGraphPipeline.ts:37`, `pipeline/hybridGraphPipeline.ts:22` | Medium |
-| R4 | Cycle breaking mutates `prerequisites` in place ("weakest edge" heuristic) and only warns. Which edge is dropped decides teaching order - a pedagogy-relevant decision hidden in a warning string. | `src/graph/knowledgeGraphPipeline.ts:383` | Medium |
-| R5 | STEP_2_5 only runs when `knowledgeTreePath` is provided. Without it, every concept stays `proposed_new` and nothing in the output flags the absence of tree anchoring. | `src/graph/knowledgeGraphPipeline.ts:322` | Medium |
-| R6 | Tree prerequisite edges are seeded only when the referenced tree concept is part of THIS analysis - partial graphs silently lose cross-analysis prerequisite edges. Matters for gold-set coverage measurement across runs. | `src/graph/knowledgeGraphPipeline.ts:363-375` | Medium |
+| R3 | **FIXED 2026-10-07.** Was: feed emission invoked from three places with drifting options (`hallucinationCount` only from the router). Now ONE shared helper `emitNormalizedFeed()` in `src/pipeline/emitFeed.ts` builds options for all three call sites (lives in pipeline/ so feed/ keeps its no-llmClient invariant). | `src/pipeline/emitFeed.ts` | Medium |
+| R4 | Cycle breaking removes the DFS back-edge positionally (mechanism verified 2026-10-07: NO "weakest edge"/link-counting weighting — that was the old misleading log text, now fixed) and only warns. Which edge is dropped decides teaching order. | `src/graph/knowledgeGraphVerifier.ts:69` | Medium |
+| R5 | **FIXED 2026-10-07.** Was: STEP_2_5 only ran when `knowledgeTreePath` was provided and the absence of tree anchoring was silent. Now the pipeline pushes an explicit warning (all concepts then stay `proposed_new`). | `src/graph/knowledgeGraphPipeline.ts` | Medium |
+| R6 | **FIXED 2026-10-07.** Was: tree prereq codes outside THIS analysis were dropped silently. Now one aggregated warning lists unmapped tree prereq codes (deduped, display capped at 10). | `src/graph/knowledgeGraphPipeline.ts` | Medium |
 | R7 | `emitCurriculumFeed` dispatches by shape (key presence) and pass-through-validates anything that "looks like a feed" (CF-8 in the deep review). | `src/feed/curriculumFeedEmitter.ts:833-839` | Low |
 | R8 | Non-fatal pedagogy issues funnel into a single lossy `warnings[]` string channel; some drops only hit `console.warn` (e.g. a concept falling out of `learning_path`, KG-3). Structured diagnostics per ADR-0002 should replace string warnings. | deep review KG-3 | Medium |
 | R9 | **FIXED 2026-10-07.** Was: ASCII-only regex assumptions broke on Vietnamese text (KG-1) - empty derived keywords threw, registries filled with fragments. Now a shared `stripDiacritics()` (NFD + combining-mark strip + `đ`→`d`, incl. the precomposed-base-letter case NFD cannot decompose) backs every tokenizer; regression tests in `src/__tests__/vietnamese-tokenization.test.ts`. | `src/utils/diacritics.ts` | Critical |
@@ -255,7 +255,7 @@ recomputed at the branch.
  STEP_1    parseSyllabus(syllabusText) → structured units/topics
            LLM decomposes the subject → concepts + prerequisites + problem_types
            + categories + techniques. The syllabus is truncated at
-           KG_MAX_SYLLABUS_CHARS (default 16000) with a warning.
+           KG_MAX_SYLLABUS_CHARS (default 32000) with a warning.
 
  STEP_2    validate & clean
            · description shorter than 50 chars → warning
@@ -527,7 +527,7 @@ LLM_MAX_TOKENS=65536          # Max output tokens (default)
 LLM_MIN_REQUEST_INTERVAL_MS=2500  # process-wide pacing; 0 disables
 
 # Extra knobs the pipelines read directly:
-KG_MAX_SYLLABUS_CHARS=16000   # syllabus truncation before decomposition
+KG_MAX_SYLLABUS_CHARS=32000   # syllabus truncation before decomposition
 KG_MAX_TOKENS=65536           # knowledge-graph generation budget
 LLM_REQUEST_TIMEOUT_MS=300000 # per-request cap
 ```
@@ -1278,7 +1278,7 @@ orchestrators call it for theirs.
 
 | Step | What happens | Fail mode |
 |---|---|---|
-| `STEP_1` | parse the syllabus into structured units/topics, then LLM-decompose the subject into concepts, prerequisites, `problem_types`, categories | syllabus truncated at `KG_MAX_SYLLABUS_CHARS` (default 16000) + warning |
+| `STEP_1` | parse the syllabus into structured units/topics, then LLM-decompose the subject into concepts, prerequisites, `problem_types`, categories | syllabus truncated at `KG_MAX_SYLLABUS_CHARS` (default 32000) + warning |
 | `STEP_2` | validate & clean — description ≥ 50 chars; **keyword registry fail-closed** (empty keywords are derived from name/description; nothing derivable → `throw`); dangling prerequisite edges removed | `throw` when keywords cannot be derived |
 | `STEP_2_5` | *only when `knowledgeTreePath` is supplied* — standardize every concept against `mlo-knowlege-tree.tsv` into `standardized` or `proposed_new`; adopted tech-stack keywords merge into the registry; tree prerequisite codes seed internal edges | missing decision for a concept → `throw` |
 | `STEP_3` | `detectPrerequisiteCycles()` → `breakPrerequisiteCycles()` at the weakest edge | warnings; edges are removed |

package/dist/assembly/index.d.cts

@@ -1,7 +1,8 @@
 import { u as ProjectGraph, c as Step } from '../projectGraphSchema-DnD7orZV.cjs';
-import { C as ConceptMapping } from '../conceptEscalator-DPrs4PwC.cjs';
+import { C as ConceptMapping } from '../conceptEscalator-BmMJD8C5.cjs';
 import 'zod';
-import '../llmClient-CX5uUiQ1.cjs';
+import '../utils/llmClient.cjs';
+import '../utils/llmTransportResult.cjs';
 
 /**
  * Roadmap assembly — assembles final roadmap with keyword tracking and time awareness.

package/dist/assembly/index.d.ts

@@ -1,7 +1,8 @@
 import { u as ProjectGraph, c as Step } from '../projectGraphSchema-DnD7orZV.js';
-import { C as ConceptMapping } from '../conceptEscalator-BCsw0ys_.js';
+import { C as ConceptMapping } from '../conceptEscalator-DMsehscs.js';
 import 'zod';
-import '../llmClient-CX5uUiQ1.js';
+import '../utils/llmClient.js';
+import '../utils/llmTransportResult.js';
 
 /**
  * Roadmap assembly — assembles final roadmap with keyword tracking and time awareness.