АРЧ АП11 - Оркестрация: дожать агентный прогон по выбору компании и возрасту активности в 1С
This commit is contained in:
@@ -14,7 +14,7 @@ The goal is to turn it from a god-service into a thinner coordinator.
|
||||
|
||||
Approximate size:
|
||||
|
||||
- `5770` lines
|
||||
- `5198` lines
|
||||
|
||||
It currently mixes concerns from:
|
||||
|
||||
@@ -48,7 +48,7 @@ Current owner:
|
||||
|
||||
Current location:
|
||||
|
||||
- [assistantService.ts:4248](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:4248)
|
||||
- [assistantService.ts:3999](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:3999)
|
||||
|
||||
Target owner:
|
||||
|
||||
@@ -72,8 +72,8 @@ Current owner:
|
||||
|
||||
Current location:
|
||||
|
||||
- [assistantService.ts:2828](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:2828)
|
||||
- [assistantService.ts:3111](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:3111)
|
||||
- [assistantService.ts:2927](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:2927)
|
||||
- [assistantService.ts:2930](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:2930)
|
||||
|
||||
Target owner:
|
||||
|
||||
@@ -201,13 +201,13 @@ This order is chosen because route and transition pressure are currently the mai
|
||||
|
||||
This extraction is materially underway and no longer just a proposal.
|
||||
|
||||
Current active owner creation and wiring in [assistantService.ts](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:4725):
|
||||
Current active owner creation and wiring in [assistantService.ts](/x:/1C/NDC_1C/llm_normalizer/backend/src/services/assistantService.ts:4283):
|
||||
|
||||
- provider owner near `4725`
|
||||
- meta and memory owners near `4738-4743`
|
||||
- route owner near `4748`
|
||||
- transition owner near `4785`
|
||||
- boundary owner near `5439`
|
||||
- provider owner near `4283`
|
||||
- meta and memory owners near `4296-4301`
|
||||
- route owner near `4306`
|
||||
- transition owner near `4343`
|
||||
- boundary owner near `4997`
|
||||
|
||||
What is already true:
|
||||
|
||||
@@ -216,7 +216,7 @@ What is already true:
|
||||
|
||||
What is still not fully true:
|
||||
|
||||
- legacy helper bodies still physically remain inside `assistantService.ts`
|
||||
- some legacy helper bodies still physically remain inside `assistantService.ts`
|
||||
- the coordinator is still too large to be called thin
|
||||
- some reviews still require reading both extracted owners and the coordinator to understand final behavior
|
||||
|
||||
|
||||
@@ -199,7 +199,7 @@ Reason:
|
||||
|
||||
Remaining debt:
|
||||
|
||||
- `assistantService.ts` is still about `5770` lines;
|
||||
- `assistantService.ts` is still about `5198` lines;
|
||||
- runtime uses extracted owners, but legacy bodies and fallback branches still live in the coordinator file;
|
||||
- code review still sometimes requires reading `assistantService` together with extracted owners.
|
||||
|
||||
|
||||
Reference in New Issue
Block a user