Skip to content

Commit d83c2c1

Browse files
Copilotjkordick
andcommitted
docs: fill three documentation gaps from PR #45 (BL injection)
Co-authored-by: jkordick <52427852+jkordick@users.noreply.github.com>
1 parent cc22973 commit d83c2c1

File tree

2 files changed

+5
-2
lines changed

2 files changed

+5
-2
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
99

1010
### Added
1111
- **Business Logic Persistence**`ReverseEngineeringProcess` and `ChunkedReverseEngineeringProcess` now persist extracted `BusinessLogic` records to a new `business_logic` SQLite table via `IMigrationRepository.SaveBusinessLogicAsync`. Added `GetBusinessLogicAsync` and `DeleteBusinessLogicAsync` to `IMigrationRepository`, `SqliteMigrationRepository`, and `HybridMigrationRepository`.
12+
- **Business Logic Injection into Conversion Prompts** — All four converter agents (`JavaConverterAgent`, `CSharpConverterAgent`, `ChunkAwareJavaConverter`, `ChunkAwareCSharpConverter`) now receive extracted `BusinessLogic` records via `SetBusinessLogicContext()` (new method on `ICodeConverterAgent`). In full-pipeline runs, `SmartMigrationOrchestrator` wires RE output directly into conversion; `--reuse-re` loads the same context from a previous persisted RE run. A shared `FormatBusinessLogicContext()` helper in `AgentBase` formats the context for all four converters.
1213
- **`--reuse-re` CLI flag** — When combined with `--skip-reverse-engineering`, loads business logic from the latest persisted RE run and injects it into conversion prompts. `doctor.sh convert-only` now prompts interactively for this choice.
1314
- **REST API: `GET/DELETE /api/runs/{runId}/business-logic`** — Returns per-file business logic summary (story/feature/rule counts); DELETE removes persisted results to allow re-running RE for that run.
1415
- **Portal: per-run `🔬 RE Results` button** — Shows the business logic summary table for a run and allows deletion of persisted results directly from the UI.

README.md

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -791,12 +791,13 @@ sequenceDiagram
791791
BizLogic-->>RE: BusinessLogic[]
792792
RE->>Repo: persist analyses + documentation
793793
RE->>Repo: persist BusinessLogic[] to business_logic table
794-
RE-->>CLI: ReverseEngineeringResult (with RunId)
794+
RE-->>CLI: ReverseEngineeringResult (BusinessLogic[], RunId)
795+
CLI->>Migration: SetBusinessLogicContext(BusinessLogic[])
795796
CLI->>Migration: start migration run with latest analyses
796797
Migration->>Analyzer: reuse or refresh CobolAnalysis
797798
Migration->>DepMap: build dependency graph (CALL/COPY/...)
798799
DepMap-->>Migration: DependencyMap
799-
Migration->>Converter: convert to Java/C# (AI-limited concurrency)
800+
Migration->>Converter: convert to Java/C# with business logic context
800801
Converter-->>Migration: CodeFile artifacts
801802
Migration->>Repo: persist run metadata, graph edges, code files
802803
Repo-->>Portal: expose MCP resources + REST APIs
@@ -838,6 +839,7 @@ sequenceDiagram
838839
- `CobolAnalysis` per file
839840
- Target language settings (Quarkus vs. .NET)
840841
- Migration run metadata (for logging & metrics)
842+
- `BusinessLogic` records per file (user stories, features, business rules) — injected automatically from RE output in full-pipeline runs, or loaded from DB when `--reuse-re` is used
841843
- **Outputs:** `CodeFile` records saved under `output/java/` or `output/csharp/`.
842844
- **Interactions:**
843845
- Concurrency guards (pipeline slots vs. AI calls) ensure Azure OpenAI limits respected.

0 commit comments

Comments
 (0)