Compare commits
36 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
| cf0be8ac0f | |||
| 5e3d20fa61 | |||
| 65722c76cb | |||
| 906ae4f587 | |||
| 20cc132777 | |||
| 4d7e9ca0e4 | |||
| 6174830311 | |||
| 3ef1e625eb | |||
| 2c28f1cb86 | |||
| 6dec34403f | |||
| 4f16cc3c83 | |||
| 45691d0324 | |||
| 8c350572df | |||
| 22e81cc896 | |||
| 3bbd7787d8 | |||
| 07d73870ba | |||
| 7408bc4232 | |||
| 9d68fc0ad6 | |||
| e2fb20ef05 | |||
| 5427cdc740 | |||
| f3250ce178 | |||
| 472ea8c75c | |||
| b053796182 | |||
| 3b6992d8e9 | |||
| 233f9984fc | |||
| 54b1de78b8 | |||
| ee5bc56f2d | |||
| df2d091174 | |||
| 4c9a1e9ccb | |||
| 65c2c5e471 | |||
| f6ba9bc1e7 | |||
| 05076c4f1d | |||
| f943b229c0 | |||
| 28671cb012 | |||
| d86d375ce4 | |||
| 4f5cf263f6 |
@@ -38,5 +38,7 @@ See `references/aggregation-template.md`.
|
||||
|
||||
## Behavioral rule reminders
|
||||
|
||||
- **«Не использован ≠ проблема»** — when reporting node usage counts, NEVER mark unused nodes as «zombie» / «removal candidate». Cite `memory/feedback_brain_unused_tools_not_problem.md`.
|
||||
- **«Не использован ≠ проблема» (условное, Pravila §16.4 v1.36)** — when reporting node usage counts, distinguish two cases:
|
||||
1. **Unused + no profile task in episodes** → capability-readiness, do NOT flag.
|
||||
2. **Unused + profile task present (missed activation)** → mandatory section in the report. Cite `tools/observer-classification-map.json` for the classification→node mapping and `tools/.node-dormancy.json` for DEFERRED exclusions. NEVER mark unused-by-design nodes as «zombie» / «removal candidate».
|
||||
- **No auto-edit** — every regulatory suggestion is a candidate, not an action.
|
||||
|
||||
@@ -55,6 +55,32 @@ For each factor below, render a table: factor value × outcome counts
|
||||
|
||||
(one table each — same columns)
|
||||
|
||||
## Missed Activations (Pravila §16.4 v1.36)
|
||||
|
||||
Surface candidates where a profile-classified task ran with `node_chosen === 'direct'` and at least one non-dormant recommended node was available. The analyzer returns `missedActivations: { totalMissed, byNode, byClassification }` — render the two breakdowns below.
|
||||
|
||||
**Source:** `analyze(episodes, { classificationMap, dormancy }).missedActivations`.
|
||||
|
||||
### By node
|
||||
|
||||
| Node | Episodes missed | Classifications hit |
|
||||
|---|---|---|
|
||||
| #NN | N | refactor (a), bugfix (b) |
|
||||
|
||||
### By classification
|
||||
|
||||
| Classification | Missed episodes | Top recommended nodes (non-dormant) |
|
||||
|---|---|---|
|
||||
| refactor | N | #11, #12, #43 |
|
||||
|
||||
**Interpretation guide:**
|
||||
|
||||
- High count on one node → router-miss pattern. Suggest updating `tools/observer-classification-map.json` or a workflow nudge.
|
||||
- Spread across many nodes with classification leaning to `other` → the classification dictionary may need refinement (separate concern, not a missed activation).
|
||||
- All zero → either no profile work this period, or the router is operating cleanly.
|
||||
|
||||
**NOT to be auto-applied:** these are candidates for human review in retro, not commits or hook blocks.
|
||||
|
||||
## Episodes → tasks (from analyzer `tasks`)
|
||||
|
||||
| task_ref | episodes | turns that are rework |
|
||||
@@ -70,10 +96,14 @@ For each factor below, render a table: factor value × outcome counts
|
||||
- `observerErrorCount` from the analyzer — observer_error markers in the period.
|
||||
Non-zero = the observer failed silently somewhere; investigate.
|
||||
|
||||
## Canonical chains L1–L12 hit rate
|
||||
## Canonical chains L1–L13+ hit rate (from analyzer `factorMatrix.chain_ref`)
|
||||
|
||||
| chain | times | notes |
|
||||
|---|---|---|
|
||||
| chain | times | outcome split | notes |
|
||||
|---|---|---|---|
|
||||
|
||||
Each node may belong to several L (a multi-chain episode is counted in each).
|
||||
`null` = episodes outside any chain (`direct` + nodes not in L1–L13+) — **not a
|
||||
problem** per `memory/feedback_brain_unused_tools_not_problem`.
|
||||
|
||||
## Improvised chains (path_type=improvised, repeated ≥2)
|
||||
|
||||
@@ -109,4 +139,4 @@ For each factor below, render a table: factor value × outcome counts
|
||||
## Informational metrics (NOT alerts)
|
||||
|
||||
- Nodes used at least once this period: K / 60+
|
||||
- Nodes never used since beginning of observer logs: L / 60+ — **not a problem** per [feedback_brain_unused_tools_not_problem](../../../memory/feedback_brain_unused_tools_not_problem.md)
|
||||
- Nodes never used since beginning of observer logs: L / 67 — **not a problem if there was no profile task** per Pravila §16.4 v1.36 and [feedback_brain_unused_tools_not_problem](../../../memory/feedback_brain_unused_tools_not_problem.md). See `## Missed Activations` above for profile-task-present cases.
|
||||
|
||||
@@ -0,0 +1,6 @@
|
||||
# gitleaks false-positive allowlist (fingerprints).
|
||||
# Format: one fingerprint per line. `gitleaks detect --report-format json` outputs them.
|
||||
|
||||
# Nuclei docs `-u http://...` — nuclei's -u flag is "target URL", not curl basic-auth.
|
||||
# Rule `curl-auth-user` matches the pattern but it's not authentication.
|
||||
f696ca50266eb1c2974b5fc89f6fa585edaf4b6b:docs/security/nuclei-setup.md:curl-auth-user:27
|
||||
@@ -74,7 +74,6 @@ class DashboardController extends Controller
|
||||
// --- active projects ---
|
||||
$activeProjects = DB::table('projects')
|
||||
->where('tenant_id', $tenantId)
|
||||
->whereNull('archived_at')
|
||||
->where('is_active', true)
|
||||
->count();
|
||||
$maxProjects = (int) (($tenant->limits['max_projects'] ?? 0));
|
||||
|
||||
@@ -52,16 +52,12 @@ class ProjectController extends Controller
|
||||
|
||||
// Фильтр по статусу жизненного цикла
|
||||
$status = $request->query('status');
|
||||
if ($status === 'archived') {
|
||||
$query->archived();
|
||||
} elseif ($status === 'active') {
|
||||
$query->active()->where('is_active', true);
|
||||
if ($status === 'active') {
|
||||
$query->where('is_active', true);
|
||||
} elseif ($status === 'paused') {
|
||||
$query->active()->where('is_active', false);
|
||||
} else {
|
||||
// По умолчанию: все не архивированные (active + paused)
|
||||
$query->active();
|
||||
$query->where('is_active', false);
|
||||
}
|
||||
// default → no extra filter
|
||||
|
||||
// Поиск по name и signal_identifier
|
||||
if ($search = $request->query('search')) {
|
||||
@@ -111,11 +107,11 @@ class ProjectController extends Controller
|
||||
return response()->json(['data' => new ProjectResource($project)]);
|
||||
}
|
||||
|
||||
/** DELETE /api/projects/{id} — soft-archive (sets archived_at, is_active=false) */
|
||||
/** DELETE /api/projects/{id} — hard delete (guard по сделкам: 422 если есть сделки) */
|
||||
public function destroy(Request $request, int $id): JsonResponse
|
||||
{
|
||||
$project = Project::where('tenant_id', $request->user()->tenant_id)->findOrFail($id);
|
||||
$this->projects->archive($project);
|
||||
$this->projects->delete($project);
|
||||
|
||||
return response()->json(null, 204);
|
||||
}
|
||||
@@ -139,7 +135,7 @@ class ProjectController extends Controller
|
||||
return response()->json(['data' => new ProjectResource($project->fresh())]);
|
||||
}
|
||||
|
||||
/** POST /api/projects/bulk — batch pause/resume/archive/update_regions/update_days/update_limit */
|
||||
/** POST /api/projects/bulk — batch pause/resume/delete/update_regions/update_days/update_limit */
|
||||
public function bulk(BulkProjectActionRequest $request): JsonResponse
|
||||
{
|
||||
$tenantId = $request->user()->tenant_id;
|
||||
|
||||
@@ -20,7 +20,7 @@ class BulkProjectActionRequest extends FormRequest
|
||||
|
||||
$rules = [
|
||||
'action' => ['required', Rule::in([
|
||||
'pause', 'resume', 'archive',
|
||||
'pause', 'resume', 'delete',
|
||||
'update_regions', 'update_days', 'update_limit',
|
||||
])],
|
||||
'ids' => ['nullable', 'array', 'max:500'],
|
||||
@@ -28,7 +28,7 @@ class BulkProjectActionRequest extends FormRequest
|
||||
'scope' => ['nullable', 'array'],
|
||||
'scope.filter' => ['nullable', 'array'],
|
||||
'scope.filter.signal_type' => ['nullable', 'string', Rule::in(['site', 'call', 'sms'])],
|
||||
'scope.filter.status' => ['nullable', 'string', Rule::in(['active', 'paused', 'archived'])],
|
||||
'scope.filter.status' => ['nullable', 'string', Rule::in(['active', 'paused'])],
|
||||
'scope.filter.search' => ['nullable', 'string', 'max:255'],
|
||||
];
|
||||
|
||||
|
||||
@@ -13,9 +13,6 @@ class ProjectResource extends JsonResource
|
||||
{
|
||||
public function toArray(Request $request): array
|
||||
{
|
||||
/** @var Project $project */
|
||||
$project = $this->resource;
|
||||
|
||||
return [
|
||||
'id' => $this->id,
|
||||
'name' => $this->name,
|
||||
@@ -28,7 +25,6 @@ class ProjectResource extends JsonResource
|
||||
'delivered_today' => $this->delivered_today,
|
||||
'delivered_in_month' => $this->delivered_in_month,
|
||||
'is_active' => $this->is_active,
|
||||
'archived_at' => $project->archived_at?->toIso8601String(),
|
||||
'region_mask' => $this->region_mask,
|
||||
'region_mode' => $this->region_mode,
|
||||
'regions' => $this->regions,
|
||||
|
||||
@@ -0,0 +1,80 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
namespace App\Jobs\Supplier;
|
||||
|
||||
use App\Models\SupplierProject;
|
||||
use App\Services\Supplier\SupplierPortalClient;
|
||||
use Illuminate\Bus\Queueable;
|
||||
use Illuminate\Contracts\Queue\ShouldQueue;
|
||||
use Illuminate\Foundation\Bus\Dispatchable;
|
||||
use Illuminate\Queue\InteractsWithQueue;
|
||||
use Illuminate\Queue\SerializesModels;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
use Illuminate\Support\Facades\Log;
|
||||
use Throwable;
|
||||
|
||||
/**
|
||||
* Удаление/пере-синк доноров у поставщика после удаления Лидерра-проекта.
|
||||
*
|
||||
* Для каждого supplier_project S (донора), к которому был привязан удалённый проект:
|
||||
* - остались другие потребители (project_supplier_links) → донор нужен другим клиентам:
|
||||
* НЕ удаляем у поставщика, пере-синкаем агрегат (SyncSupplierProjectsJob).
|
||||
* - потребителей не осталось → удаляем у поставщика (deleteProject) + локальную запись S.
|
||||
*
|
||||
* Spec: docs/superpowers/specs/2026-05-21-project-delete-dedup-errors-design.md §Решение 2.
|
||||
*/
|
||||
class DeleteSupplierProjectJob implements ShouldQueue
|
||||
{
|
||||
use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
|
||||
|
||||
public int $tries = 3;
|
||||
|
||||
public int $backoff = 60;
|
||||
|
||||
public const string DB_CONNECTION = 'pgsql_supplier';
|
||||
|
||||
/** @param array<int,int> $supplierProjectIds */
|
||||
public function __construct(public array $supplierProjectIds) {}
|
||||
|
||||
public function handle(SupplierPortalClient $client): void
|
||||
{
|
||||
$needsResync = false;
|
||||
|
||||
foreach ($this->supplierProjectIds as $id) {
|
||||
$sp = SupplierProject::on(self::DB_CONNECTION)->find($id);
|
||||
if ($sp === null) {
|
||||
continue;
|
||||
}
|
||||
|
||||
$remaining = DB::connection(self::DB_CONNECTION)
|
||||
->table('project_supplier_links')
|
||||
->where('supplier_project_id', $id)
|
||||
->count();
|
||||
|
||||
if ($remaining > 0) {
|
||||
$needsResync = true;
|
||||
|
||||
continue;
|
||||
}
|
||||
|
||||
if ($sp->supplier_external_id !== null && $sp->supplier_external_id !== '') {
|
||||
try {
|
||||
$client->deleteProject((int) $sp->supplier_external_id);
|
||||
} catch (Throwable $e) {
|
||||
Log::warning('supplier.delete_donor_failed', [
|
||||
'supplier_project_id' => $id, 'error' => $e->getMessage(),
|
||||
]);
|
||||
throw $e; // retry the job
|
||||
}
|
||||
}
|
||||
|
||||
$sp->delete();
|
||||
}
|
||||
|
||||
if ($needsResync) {
|
||||
SyncSupplierProjectsJob::dispatch();
|
||||
}
|
||||
}
|
||||
}
|
||||
@@ -37,7 +37,7 @@ use Throwable;
|
||||
* (расписание перенесено 20:30 → 18:00, см. routes/console.php).
|
||||
*
|
||||
* Алгоритм (план 3 Task 5 → переработан: one-group-per-identifier):
|
||||
* 1. Загрузить активные Лидерра-projects (is_active=true, archived_at IS NULL).
|
||||
* 1. Загрузить активные Лидерра-projects (is_active=true).
|
||||
* 2. Сгруппировать по (signal_type, identifier) — БЕЗ subject_code:
|
||||
* - identifier = buildUniqueKeyAgnostic() (site/call → signal_identifier; sms+keyword → sender+keyword; sms → sender).
|
||||
* - platforms = resolvePlatforms() (site/call → B1+B2+B3; sms+keyword → B2+B3; sms → B3).
|
||||
@@ -86,7 +86,6 @@ class SyncSupplierProjectsJob implements ShouldQueue
|
||||
/** @var Collection<int, Project> $projects */
|
||||
$projects = Project::on(self::DB_CONNECTION)
|
||||
->where('is_active', true)
|
||||
->whereNull('archived_at')
|
||||
->orderBy('id')
|
||||
->get();
|
||||
|
||||
|
||||
@@ -40,8 +40,6 @@ class Project extends Model
|
||||
'tag',
|
||||
'type',
|
||||
'is_active',
|
||||
// Plan 5 Task 1 (schema v8.20): soft archive flow — lifecycle-state рядом с is_active.
|
||||
'archived_at',
|
||||
'daily_limit_target',
|
||||
'effective_daily_limit_today',
|
||||
'effective_limit_calculated_at',
|
||||
@@ -87,8 +85,6 @@ class Project extends Model
|
||||
'sms_senders' => 'array',
|
||||
'delivered_in_month' => 'integer',
|
||||
'delivered_today' => 'integer',
|
||||
// Plan 5 Task 1 (schema v8.20): soft archive.
|
||||
'archived_at' => 'datetime',
|
||||
];
|
||||
}
|
||||
|
||||
@@ -151,33 +147,6 @@ class Project extends Model
|
||||
return $query->where('signal_type', $signalType)->where('signal_identifier', $identifier);
|
||||
}
|
||||
|
||||
/**
|
||||
* Не архивированные проекты (archived_at IS NULL).
|
||||
*
|
||||
* Внимание: scope не фильтрует is_active. Приостановленные (is_active=false)
|
||||
* проекты сюда попадают — это разные lifecycle-состояния. Если нужны только
|
||||
* «работающие» (не архив И не на паузе) — комбинируйте:
|
||||
* ->active()->where('is_active', true).
|
||||
*
|
||||
* @param Builder<Project> $query
|
||||
* @return Builder<Project>
|
||||
*/
|
||||
public function scopeActive(Builder $query): Builder
|
||||
{
|
||||
return $query->whereNull('archived_at');
|
||||
}
|
||||
|
||||
/**
|
||||
* Архивированные проекты (archived_at IS NOT NULL).
|
||||
*
|
||||
* @param Builder<Project> $query
|
||||
* @return Builder<Project>
|
||||
*/
|
||||
public function scopeArchived(Builder $query): Builder
|
||||
{
|
||||
return $query->whereNotNull('archived_at');
|
||||
}
|
||||
|
||||
/**
|
||||
* Все связанные SupplierProject из eager-loaded BelongsTo отношений.
|
||||
*
|
||||
|
||||
@@ -4,10 +4,12 @@ declare(strict_types=1);
|
||||
|
||||
namespace App\Services\Project;
|
||||
|
||||
use App\Jobs\Supplier\DeleteSupplierProjectJob;
|
||||
use App\Jobs\SyncSupplierProjectJob;
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
use Illuminate\Http\Exceptions\HttpResponseException;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
|
||||
class ProjectService
|
||||
{
|
||||
@@ -19,7 +21,6 @@ class ProjectService
|
||||
$data['tenant_id'], $data['signal_type'],
|
||||
$data['delivered_today'], $data['delivered_in_month'],
|
||||
$data['supplier_b1_project_id'], $data['supplier_b2_project_id'], $data['supplier_b3_project_id'],
|
||||
$data['archived_at'],
|
||||
);
|
||||
|
||||
if (isset($data['daily_limit_target']) && $data['daily_limit_target'] < $project->delivered_today) {
|
||||
@@ -41,6 +42,18 @@ class ProjectService
|
||||
|| array_key_exists('daily_limit_target', $data)
|
||||
|| array_key_exists('delivery_days_mask', $data);
|
||||
|
||||
if (array_key_exists('signal_identifier', $data) || array_key_exists('sms_senders', $data) || array_key_exists('sms_keyword', $data)) {
|
||||
$this->assertSourceUnique($project->tenant_id, array_merge([
|
||||
'signal_type' => $project->signal_type,
|
||||
'signal_identifier' => $project->signal_identifier,
|
||||
'sms_senders' => $project->sms_senders,
|
||||
'sms_keyword' => $project->sms_keyword,
|
||||
], $data), exceptId: $project->id);
|
||||
}
|
||||
if (array_key_exists('name', $data)) {
|
||||
$this->assertNameUnique($project->tenant_id, (string) $data['name'], exceptId: $project->id);
|
||||
}
|
||||
|
||||
$project->update($data);
|
||||
|
||||
if ($needsResync) {
|
||||
@@ -50,17 +63,26 @@ class ProjectService
|
||||
return $project->fresh();
|
||||
}
|
||||
|
||||
public function archive(Project $project): void
|
||||
public function delete(Project $project): void
|
||||
{
|
||||
if ($project->archived_at !== null) {
|
||||
$hasDeals = DB::table('deals')->where('project_id', $project->id)->exists();
|
||||
if ($hasDeals) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'message' => 'Project уже архивирован.',
|
||||
], 409));
|
||||
'errors' => ['project' => ['Нельзя удалить проект: по нему есть сделки. Поставьте приём на паузу, чтобы скрыть проект из работы.']],
|
||||
], 422));
|
||||
}
|
||||
|
||||
// Капчим доноров ДО удаления — pivot уйдёт каскадом.
|
||||
$supplierProjectIds = DB::table('project_supplier_links')
|
||||
->where('project_id', $project->id)
|
||||
->pluck('supplier_project_id')
|
||||
->all();
|
||||
|
||||
$project->delete(); // hard delete (Project без SoftDeletes); cascade чистит pivot + служебные.
|
||||
|
||||
if ($supplierProjectIds !== []) {
|
||||
DeleteSupplierProjectJob::dispatch(array_map('intval', $supplierProjectIds));
|
||||
}
|
||||
$project->update([
|
||||
'is_active' => false,
|
||||
'archived_at' => now(),
|
||||
]);
|
||||
}
|
||||
|
||||
public function triggerSync(Project $project): void
|
||||
@@ -83,9 +105,8 @@ class ProjectService
|
||||
}
|
||||
if (! empty($filter['status'])) {
|
||||
match ($filter['status']) {
|
||||
'active' => $query->where('is_active', true)->whereNull('archived_at'),
|
||||
'paused' => $query->where('is_active', false)->whereNull('archived_at'),
|
||||
'archived' => $query->whereNotNull('archived_at'),
|
||||
'active' => $query->where('is_active', true),
|
||||
'paused' => $query->where('is_active', false),
|
||||
default => null,
|
||||
};
|
||||
}
|
||||
@@ -108,7 +129,7 @@ class ProjectService
|
||||
return match ($action) {
|
||||
'pause' => $this->bulkSimpleUpdate($query, ['is_active' => false]),
|
||||
'resume' => $this->bulkSimpleUpdate($query, ['is_active' => true]),
|
||||
'archive' => $this->bulkSimpleUpdate($query, ['is_active' => false, 'archived_at' => now()]),
|
||||
'delete' => $this->bulkDelete($query),
|
||||
'update_regions' => $this->bulkUpdateRegions($query, $payload),
|
||||
'update_days' => $this->bulkUpdateDays($query, $payload),
|
||||
'update_limit' => $this->bulkUpdateLimit($query, $payload),
|
||||
@@ -122,6 +143,29 @@ class ProjectService
|
||||
return ['updated' => $updated, 'skipped' => [], 'warnings' => []];
|
||||
}
|
||||
|
||||
private function bulkDelete($query): array
|
||||
{
|
||||
$projects = (clone $query)->get(['id']);
|
||||
$deleted = 0;
|
||||
$skipped = [];
|
||||
|
||||
foreach ($projects as $p) {
|
||||
$model = Project::find($p->id);
|
||||
if ($model === null) {
|
||||
continue;
|
||||
}
|
||||
|
||||
try {
|
||||
$this->delete($model);
|
||||
$deleted++;
|
||||
} catch (HttpResponseException) {
|
||||
$skipped[] = ['id' => $p->id, 'reason' => 'has_deals'];
|
||||
}
|
||||
}
|
||||
|
||||
return ['updated' => $deleted, 'skipped' => $skipped, 'warnings' => []];
|
||||
}
|
||||
|
||||
/**
|
||||
* Plan 6.5: субъект-уровневый bulk-edit `regions` INT[].
|
||||
*
|
||||
@@ -213,10 +257,60 @@ class ProjectService
|
||||
return ['updated' => $updated, 'skipped' => $skipped, 'warnings' => []];
|
||||
}
|
||||
|
||||
private function assertNameUnique(int $tenantId, string $name, ?int $exceptId = null): void
|
||||
{
|
||||
$q = Project::where('tenant_id', $tenantId)->where('name', $name);
|
||||
if ($exceptId !== null) {
|
||||
$q->where('id', '!=', $exceptId);
|
||||
}
|
||||
if ($q->exists()) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'errors' => ['name' => ['Проект с таким названием у вас уже есть. Выберите другое название.']],
|
||||
], 422));
|
||||
}
|
||||
}
|
||||
|
||||
/** @param array<string,mixed> $data */
|
||||
private function assertSourceUnique(int $tenantId, array $data, ?int $exceptId = null): void
|
||||
{
|
||||
$signalType = $data['signal_type'] ?? null;
|
||||
$q = Project::where('tenant_id', $tenantId)->where('signal_type', $signalType);
|
||||
if ($exceptId !== null) {
|
||||
$q->where('id', '!=', $exceptId);
|
||||
}
|
||||
|
||||
if (in_array($signalType, ['call', 'site'], true)) {
|
||||
$identifier = (string) ($data['signal_identifier'] ?? '');
|
||||
if ($identifier === '') {
|
||||
return;
|
||||
}
|
||||
$q->where('signal_identifier', $identifier);
|
||||
} elseif ($signalType === 'sms') {
|
||||
$senders = (array) ($data['sms_senders'] ?? []);
|
||||
$norm = collect($senders)->map(fn ($s) => mb_strtolower(trim((string) $s)))->sort()->values()->all();
|
||||
if ($norm === []) {
|
||||
return;
|
||||
}
|
||||
$keyword = $data['sms_keyword'] ?? null;
|
||||
$q->where('sms_keyword', $keyword)
|
||||
->whereJsonContains('sms_senders', $norm)
|
||||
->whereRaw('jsonb_array_length(sms_senders::jsonb) = ?', [count($norm)]);
|
||||
} else {
|
||||
return;
|
||||
}
|
||||
|
||||
$existing = $q->first();
|
||||
if ($existing !== null) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'errors' => ['signal_identifier' => ["У вас уже есть проект с этим источником: «{$existing->name}»."]],
|
||||
], 422));
|
||||
}
|
||||
}
|
||||
|
||||
public function create(Tenant $tenant, array $data): Project
|
||||
{
|
||||
$limit = (int) ($tenant->limits['max_projects'] ?? 10);
|
||||
$current = Project::where('tenant_id', $tenant->id)->active()->count();
|
||||
$current = Project::where('tenant_id', $tenant->id)->count();
|
||||
if ($current >= $limit) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'message' => "Достигнут лимит проектов ({$limit}). Смените тариф.",
|
||||
@@ -230,6 +324,10 @@ class ProjectService
|
||||
// PhonePrefixService / LeadRouter, удаляются в Plan 6.5 после переключения читателей.
|
||||
$data['region_mask'] = 255;
|
||||
$data['region_mode'] = 'include';
|
||||
|
||||
$this->assertNameUnique($tenant->id, (string) $data['name']);
|
||||
$this->assertSourceUnique($tenant->id, $data);
|
||||
|
||||
$project = Project::create($data);
|
||||
|
||||
SyncSupplierProjectJob::dispatch($project->id);
|
||||
|
||||
+17
-1
@@ -2,9 +2,12 @@
|
||||
|
||||
use App\Http\Middleware\EnsureSaasAdmin;
|
||||
use App\Http\Middleware\SetTenantContext;
|
||||
use Illuminate\Database\QueryException;
|
||||
use Illuminate\Foundation\Application;
|
||||
use Illuminate\Foundation\Configuration\Exceptions;
|
||||
use Illuminate\Foundation\Configuration\Middleware;
|
||||
use Illuminate\Http\Request;
|
||||
use Illuminate\Support\Facades\Log;
|
||||
|
||||
return Application::configure(basePath: dirname(__DIR__))
|
||||
->withRouting(
|
||||
@@ -30,5 +33,18 @@ return Application::configure(basePath: dirname(__DIR__))
|
||||
]);
|
||||
})
|
||||
->withExceptions(function (Exceptions $exceptions): void {
|
||||
//
|
||||
$exceptions->render(function (QueryException $e, Request $request) {
|
||||
Log::error('db.query_exception', [
|
||||
'message' => $e->getMessage(),
|
||||
'sql' => $e->getSql(),
|
||||
'path' => $request->path(),
|
||||
]);
|
||||
if ($request->expectsJson()) {
|
||||
return response()->json([
|
||||
'message' => 'Не удалось сохранить. Проверьте данные или попробуйте ещё раз.',
|
||||
], 422);
|
||||
}
|
||||
|
||||
return null; // default render for non-JSON
|
||||
});
|
||||
})->create();
|
||||
|
||||
@@ -0,0 +1,19 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use Illuminate\Database\Migrations\Migration;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
|
||||
return new class extends Migration
|
||||
{
|
||||
public function up(): void
|
||||
{
|
||||
DB::statement('ALTER TABLE projects DROP COLUMN IF EXISTS archived_at');
|
||||
}
|
||||
|
||||
public function down(): void
|
||||
{
|
||||
DB::statement('ALTER TABLE projects ADD COLUMN archived_at TIMESTAMPTZ NULL');
|
||||
}
|
||||
};
|
||||
@@ -258,6 +258,90 @@ parameters:
|
||||
count: 1
|
||||
path: app/Services/Supplier/SupplierProjectGrouping.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Insights\\ForbiddenDefineFunctions not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Insights\\ForbiddenFinalClasses not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Insights\\ForbiddenNormalClasses not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Insights\\ForbiddenPrivateMethods not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Insights\\ForbiddenTraits not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Insights\\SyntaxCheck not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class NunoMaduro\\PhpInsights\\Domain\\Metrics\\Architecture\\Classes not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\Commenting\\UselessFunctionDocCommentSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\Namespaces\\AlphabeticallySortedUsesSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\TypeHints\\DeclareStrictTypesSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\TypeHints\\DisallowMixedTypeHintSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\TypeHints\\ParameterTypeHintSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\TypeHints\\PropertyTypeHintSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Class SlevomatCodingStandard\\Sniffs\\TypeHints\\ReturnTypeHintSniff not found\.$#'
|
||||
identifier: class.notFound
|
||||
count: 1
|
||||
path: config/insights.php
|
||||
|
||||
-
|
||||
message: '#^Return type \(array\<string, mixed\>\) of method Database\\Factories\\BalanceTransactionFactory\:\:definition\(\) should be compatible with return type \(array\<model property of App\\Models\\BalanceTransaction, mixed\>\) of method Illuminate\\Database\\Eloquent\\Factories\\Factory\<App\\Models\\BalanceTransaction\>\:\:definition\(\)$#'
|
||||
identifier: method.childReturnType
|
||||
@@ -1572,6 +1656,12 @@ parameters:
|
||||
count: 14
|
||||
path: tests/Feature/Plan5/Projects/ProjectsUpdateTest.php
|
||||
|
||||
-
|
||||
message: '#^Call to an undefined method Pest\\PendingCalls\\TestCall\:\:getJson\(\)\.$#'
|
||||
identifier: method.notFound
|
||||
count: 1
|
||||
path: tests/Feature/Project/QueryExceptionRenderTest.php
|
||||
|
||||
-
|
||||
message: '#^Access to an undefined property Pest\\PendingCalls\\TestCall\:\:\$tenant\.$#'
|
||||
identifier: property.notFound
|
||||
@@ -1848,6 +1938,12 @@ parameters:
|
||||
count: 2
|
||||
path: tests/Feature/Supplier/FailoverProjectChannelLiveSmokeTest.php
|
||||
|
||||
-
|
||||
message: '#^Call to an undefined method Mockery\\ExpectationInterface\|Mockery\\HigherOrderMessage\:\:once\(\)\.$#'
|
||||
identifier: method.notFound
|
||||
count: 1
|
||||
path: tests/Feature/Supplier/DeleteSupplierProjectJobTest.php
|
||||
|
||||
-
|
||||
message: '#^Call to an undefined method Pest\\PendingCalls\\TestCall\:\:artisan\(\)\.$#'
|
||||
identifier: method.notFound
|
||||
@@ -2015,3 +2111,21 @@ parameters:
|
||||
identifier: argument.type
|
||||
count: 1
|
||||
path: tests/Unit/Supplier/SupplierQuotaAllocatorTest.php
|
||||
|
||||
-
|
||||
message: '#^Access to an undefined property Pest\\PendingCalls\\TestCall\:\:\$tenant\.$#'
|
||||
identifier: property.notFound
|
||||
count: 6
|
||||
path: tests/Feature/Project/ProjectCreateDedupTest.php
|
||||
|
||||
-
|
||||
message: '#^Call to an undefined method Pest\\PendingCalls\\TestCall\:\:fail\(\)\.$#'
|
||||
identifier: method.notFound
|
||||
count: 1
|
||||
path: tests/Feature/Project/ProjectCreateDedupTest.php
|
||||
|
||||
-
|
||||
message: '#^Call to an undefined method Symfony\\Component\\HttpFoundation\\Response\:\:getData\(\)\.$#'
|
||||
identifier: method.notFound
|
||||
count: 1
|
||||
path: tests/Feature/Project/ProjectCreateDedupTest.php
|
||||
|
||||
@@ -29,11 +29,11 @@
|
||||
|
||||
<v-btn
|
||||
color="error"
|
||||
prepend-icon="mdi-archive"
|
||||
data-testid="bulk-archive"
|
||||
@click="confirmAndRun('archive')"
|
||||
prepend-icon="mdi-delete"
|
||||
data-testid="bulk-delete"
|
||||
@click="confirmAndRun('delete')"
|
||||
>
|
||||
Архивировать
|
||||
Удалить
|
||||
</v-btn>
|
||||
|
||||
<v-spacer />
|
||||
@@ -92,11 +92,10 @@ const skipToastText = ref('');
|
||||
const messages: Record<string, string> = {
|
||||
pause: 'Приостановить выбранные проекты?',
|
||||
resume: 'Возобновить выбранные проекты?',
|
||||
archive:
|
||||
'Архивировать выбранные проекты?\nДействие необратимо в Plan 5 (восстановление потребует ручного запроса).',
|
||||
delete: 'Удалить выбранные проекты? Действие необратимо. Проекты со сделками будут пропущены.',
|
||||
};
|
||||
|
||||
async function confirmAndRun(action: 'pause' | 'resume' | 'archive') {
|
||||
async function confirmAndRun(action: 'pause' | 'resume' | 'delete') {
|
||||
if (!window.confirm(messages[action])) return;
|
||||
await runBulk({ action });
|
||||
}
|
||||
|
||||
@@ -9,7 +9,6 @@ const base = {
|
||||
daily_limit_target: 50,
|
||||
delivered_today: 32,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
};
|
||||
|
||||
|
||||
@@ -48,9 +48,9 @@
|
||||
<template #prepend><v-icon>mdi-refresh</v-icon></template>
|
||||
<v-list-item-title>Синхронизировать</v-list-item-title>
|
||||
</v-list-item>
|
||||
<v-list-item @click="$emit('archive', project)">
|
||||
<template #prepend><v-icon>mdi-archive</v-icon></template>
|
||||
<v-list-item-title>Архивировать</v-list-item-title>
|
||||
<v-list-item @click="$emit('delete', project)">
|
||||
<template #prepend><v-icon>mdi-delete</v-icon></template>
|
||||
<v-list-item-title>Удалить</v-list-item-title>
|
||||
</v-list-item>
|
||||
</v-list>
|
||||
</v-menu>
|
||||
@@ -97,7 +97,7 @@ defineEmits<{
|
||||
edit: [project: Project];
|
||||
'toggle-active': [project: Project];
|
||||
'sync-now': [project: Project];
|
||||
archive: [project: Project];
|
||||
delete: [project: Project];
|
||||
}>();
|
||||
|
||||
const typeLabel = computed(() => ({ site: 'Сайт', call: 'Звонок', sms: 'СМС' })[props.project.signal_type]);
|
||||
|
||||
@@ -63,10 +63,10 @@ async function onPause(): Promise<void> {
|
||||
async function onDelete(): Promise<void> {
|
||||
if (!props.project) return;
|
||||
const ok = window.confirm(
|
||||
'Архивировать проект? Действие необратимо в Plan 5 (восстановление потребует ручного запроса).',
|
||||
'Удалить проект? Действие необратимо. Если по проекту есть сделки — удаление будет заблокировано.',
|
||||
);
|
||||
if (!ok) return;
|
||||
await store.archive(props.project.id);
|
||||
await store.del(props.project.id);
|
||||
emit('close');
|
||||
}
|
||||
|
||||
|
||||
@@ -168,6 +168,7 @@ const lucideMap: Record<string, Component> = {
|
||||
'mdi-content-save-outline': Save,
|
||||
'mdi-credit-card-outline': CreditCard,
|
||||
'mdi-currency-rub': RussianRuble,
|
||||
'mdi-delete': Trash2,
|
||||
'mdi-delete-outline': Trash2,
|
||||
'mdi-dots-vertical': MoreVertical,
|
||||
'mdi-download': Download,
|
||||
|
||||
@@ -13,7 +13,6 @@ export interface Project {
|
||||
delivered_today: number;
|
||||
delivered_in_month?: number;
|
||||
is_active: boolean;
|
||||
archived_at: string | null;
|
||||
region_mask?: number;
|
||||
region_mode?: string;
|
||||
regions?: number[]; // Plan 6 — subject codes 1..89; пустой массив = вся РФ
|
||||
@@ -65,7 +64,7 @@ export const useProjectsStore = defineStore('projects', () => {
|
||||
return data.data;
|
||||
}
|
||||
|
||||
async function archive(id: number) {
|
||||
async function del(id: number) {
|
||||
await axios.delete(`/api/projects/${id}`);
|
||||
await fetch();
|
||||
}
|
||||
@@ -94,7 +93,7 @@ export const useProjectsStore = defineStore('projects', () => {
|
||||
selectedIds.value.clear();
|
||||
}
|
||||
|
||||
async function bulkAction(action: 'pause' | 'resume' | 'archive') {
|
||||
async function bulkAction(action: 'pause' | 'resume' | 'delete') {
|
||||
const ids = Array.from(selectedIds.value);
|
||||
if (!ids.length) return;
|
||||
await axios.post('/api/projects/bulk', { action, ids });
|
||||
@@ -103,7 +102,7 @@ export const useProjectsStore = defineStore('projects', () => {
|
||||
}
|
||||
|
||||
interface BulkPayload {
|
||||
action: 'pause' | 'resume' | 'archive' | 'update_regions' | 'update_days' | 'update_limit';
|
||||
action: 'pause' | 'resume' | 'delete' | 'update_regions' | 'update_days' | 'update_limit';
|
||||
add?: number;
|
||||
remove?: number;
|
||||
// Plan 6.5 — update_regions оперирует кодами субъектов (1..89), не bitmask ФО.
|
||||
@@ -200,7 +199,7 @@ export const useProjectsStore = defineStore('projects', () => {
|
||||
fetch,
|
||||
create,
|
||||
update,
|
||||
archive,
|
||||
del,
|
||||
syncNow,
|
||||
toggleActive,
|
||||
toggleSelect,
|
||||
|
||||
@@ -71,7 +71,7 @@
|
||||
@edit="openEdit"
|
||||
@toggle-active="store.toggleActive"
|
||||
@sync-now="(p: Project) => store.syncNow(p.id)"
|
||||
@archive="(p: Project) => store.archive(p.id)"
|
||||
@delete="(p: Project) => store.del(p.id)"
|
||||
/>
|
||||
</div>
|
||||
|
||||
@@ -123,7 +123,6 @@ const typeFilters = [
|
||||
const statusFilters = [
|
||||
{ title: 'Активные', value: 'active' },
|
||||
{ title: 'На паузе', value: 'paused' },
|
||||
{ title: 'Архивные', value: 'archived' },
|
||||
];
|
||||
|
||||
let searchTimer: ReturnType<typeof setTimeout> | null = null;
|
||||
|
||||
@@ -25,7 +25,6 @@ const sampleProject = {
|
||||
daily_limit_target: 50,
|
||||
delivered_today: 12,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
region_mask: 0,
|
||||
region_mode: 'include' as const,
|
||||
|
||||
@@ -94,12 +94,11 @@ it('conversion = доля статуса won в окне', function () {
|
||||
->assertJsonPath('conversion.value', 25);
|
||||
});
|
||||
|
||||
it('active_projects считает archived_at IS NULL AND is_active=true + limit из limits', function () {
|
||||
it('active_projects считает is_active=true + limit из limits', function () {
|
||||
$tenant = Tenant::factory()->create(['limits' => ['max_projects' => 10]]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => null, 'is_active' => true]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => null, 'is_active' => true]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => now(), 'is_active' => true]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => null, 'is_active' => false]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => true]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => true]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => false]);
|
||||
$this->getJson("/api/dashboard/summary?tenant_id={$tenant->id}")
|
||||
->assertOk()
|
||||
->assertJsonPath('active_projects.active', 2)
|
||||
|
||||
@@ -6,28 +6,34 @@ use App\Jobs\SyncSupplierProjectJob;
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
use App\Models\User;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
use Illuminate\Support\Facades\Queue;
|
||||
|
||||
beforeEach(fn () => Queue::fake());
|
||||
|
||||
it('destroy archives project (sets archived_at, is_active=false)', function () {
|
||||
it('destroy hard-deletes a project with no deals', function () {
|
||||
$tenant = Tenant::factory()->create();
|
||||
$user = User::factory()->create(['tenant_id' => $tenant->id]);
|
||||
$project = Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => true]);
|
||||
|
||||
$this->actingAs($user)->deleteJson("/api/projects/{$project->id}")->assertNoContent();
|
||||
|
||||
$project->refresh();
|
||||
expect($project->is_active)->toBeFalse();
|
||||
expect($project->archived_at)->not->toBeNull();
|
||||
expect(Project::find($project->id))->toBeNull();
|
||||
});
|
||||
|
||||
it('destroy returns 409 if already archived', function () {
|
||||
it('destroy returns 422 if project has deals', function () {
|
||||
$tenant = Tenant::factory()->create();
|
||||
$user = User::factory()->create(['tenant_id' => $tenant->id]);
|
||||
$project = Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => now()]);
|
||||
$project = Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => true]);
|
||||
DB::table('deals')->insert([
|
||||
'tenant_id' => $tenant->id, 'project_id' => $project->id,
|
||||
'phone' => '79990001100', 'status' => 'new',
|
||||
'received_at' => now(), 'created_at' => now(),
|
||||
]);
|
||||
|
||||
$this->actingAs($user)->deleteJson("/api/projects/{$project->id}")->assertStatus(409);
|
||||
$this->actingAs($user)->deleteJson("/api/projects/{$project->id}")->assertStatus(422);
|
||||
|
||||
expect(Project::find($project->id))->not->toBeNull();
|
||||
});
|
||||
|
||||
it('sync re-dispatches SyncSupplierProjectJob', function () {
|
||||
@@ -81,16 +87,16 @@ it('bulk filters out cross-tenant ids silently', function () {
|
||||
expect($pB->fresh()->is_active)->toBeTrue();
|
||||
});
|
||||
|
||||
it('bulk archive sets archived_at on multiple', function () {
|
||||
it('bulk delete removes project with no deals', function () {
|
||||
$tenant = Tenant::factory()->create();
|
||||
$user = User::factory()->create(['tenant_id' => $tenant->id]);
|
||||
$p1 = Project::factory()->create(['tenant_id' => $tenant->id]);
|
||||
|
||||
$this->actingAs($user)->postJson('/api/projects/bulk', [
|
||||
'action' => 'archive', 'ids' => [$p1->id],
|
||||
])->assertOk();
|
||||
'action' => 'delete', 'ids' => [$p1->id],
|
||||
])->assertOk()->assertJsonPath('updated', 1);
|
||||
|
||||
expect($p1->fresh()->archived_at)->not->toBeNull();
|
||||
expect(Project::find($p1->id))->toBeNull();
|
||||
});
|
||||
|
||||
it('bulk rejects > 500 ids', function () {
|
||||
|
||||
@@ -16,7 +16,7 @@ it('returns paginated list of active projects for current tenant', function () {
|
||||
$response->assertOk();
|
||||
$response->assertJsonStructure([
|
||||
'data' => [['id', 'name', 'signal_type', 'signal_identifier', 'daily_limit_target',
|
||||
'delivered_today', 'is_active', 'archived_at', 'sync_status']],
|
||||
'delivered_today', 'is_active', 'sync_status']],
|
||||
'meta' => ['current_page', 'per_page', 'total'],
|
||||
]);
|
||||
expect($response->json('meta.total'))->toBe(3);
|
||||
@@ -45,23 +45,24 @@ it('isolates projects per tenant (RLS)', function () {
|
||||
expect($response->json('meta.total'))->toBe(2);
|
||||
});
|
||||
|
||||
it('excludes archived projects by default', function () {
|
||||
it('returns all projects by default (archive feature removed in v8.27)', function () {
|
||||
$tenant = Tenant::factory()->create();
|
||||
$user = User::factory()->create(['tenant_id' => $tenant->id]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => now()]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id]);
|
||||
|
||||
$response = $this->actingAs($user)->getJson('/api/projects');
|
||||
|
||||
expect($response->json('meta.total'))->toBe(1);
|
||||
expect($response->json('meta.total'))->toBe(2);
|
||||
});
|
||||
|
||||
it('returns archived when status=archived requested', function () {
|
||||
it('status=active returns only is_active=true projects', function () {
|
||||
$tenant = Tenant::factory()->create();
|
||||
$user = User::factory()->create(['tenant_id' => $tenant->id]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'archived_at' => now()]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => true]);
|
||||
Project::factory()->create(['tenant_id' => $tenant->id, 'is_active' => false]);
|
||||
|
||||
$response = $this->actingAs($user)->getJson('/api/projects?status=archived');
|
||||
$response = $this->actingAs($user)->getJson('/api/projects?status=active');
|
||||
|
||||
expect($response->json('meta.total'))->toBe(1);
|
||||
});
|
||||
@@ -140,19 +141,18 @@ it('search is case-insensitive for Cyrillic substrings', function () {
|
||||
expect($partial->json('meta.total'))->toBe(1);
|
||||
});
|
||||
|
||||
it('show returns 200 for archived project (read access preserved)', function () {
|
||||
it('show returns 200 for any project by id', function () {
|
||||
$tenant = Tenant::factory()->create();
|
||||
$user = User::factory()->create(['tenant_id' => $tenant->id]);
|
||||
$project = Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'archived_at' => now(),
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'archived.ru',
|
||||
'signal_identifier' => 'myproject.ru',
|
||||
]);
|
||||
|
||||
$response = $this->actingAs($user)->getJson("/api/projects/{$project->id}");
|
||||
|
||||
$response->assertOk();
|
||||
expect($response->json('data.id'))->toBe($project->id);
|
||||
expect($response->json('data.archived_at'))->not->toBeNull();
|
||||
expect($response->json('data'))->not->toHaveKey('archived_at');
|
||||
});
|
||||
|
||||
@@ -2,7 +2,6 @@
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Project;
|
||||
use Illuminate\Foundation\Testing\DatabaseTransactions;
|
||||
use Illuminate\Support\Facades\Schema;
|
||||
|
||||
@@ -11,15 +10,6 @@ use Illuminate\Support\Facades\Schema;
|
||||
// DatabaseTransactions — изоляция; also ensures DB connection is bootstrapped.
|
||||
uses(DatabaseTransactions::class);
|
||||
|
||||
it('projects table has archived_at column nullable timestamp', function () {
|
||||
expect(Schema::hasColumn('projects', 'archived_at'))->toBeTrue();
|
||||
$type = Schema::getColumnType('projects', 'archived_at');
|
||||
// PostgreSQL TIMESTAMPTZ → Doctrine/Laravel reports 'timestamptz' (not 'timestamp').
|
||||
expect($type)->toBe('timestamptz');
|
||||
});
|
||||
|
||||
it('Project model has archived_at in fillable and casts it to datetime', function () {
|
||||
$project = new Project;
|
||||
expect(in_array('archived_at', $project->getFillable(), true))->toBeTrue();
|
||||
expect($project->getCasts()['archived_at'] ?? null)->toBe('datetime');
|
||||
it('projects table does NOT have archived_at column (feature removed in v8.27)', function () {
|
||||
expect(Schema::hasColumn('projects', 'archived_at'))->toBeFalse();
|
||||
});
|
||||
|
||||
@@ -0,0 +1,48 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
use App\Services\Project\ProjectService;
|
||||
use Illuminate\Http\Exceptions\HttpResponseException;
|
||||
use Illuminate\Support\Facades\Queue;
|
||||
|
||||
beforeEach(function () {
|
||||
Queue::fake();
|
||||
$this->tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
});
|
||||
|
||||
function makeCall(array $over = []): array
|
||||
{
|
||||
return array_merge([
|
||||
'name' => 'Проект A', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
||||
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
||||
], $over);
|
||||
}
|
||||
|
||||
it('blocks duplicate source within tenant with human message', function () {
|
||||
app(ProjectService::class)->create($this->tenant, makeCall());
|
||||
expect(fn () => app(ProjectService::class)
|
||||
->create($this->tenant, makeCall(['name' => 'Проект B'])))
|
||||
->toThrow(HttpResponseException::class);
|
||||
});
|
||||
|
||||
it('allows same source for a different tenant (sharing)', function () {
|
||||
$other = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
app(ProjectService::class)->create($this->tenant, makeCall());
|
||||
$p = app(ProjectService::class)->create($other, makeCall(['name' => 'Проект B']));
|
||||
expect($p)->toBeInstanceOf(Project::class);
|
||||
});
|
||||
|
||||
it('blocks duplicate name within tenant with human message (not SQL)', function () {
|
||||
app(ProjectService::class)->create($this->tenant, makeCall());
|
||||
try {
|
||||
app(ProjectService::class)
|
||||
->create($this->tenant, makeCall(['name' => 'Проект A', 'signal_identifier' => '79992220000']));
|
||||
$this->fail('expected HttpResponseException');
|
||||
} catch (HttpResponseException $e) {
|
||||
$body = $e->getResponse()->getData(true);
|
||||
expect($body['errors']['name'][0] ?? '')->not->toContain('SQLSTATE');
|
||||
}
|
||||
});
|
||||
@@ -0,0 +1,40 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
use App\Services\Project\ProjectService;
|
||||
use Illuminate\Http\Exceptions\HttpResponseException;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
use Illuminate\Support\Facades\Queue;
|
||||
|
||||
beforeEach(fn () => Queue::fake());
|
||||
|
||||
it('hard-deletes an empty project', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$project = app(ProjectService::class)->create($tenant, [
|
||||
'name' => 'Empty', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
||||
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
||||
]);
|
||||
|
||||
app(ProjectService::class)->delete($project);
|
||||
|
||||
expect(Project::find($project->id))->toBeNull();
|
||||
});
|
||||
|
||||
it('blocks delete when project has deals', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$project = app(ProjectService::class)->create($tenant, [
|
||||
'name' => 'WithDeals', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
||||
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
||||
]);
|
||||
DB::table('deals')->insert([
|
||||
'tenant_id' => $tenant->id, 'project_id' => $project->id, 'phone' => '79990001122',
|
||||
'status' => 'new', 'received_at' => now(), 'created_at' => now(),
|
||||
]);
|
||||
|
||||
expect(fn () => app(ProjectService::class)->delete($project))
|
||||
->toThrow(HttpResponseException::class);
|
||||
expect(Project::find($project->id))->not->toBeNull();
|
||||
});
|
||||
@@ -0,0 +1,28 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Tenant;
|
||||
use App\Services\Project\ProjectService;
|
||||
use Illuminate\Http\Exceptions\HttpResponseException;
|
||||
use Illuminate\Support\Facades\Queue;
|
||||
|
||||
beforeEach(fn () => Queue::fake());
|
||||
|
||||
it('blocks update that collides source with another project of same tenant', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$svc = app(ProjectService::class);
|
||||
$a = $svc->create($tenant, ['name' => 'A', 'signal_type' => 'call', 'signal_identifier' => '79991110000', 'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31]);
|
||||
$b = $svc->create($tenant, ['name' => 'B', 'signal_type' => 'call', 'signal_identifier' => '79992220000', 'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31]);
|
||||
|
||||
expect(fn () => $svc->update($b, ['signal_identifier' => '79991110000']))
|
||||
->toThrow(HttpResponseException::class);
|
||||
});
|
||||
|
||||
it('allows update keeping same source on the same project', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$svc = app(ProjectService::class);
|
||||
$a = $svc->create($tenant, ['name' => 'A', 'signal_type' => 'call', 'signal_identifier' => '79991110000', 'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31]);
|
||||
$updated = $svc->update($a, ['signal_identifier' => '79991110000', 'daily_limit_target' => 7]);
|
||||
expect($updated->daily_limit_target)->toBe(7);
|
||||
});
|
||||
@@ -0,0 +1,17 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use Illuminate\Database\QueryException;
|
||||
use Illuminate\Support\Facades\Route;
|
||||
|
||||
it('renders QueryException as human JSON message, not SQLSTATE', function () {
|
||||
Route::get('/_test/boom-query', function () {
|
||||
throw new QueryException('pgsql', 'SELECT 1', [], new Exception('SQLSTATE[23505] duplicate key'));
|
||||
});
|
||||
|
||||
$res = $this->getJson('/_test/boom-query');
|
||||
$res->assertStatus(422);
|
||||
expect($res->json('message'))->not->toContain('SQLSTATE');
|
||||
expect($res->json('message'))->toContain('Не удалось');
|
||||
});
|
||||
@@ -0,0 +1,58 @@
|
||||
<?php
|
||||
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Jobs\Supplier\DeleteSupplierProjectJob;
|
||||
use App\Jobs\Supplier\SyncSupplierProjectsJob;
|
||||
use App\Models\Project;
|
||||
use App\Models\SupplierProject;
|
||||
use App\Models\Tenant;
|
||||
use App\Services\Supplier\SupplierPortalClient;
|
||||
use Illuminate\Foundation\Testing\DatabaseTransactions;
|
||||
use Illuminate\Support\Facades\Bus;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
use Tests\Concerns\SharesSupplierPdo;
|
||||
|
||||
uses(DatabaseTransactions::class, SharesSupplierPdo::class);
|
||||
|
||||
it('deletes donor at supplier when no consumers remain', function (): void {
|
||||
$sp = SupplierProject::query()->create([
|
||||
'platform' => 'B1', 'signal_type' => 'call', 'unique_key' => '79991110000',
|
||||
'supplier_external_id' => '555', 'current_limit' => 1,
|
||||
]);
|
||||
|
||||
$mock = Mockery::mock(SupplierPortalClient::class);
|
||||
$mock->shouldReceive('deleteProject')->once()->with(555);
|
||||
app()->instance(SupplierPortalClient::class, $mock);
|
||||
|
||||
(new DeleteSupplierProjectJob([$sp->id]))->handle(app(SupplierPortalClient::class));
|
||||
|
||||
expect(SupplierProject::find($sp->id))->toBeNull();
|
||||
});
|
||||
|
||||
it('does NOT delete donor at supplier when other consumers remain; re-syncs', function (): void {
|
||||
Bus::fake([SyncSupplierProjectsJob::class]);
|
||||
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$sp = SupplierProject::query()->create([
|
||||
'platform' => 'B1', 'signal_type' => 'call', 'unique_key' => '79991110001',
|
||||
'supplier_external_id' => '556', 'current_limit' => 1,
|
||||
]);
|
||||
$other = Project::factory()->create(['tenant_id' => $tenant->id]);
|
||||
|
||||
DB::table('project_supplier_links')->insert([
|
||||
'project_id' => $other->id,
|
||||
'supplier_project_id' => $sp->id,
|
||||
'platform' => 'B1',
|
||||
'subject_code' => null,
|
||||
]);
|
||||
|
||||
$mock = Mockery::mock(SupplierPortalClient::class);
|
||||
$mock->shouldNotReceive('deleteProject');
|
||||
app()->instance(SupplierPortalClient::class, $mock);
|
||||
|
||||
(new DeleteSupplierProjectJob([$sp->id]))->handle(app(SupplierPortalClient::class));
|
||||
|
||||
expect(SupplierProject::find($sp->id))->not->toBeNull();
|
||||
Bus::assertDispatched(SyncSupplierProjectsJob::class);
|
||||
});
|
||||
@@ -57,7 +57,6 @@ test('single-group: regions=[82,83] site → merged regions tag=РФ → 3 suppl
|
||||
$project = Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'persubject.example.com',
|
||||
'daily_limit_target' => 9,
|
||||
@@ -116,7 +115,6 @@ test('all-RF pool: regions=[] → 1 group subject_code=null tag=РФ → 3 suppl
|
||||
$project = Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'rf-pool.example.com',
|
||||
'daily_limit_target' => 6,
|
||||
@@ -167,7 +165,6 @@ test('order: 2 projects same source×subject → computeOrder(limits=[10,20])
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'order-test.example.com',
|
||||
'daily_limit_target' => 10,
|
||||
@@ -178,7 +175,6 @@ test('order: 2 projects same source×subject → computeOrder(limits=[10,20])
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'order-test.example.com',
|
||||
'daily_limit_target' => 20,
|
||||
@@ -229,7 +225,6 @@ test('sms+keyword → platforms B2+B3 (2 supplier_projects per subject)', functi
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'sms',
|
||||
'signal_identifier' => null,
|
||||
'sms_senders' => ['79001234567'],
|
||||
@@ -271,7 +266,6 @@ test('sms without keyword → platform B3 only (1 supplier_project)', function (
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'sms',
|
||||
'signal_identifier' => null,
|
||||
'sms_senders' => ['79009876543'],
|
||||
@@ -314,7 +308,6 @@ test('idempotent: repeat run with no changes → updateProject not duplicate', f
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'idempotent.example.com',
|
||||
'daily_limit_target' => 9,
|
||||
@@ -375,7 +368,6 @@ test('respects time budget by stopping at 20:55 МСК', function (): void {
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'time-budget.example.com',
|
||||
'daily_limit_target' => 9,
|
||||
@@ -397,7 +389,6 @@ test('sticky auth error throws and sends critical alert email', function (): voi
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'auth-fail.example.com',
|
||||
'daily_limit_target' => 9,
|
||||
@@ -425,7 +416,6 @@ test('aborts after 50 consecutive transient failures and sends alert', function
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => "host{$i}.abort.com",
|
||||
'daily_limit_target' => 9,
|
||||
@@ -449,7 +439,6 @@ test('writes supplier_sync_log row for each successful action', function (): voi
|
||||
Project::factory()->create([
|
||||
'tenant_id' => $tenant->id,
|
||||
'is_active' => true,
|
||||
'archived_at' => null,
|
||||
'signal_type' => 'site',
|
||||
'signal_identifier' => 'audit-log.example.com',
|
||||
'daily_limit_target' => 9,
|
||||
|
||||
@@ -169,7 +169,7 @@ describe('BulkActionsBar — extended', () => {
|
||||
expect((wrapper.vm as unknown as { regionsOpen: boolean }).regionsOpen).toBe(true);
|
||||
});
|
||||
|
||||
it('keeps existing pause/resume/archive buttons', async () => {
|
||||
it('keeps existing pause/resume/delete buttons', async () => {
|
||||
setActivePinia(createPinia());
|
||||
vi.mocked(axios.post).mockResolvedValue({ data: { updated: 1, skipped: [], warnings: [] } });
|
||||
vi.mocked(axios.get).mockResolvedValue({ data: { data: [], meta: { total: 0 } } });
|
||||
@@ -184,6 +184,6 @@ describe('BulkActionsBar — extended', () => {
|
||||
});
|
||||
expect(wrapper.find('[data-testid="bulk-pause"]').exists()).toBe(true);
|
||||
expect(wrapper.find('[data-testid="bulk-resume"]').exists()).toBe(true);
|
||||
expect(wrapper.find('[data-testid="bulk-archive"]').exists()).toBe(true);
|
||||
expect(wrapper.find('[data-testid="bulk-delete"]').exists()).toBe(true);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -23,7 +23,6 @@ const sampleProject = {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
region_mask: 0,
|
||||
region_mode: 'include',
|
||||
|
||||
@@ -12,7 +12,6 @@ const baseProject = {
|
||||
daily_limit_target: 50,
|
||||
delivered_today: 32,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
};
|
||||
|
||||
|
||||
@@ -11,7 +11,6 @@ const sample = {
|
||||
is_active: true,
|
||||
daily_limit_target: 50,
|
||||
delivered_today: 12,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
};
|
||||
|
||||
|
||||
@@ -21,7 +21,6 @@ const sampleProject: Project = {
|
||||
daily_limit_target: 30,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
region_mask: 0,
|
||||
region_mode: 'include',
|
||||
regions: [],
|
||||
@@ -152,10 +151,10 @@ describe('ProjectDetailsDrawer', () => {
|
||||
expect(wrapper.get('[data-testid="pdd-pause"]').text()).toContain('Приостановить');
|
||||
});
|
||||
|
||||
it('Delete: confirm=true → archive + close emit', async () => {
|
||||
it('Delete: confirm=true → del + close emit', async () => {
|
||||
const wrapper = mount(ProjectDetailsDrawer, { props: { project: sampleProject } });
|
||||
const store = useProjectsStore();
|
||||
const spy = vi.spyOn(store, 'archive').mockResolvedValueOnce(undefined);
|
||||
const spy = vi.spyOn(store, 'del').mockResolvedValueOnce(undefined);
|
||||
vi.stubGlobal('confirm', () => true);
|
||||
|
||||
await wrapper.get('[data-testid="pdd-delete"]').trigger('click');
|
||||
@@ -166,10 +165,10 @@ describe('ProjectDetailsDrawer', () => {
|
||||
vi.unstubAllGlobals();
|
||||
});
|
||||
|
||||
it('Delete: confirm=false → no archive, no close', async () => {
|
||||
it('Delete: confirm=false → no del, no close', async () => {
|
||||
const wrapper = mount(ProjectDetailsDrawer, { props: { project: sampleProject } });
|
||||
const store = useProjectsStore();
|
||||
const spy = vi.spyOn(store, 'archive').mockResolvedValueOnce(undefined);
|
||||
const spy = vi.spyOn(store, 'del').mockResolvedValueOnce(undefined);
|
||||
vi.stubGlobal('confirm', () => false);
|
||||
|
||||
await wrapper.get('[data-testid="pdd-delete"]').trigger('click');
|
||||
|
||||
@@ -52,7 +52,6 @@ describe('ProjectsView', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok',
|
||||
},
|
||||
],
|
||||
@@ -82,7 +81,6 @@ describe('ProjectsView', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok',
|
||||
},
|
||||
{
|
||||
@@ -93,7 +91,6 @@ describe('ProjectsView', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok',
|
||||
},
|
||||
],
|
||||
@@ -127,7 +124,6 @@ describe('ProjectsView × ProjectDetailsDrawer integration', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
};
|
||||
const projectB = {
|
||||
@@ -138,7 +134,6 @@ describe('ProjectsView × ProjectDetailsDrawer integration', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
sync_status: 'ok' as const,
|
||||
};
|
||||
|
||||
|
||||
@@ -20,7 +20,6 @@ const mountView = async () => {
|
||||
daily_limit_target: 100,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
region_mask: 1,
|
||||
region_mode: 'include',
|
||||
delivery_days_mask: 31,
|
||||
@@ -34,7 +33,6 @@ const mountView = async () => {
|
||||
daily_limit_target: 100,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
region_mask: 1,
|
||||
region_mode: 'include',
|
||||
delivery_days_mask: 31,
|
||||
|
||||
@@ -66,6 +66,14 @@ describe('projectsStore (no polling)', () => {
|
||||
expect(store.selectedIds.has(1)).toBe(false);
|
||||
});
|
||||
|
||||
it('del() calls DELETE /api/projects/{id}', async () => {
|
||||
const store = useProjectsStore();
|
||||
vi.spyOn(axios, 'delete').mockResolvedValue({ data: null });
|
||||
vi.spyOn(axios, 'get').mockResolvedValue({ data: { data: [], meta: { total: 0 } } });
|
||||
await store.del(7);
|
||||
expect(axios.delete).toHaveBeenCalledWith('/api/projects/7');
|
||||
});
|
||||
|
||||
it('bulkAction sends array of ids and clears selection', async () => {
|
||||
(axios.post as unknown as ReturnType<typeof vi.fn>).mockResolvedValue({ data: { updated: 2 } });
|
||||
(axios.get as unknown as ReturnType<typeof vi.fn>).mockResolvedValue({
|
||||
@@ -104,7 +112,6 @@ describe('projectsStore (polling)', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
region_mask: 0,
|
||||
region_mode: 'include',
|
||||
delivery_days_mask: 127,
|
||||
@@ -133,7 +140,6 @@ describe('projectsStore (polling)', () => {
|
||||
daily_limit_target: 10,
|
||||
delivered_today: 0,
|
||||
is_active: true,
|
||||
archived_at: null,
|
||||
region_mask: 0,
|
||||
region_mode: 'include',
|
||||
delivery_days_mask: 127,
|
||||
|
||||
@@ -1572,3 +1572,20 @@ lemed
|
||||
# Сквозной чек-лист портала + 6 фиксов (2026-05-21)
|
||||
захардкоженным
|
||||
смердженных
|
||||
|
||||
# Observer chain attribution L1-L13 (2026-05-20)
|
||||
инвокированный
|
||||
межэпизодные
|
||||
побочек
|
||||
диффы
|
||||
ретрофилл
|
||||
|
||||
# project delete / dedup / errors spec (2026-05-21)
|
||||
шеринг
|
||||
шеринга
|
||||
констрейнт
|
||||
дропается
|
||||
батч
|
||||
ретраит
|
||||
шеринге
|
||||
unactivated
|
||||
|
||||
@@ -6,6 +6,10 @@
|
||||
|
||||
**История записей:**
|
||||
|
||||
## v8.27 — 2026-05-21 — DROP COLUMN projects.archived_at
|
||||
|
||||
- DROP COLUMN `projects.archived_at` — фича «архив» полностью убрана и заменена настоящим удалением с защитой по сделкам (`ProjectService::delete()`). Миграция `2026_05_21_000000_drop_projects_archived_at.php`.
|
||||
|
||||
## v8.26 — 2026-05-20 — supplier_projects.subject_code (per-субъект экспорт)
|
||||
|
||||
`supplier_projects` +1 колонка `subject_code SMALLINT NULL` (1..89; NULL = пул «Вся РФ»),
|
||||
|
||||
+1
-2
@@ -1,6 +1,6 @@
|
||||
-- =============================================================================
|
||||
-- schema.sql — единая схема БД для SaaS-аналога crm.bp-gr.ru («Лидерра»)
|
||||
-- Версия: v8.26 (20.05.2026 — project-migration-redesign Plans 1-3: supplier_projects.subject_code (per-субъект экспорт) + project_supplier_links (M:N pivot projects↔supplier_projects) + deals.subject_code + CHECK chk_deals_subject_code + seed system_settings.supplier_export_mode)
|
||||
-- Версия: v8.27 (21.05.2026 — drop projects.archived_at: feature архива заменена настоящим удалением с защитой по сделкам (ProjectService::delete()))
|
||||
-- Метрики: 65 базовые таблицы (63 regular + 2 partitioned parents: deals + supplier_lead_costs) + 12 партиций / 123 индекса / 40 RLS-политик / 5 функций / 13 триггеров
|
||||
-- Базовая версия: v8.25 (19.05.2026 — supplier_manual_sync_queue: SaaS-level Tier 3 очередь резерва канала миграции проектов)
|
||||
-- Базовая версия: v8.24 (18.05.2026 — supplier_leads.vid → nullable для CSV-recovered лидов (Путь 2))
|
||||
@@ -840,7 +840,6 @@ CREATE TABLE projects (
|
||||
CHECK (ttfr_target_minutes BETWEEN 1 AND 1440),
|
||||
created_at TIMESTAMPTZ DEFAULT NOW(),
|
||||
updated_at TIMESTAMPTZ,
|
||||
archived_at TIMESTAMPTZ NULL, -- v8.20 (Plan 5): soft archive flow (отличие от is_active=false который = pause)
|
||||
UNIQUE (tenant_id, name),
|
||||
CONSTRAINT chk_projects_daily_limit_positive
|
||||
CHECK (daily_limit_target > 0),
|
||||
|
||||
@@ -1,10 +1,12 @@
|
||||
# Правила работы Claude в проекте «Лидерра»
|
||||
|
||||
**Версия:** v1.35 (20.05.2026)
|
||||
**Дата:** 20.05.2026
|
||||
**Версия:** v1.36 (21.05.2026)
|
||||
**Дата:** 21.05.2026
|
||||
**Назначение:** настройки проекта (Project instructions) — Claude читает этот файл в каждом чате и следует правилам ниже.
|
||||
**Статус документа:** ✅ утверждён. Содержимое скопировано в поле "Project instructions" Claude.ai. Файл хранится в архиве как служебный документ.
|
||||
|
||||
**Что изменилось в v1.36 относительно v1.35:** §16.4 расширен симметрией missed activation (условное правило): §16.4 заголовок уточнён «(условное)»; тело расширено — поведенческое правило теперь содержит условие «если профильной задачи в эпизодах не было»; добавлено **симметричное правило (missed activation)**: эпизоды с профильной классификацией без активации релевантного non-dormant узла — сигнал, surface в STATUS.md (C5: `missed_activations: N`, ⚠️ при N>0) и в выводе `/brain-retro`, не блок коммита; хранение mapping в `tools/observer-classification-map.json` + `tools/.node-dormancy.json` (двойной сигнал dormant=true ИЛИ DEFERRED в boundaries); DEFERRED-узлы (#17/#44/#50/#54/#67) — в missed activations не учитываются. Архитектурных изменений в §§1–15: 0. Связано: план `docs/superpowers/plans/2026-05-21-observer-missed-activations.md`.
|
||||
|
||||
**Что изменилось в v1.35 относительно v1.34:** A1 backend-tooling — §13.2 +абзац «Off-phase backend-tooling»: #64 Rector + rector-laravel (Composer dev-dep, авто-рефакторинг/version-upgrade, manual/CI — dry-run baseline 16 файлов, не блокирующий), #65 PHP Insights (Composer dev-dep, метрики complexity/architecture, on-demand/CI — не блокирующий), #66 laravel-backend-patterns (self-authored project-скил, backend-конвенции Лидерры), #67 NightOwl (self-hosted runtime-телеметрия — **DEFERRED**: native-Windows нет pcntl/posix, OSS без MCP, hosted 152-ФЗ). 16-я off-phase подкатегория, раздел A1. Не UI → вне R6.0/R6.1/R14. Границы — ADR-013. Архитектурных изменений §§1–12, §14–§16: 0. Связано: Tooling v2.19, PSR_v1 v3.19, CLAUDE.md v2.22; план `docs/superpowers/plans/2026-05-20-a1-backend-tooling.md`.
|
||||
|
||||
**Что изменилось в v1.34 относительно v1.33:** finance-tooling (C6+C7) — §13.2 +абзац «Off-phase finance-tooling»: #61 finance plugin (marketplace `finance@knowledge-work-plugins`, Anthropic Verified, homed C7, cross-ref C6; РФ-применимость частична — US-GAAP-скилы ⚠️, SOX-скилы not-applicable, warehouse-MCP DEFERRED), #62 billing-audit (self-authored project-скил, C6 — денежные инварианты биллинга), #63 ru-tax-accounting (self-authored project-скил, C7 — РСБУ/НК РФ). 15-я off-phase подкатегория. Не UI → вне R6.0/R6.1/R14. Границы — ADR-012. Архитектурных изменений §§1–12, §14–§16: 0. Связано: Tooling v2.18, PSR_v1 v3.18, CLAUDE.md v2.21; план `docs/superpowers/plans/2026-05-20-finance-tooling-c6-c7.md`.
|
||||
@@ -982,11 +984,15 @@ git fetch origin && git log HEAD..origin/main --oneline
|
||||
|
||||
Все 5 — механические, 0 LLM-вызовов в hot path.
|
||||
|
||||
### 16.4. Поведенческое правило «не использован ≠ проблема»
|
||||
### 16.4. Поведенческое правило «не использован ≠ проблема» (условное)
|
||||
|
||||
Узел «мозга», не задействованный на реальной задаче, **не** считается проблемой и **не** подлежит автоматической пометке. Это — capability-readiness, осознанная стратегия заказчика. См. `memory/feedback_brain_unused_tools_not_problem.md`.
|
||||
Узел «мозга», не задействованный в реальной работе, **не** считается проблемой и **не** подлежит автоматической пометке **при условии, что профильной задачи для него в эпизодах не было**. Это — capability-readiness, осознанная стратегия заказчика.
|
||||
|
||||
**Исключение**: deprecated upstream-пакеты или физически сломанные инструменты (отдельная категория, `npm audit` / `composer outdated`).
|
||||
**Симметричное правило (missed activation):** если в эпизодах присутствует **хотя бы один** эпизод с `primary_rationale.task_classification`, соответствующим набору рекомендуемых узлов из `tools/observer-classification-map.json`, при этом `primary_rationale.node_chosen === 'direct'` и среди рекомендуемых узлов есть хотя бы один non-dormant (по `tools/.node-dormancy.json`, экстракт из [Tooling Прил.Н §3.5/§4.X](Tooling_v8_3.md) с двойным сигналом: `dormant: true` ИЛИ ключевое слово `DEFERRED` в колонке boundaries) — это **сигнал**, кандидат на разбор. Surface в STATUS.md (C5: `missed_activations: N`, ⚠️ при N>0) и в выводе `/brain-retro`. Не блок коммита, не auto-edit.
|
||||
|
||||
**Исключения:** DEFERRED-узлы (на момент v1.36 — #17 pg_partman, #44 Figma MCP, #50 Jupyter MCP, #54 n8n-mcp, #67 NightOwl) — для них «не активирован» = ожидаемое состояние, в missed activations не учитываются.
|
||||
|
||||
См. `memory/feedback_brain_unused_tools_not_problem.md`.
|
||||
|
||||
### 16.5. Не override-floor §9
|
||||
|
||||
|
||||
File diff suppressed because one or more lines are too long
@@ -100,3 +100,36 @@ The observer episode is extended to `schema_version: 2` so a real factor analysi
|
||||
- Pravila §12 / §14 / §15 (hard-floor for router procedure step 1)
|
||||
- PSR_v1 R15 (off-phase routing extends to brain governance)
|
||||
- memory: `feedback_brain_unused_tools_not_problem.md`, `project_brain_governance_design.md`
|
||||
|
||||
## Amendment 2026-05-21: Conditional missed-activation rule (§16.4 v1.36)
|
||||
|
||||
The original §16.4 stated unconditionally that an unused node is not a problem. Real-world episodes show this is too permissive: when a profile-classified task (e.g. `refactor`) runs with `node_chosen === 'direct'` and a relevant non-dormant node exists in Tooling Прил.Н, the absence of activation IS a signal (router miss, not a problem in the node itself).
|
||||
|
||||
The rule now reads:
|
||||
|
||||
- **Unused + no profile task** → still not an alert (capability-readiness).
|
||||
- **Unused + profile task present** → "missed activation", surfaced in STATUS.md C5 and `/brain-retro`. Not a commit block.
|
||||
|
||||
**Mapping artefacts:**
|
||||
|
||||
- `tools/observer-classification-map.json` — manual mapping `classification → recommended_node_ids[]` (single source of truth). 10 classification buckets, populated from the real `tools/observer-transcript-parser.mjs` `classifyTask` dictionary (bugfix / cleanup / feature / memory-sync / monitoring / other / planning / question / refactor / analysis).
|
||||
- `tools/.node-dormancy.json` — generated from Прил.Н by `tools/extract-node-dormancy.mjs` (pre-commit job `extract-node-dormancy` in `lefthook.yml`). Uses a **two-signal** availability check: `dormant: true` in the 9-attribute row OR keyword `DEFERRED` in the boundaries column. Both signals normalize to the same JSON value, so consumers don't distinguish "permanent dormant" (#17) from "deferred-pending" (#44 / #50 / #54 / #67) — they're all "cannot activate right now".
|
||||
- `tools/missed-activations.mjs` — pure deterministic matcher. Exports `detectMissedActivations(episodes, classificationMap, dormancy)`. No fs, no exec.
|
||||
|
||||
**Detection threshold:** single episode (per user decision 2026-05-21). No smoothing; every qualifying episode counts.
|
||||
|
||||
**DEFERRED exclusion:** nodes flagged as unavailable in `.node-dormancy.json` are filtered before counting. Current dormant set: #1 (replaced), #17 (pg_partman, native-Windows), #44 (Figma MCP, no Figma account), #50 (Jupyter MCP, no Python ML env), #54 (n8n-mcp, n8n not in stack), #67 (NightOwl, pending Б-1 / Linux).
|
||||
|
||||
**Surfacing:**
|
||||
|
||||
- C5 `observer-coverage-checker` includes `missed.totalMissed` in its return value; the CLI emits `WARN — missed activations: N (see /brain-retro)` when N > 0.
|
||||
- `status-md-generator` renders `missed_activations: N` in the metrics block; C5 row turns ⚠️ when N > 0.
|
||||
- `/brain-retro` `analyze(episodes, { classificationMap, dormancy })` returns `missedActivations: { totalMissed, byNode, byClassification }` — the retro skill renders a per-node + per-classification breakdown.
|
||||
|
||||
**Initial measurement on May 2026 episodes:** 16 missed activations, dominated by memory-sync × 7 (CLAUDE.md edits without `#33 claude-md-management` chosen) and feature × 4 (no Superpowers brainstorming invocation). This is the kind of "router miss" signal the rule is designed to surface, not a problem in the unactivated nodes themselves.
|
||||
|
||||
**Linkage:**
|
||||
|
||||
- Pravila §16.4 v1.36 (2026-05-21).
|
||||
- Plan: `docs/superpowers/plans/2026-05-21-observer-missed-activations.md`.
|
||||
- Spec / decision rationale: this amendment.
|
||||
|
||||
@@ -1,21 +1,22 @@
|
||||
# Brain Status (auto-generated)
|
||||
|
||||
Last updated: 2026-05-21T01:18:52.154Z
|
||||
Last updated: 2026-05-21T06:54:27.698Z
|
||||
|
||||
| Контролёр | Состояние | Детали |
|
||||
|---|---|---|
|
||||
| C1 L1-watcher | ✅ | [l1-watcher] OK — 0 drift |
|
||||
| C2 Cross-ref consistency | ✅ | [cross-ref-checker] OK — 0 drift in 4 files |
|
||||
| C2 Cross-ref consistency | 🔴 | Update cross-refs in offending files. |
|
||||
| C3 Observer-of-observer | ✅ | [observer-of-observer] OK — last read 0 week(s) ago |
|
||||
| C4 Сигнальный статус | ✅ | This file (self-reference) |
|
||||
| C5 Observer-coverage | ⚠️ | 16 episode(s) this month · .git/hooks/post-commit not installed (run: npx lefthook install --force) |
|
||||
| C5 Observer-coverage | ⚠️ | 39 episode(s) this month · .git/hooks/post-commit not installed (run: npx lefthook install --force) · 16 missed activation(s) — see /brain-retro |
|
||||
| C6 Chain map sync | ✅ | [chain-map-checker] OK — 14 chains in sync |
|
||||
|
||||
## Метрики (информационные, не алерты)
|
||||
|
||||
- Observer evidence: 16 episodes this month, 0 observer_error markers, 3 PII matches before filter
|
||||
- Observer evidence: 39 episodes this month, 0 observer_error markers, 0 PII matches before filter
|
||||
- Legacy v1 episodes (not in factor analysis): 5
|
||||
- Last /brain-retro: 2 day(s) ago
|
||||
- Использование узлов: см. `/brain-retro` (раз в спринт). **Неиспользованные узлы — не проблема** (capability-readiness; см. memory `feedback_brain_unused_tools_not_problem` — outside-repo memory store).
|
||||
- Использование узлов: см. `/brain-retro` (раз в спринт). missed_activations: 16. **Неиспользованные узлы — не алерт, если профильной задачи не было** (Pravila §16.4 v1.36; capability-readiness; см. memory `feedback_brain_unused_tools_not_problem` — outside-repo memory store).
|
||||
|
||||
## Алерт-индикаторы
|
||||
|
||||
|
||||
File diff suppressed because one or more lines are too long
@@ -0,0 +1,219 @@
|
||||
# Brain-retro #2 — весь май 2026 (полный срез)
|
||||
|
||||
**Дата:** 2026-05-20 (вечер, ~17:55 MSK)
|
||||
**Период:** весь май 2026 — 2026-05-19T05:18Z .. 2026-05-20T08:58Z (28 строк JSONL; 23 v2-эпизода + 5 v1 пропущено).
|
||||
**Источник:** `docs/observer/episodes-2026-05.jsonl` (28 строк) + `docs/observer/.read-counter.json`.
|
||||
**Анализатор:** `node tools/brain-retro-analyzer.mjs docs/observer/episodes-2026-05.jsonl`.
|
||||
**Отношение к предыдущему ретро:** надстройка над [2026-05-20-brain-retro.md](2026-05-20-brain-retro.md) (то — 17 v2-эпизодов, 12:25 MSK); здесь — те же 17 + дельта в 6 новых.
|
||||
**Уровень анализа:** верхнеуровневый по запросу заказчика; экономия 100%.
|
||||
|
||||
> Анализатор: `episodeCount=23`, `v1SkippedCount=5`, `observerErrorCount=0`. Все цифры по 23 v2-эпизодам, если не отмечено иное.
|
||||
|
||||
---
|
||||
|
||||
## Period
|
||||
|
||||
2026-05-19T05:18:16Z .. 2026-05-20T08:58:44Z. **7 уникальных task_id (сессий)**, 23 v2-анализируемых эпизода.
|
||||
|
||||
Дельта vs прошлое ретро (6 новых эпизодов после 2026-05-20T08:12:29Z):
|
||||
|
||||
| task_id | turn | start..end (Z) | path_type | provenance | node_chosen | econ | tool_calls | files | events примечательное | inferred outcome |
|
||||
|---|---|---|---|---|---|---|---|---|---|---|
|
||||
| `98298ec2` | 5 | 08:13..08:19 | improvised | autonomous | direct | 100 | 19 | 4 | 2× error tool_result + 2× retry | success (continuation) |
|
||||
| `35fc31da` | 1 | 08:16..08:24 | improvised | autonomous | **brain-retro** | 0 | 20 | 6 | skill_invoked brain-retro | **rework** (следующий ход — correction) |
|
||||
| `35fc31da` | 2 | 08:30..08:36 | improvised | autonomous | direct | 5 | 17 | 4 | 1× error + retry; prompt_signal=**correction** | success |
|
||||
| `35fc31da` | 3 | 08:36..08:37 | improvised | user_directed_method (`claude_would_have_chosen=brain-retro`) | direct | null | 0 | 0 | — | unknown (no-op ход) |
|
||||
| `35fc31da` | 4 | 08:39..08:46 | improvised | autonomous | direct | 0 | 15 | 14 | 14× Read mass | success |
|
||||
| `286dd904` | 2 | 07:52..08:58 | **regulated** | **user_chose_from_options** ("На Plan 3 (экспорт)") | **superpowers:verification-before-completion** | 5 | 133 | 25 | skill_invoked verification; 1× error; **time_burn 66 мин** | unknown (нет следующего эпизода) |
|
||||
|
||||
---
|
||||
|
||||
## Path-type distribution (v2, n=23)
|
||||
|
||||
| path_type | count | % |
|
||||
|---|---|---|
|
||||
| improvised | 20 | 87.0% |
|
||||
| regulated | 3 | 13.0% |
|
||||
| alternative | 0 | 0% |
|
||||
| mixed | 0 | 0% |
|
||||
|
||||
Доля regulated на 4.6 п.п. ниже прошлого ретро (17.6% → 13.0%) — три новых improvised-эпизода без skill в дельте сдвинули долю.
|
||||
|
||||
## Outcome distribution
|
||||
|
||||
| outcome | count | % |
|
||||
|---|---|---|
|
||||
| success (inferred) | 9 | 39.1% |
|
||||
| rework (inferred) | 1 | 4.3% |
|
||||
| unknown (последние/нет следующего) | 13 | 56.5% |
|
||||
|
||||
«Unknown» здесь — это эпизоды, после которых нет хода с positive/correction-сигналом (хвост сессий) — не provenance-bug.
|
||||
|
||||
## Skill invocations (events `skill_invoked`, n=6)
|
||||
|
||||
| skill | times | sessions |
|
||||
|---|---|---|
|
||||
| superpowers:verification-before-completion | 2 | `553717ec`, `286dd904` |
|
||||
| superpowers:systematic-debugging | 1 | `553717ec` |
|
||||
| superpowers:test-driven-development | 1 | `553717ec` |
|
||||
| claude-md-management:claude-md-improver | 1 | `553717ec` |
|
||||
| brain-retro | 1 | `35fc31da` |
|
||||
|
||||
## Factor analysis matrix (analyzer `factorMatrix`)
|
||||
|
||||
### decision_provenance — «rework мой или роутера?»
|
||||
|
||||
| provenance | success | rework | unknown |
|
||||
|---|---|---|---|
|
||||
| autonomous | 6 | 1 | 10 |
|
||||
| user_directed_method | 2 | 0 | 2 |
|
||||
| user_chose_from_options | 0 | 0 | 2 |
|
||||
|
||||
Единственный rework (`brain-retro` turn 1) — autonomous-выбор узла brain-retro заказчиком (это сам skill, инвокированный по `/brain-retro`); коррекция — про точность аналитики прошлого ретро, не про routing. **«Rework мой, не роутера.»**
|
||||
|
||||
### economy_level
|
||||
|
||||
| economy_level | success | rework | unknown |
|
||||
|---|---|---|---|
|
||||
| null | 3 | 0 | 2 |
|
||||
| 0 | 0 | 1 | 2 |
|
||||
| 5 | 4 | 0 | 6 |
|
||||
| 100 | 1 | 0 | 4 |
|
||||
|
||||
Слишком маленькая выборка для выводов; единственный rework на 0% — это brain-retro turn 1 (для самого ретро economy=0% это норма, заказчик так попросил).
|
||||
|
||||
### model · post_compaction · task_size
|
||||
|
||||
| factor value | success | rework | unknown |
|
||||
|---|---|---|---|
|
||||
| model: claude-opus-4-7 | 8 | 1 | 14 |
|
||||
| post_compaction=true | 6 | 0 | 5 |
|
||||
| post_compaction=false | 2 | 1 | 9 |
|
||||
| session_turn late (≥10) | 6 | 0 | 5 |
|
||||
| session_turn early (<10) | 2 | 1 | 9 |
|
||||
| task_size small | 8 | 0 | 11 |
|
||||
| task_size medium | 0 | 1 | 2 |
|
||||
| task_size large | 0 | 0 | 1 |
|
||||
|
||||
Все эпизоды на одной модели → строка про model — не сигнал. Post_compaction=true и late session_turn — это одна и та же длинная brain-governance сессия `553717ec` (turn 82+); концентрация success там — артефакт сессии, не закономерность.
|
||||
|
||||
### node_chosen · task_classification
|
||||
|
||||
| node_chosen | success | rework | unknown |
|
||||
|---|---|---|---|
|
||||
| direct | 8 | 0 | 11 |
|
||||
| superpowers:verification-before-completion | 0 | 0 | 1 |
|
||||
| superpowers:systematic-debugging | 0 | 0 | 1 |
|
||||
| superpowers:test-driven-development | 0 | 0 | 1 |
|
||||
| brain-retro | 0 | 1 | 0 |
|
||||
|
||||
| task_classification | success | rework | unknown |
|
||||
|---|---|---|---|
|
||||
| bugfix | 2 | 0 | 1 |
|
||||
| feature | 2 | 0 | 2 |
|
||||
| other | 3 | 0 | 9 |
|
||||
| refactor | 1 | 0 | 0 |
|
||||
| question | 0 | 1 | 2 |
|
||||
|
||||
«direct» — 8/0/11 — основная масса задач без skill-маршрутизации, всё работает. superpowers-узлы (3 эпизода, все unknown) сидят в хвостах своих сессий — нет следующего хода с явным signal.
|
||||
|
||||
## Episodes → tasks (analyzer `tasks`, 15 task-групп)
|
||||
|
||||
| task_ref | episodes | rework turns |
|
||||
|---|---|---|
|
||||
| `553717ec#1..#10` | 10 | turn 82 (rework — improvised CLAUDE.md edit, retry-recovered) |
|
||||
| `24acfa10#1` | 1 | — |
|
||||
| `a42e4ba5#1` | 1 | — |
|
||||
| `dd905ea0#1` | 1 | — |
|
||||
| `98298ec2#1..#3` | 3 (continuation across 5 turns) | — |
|
||||
| `35fc31da#1..#4` | 4 | turn 1 (brain-retro) — correction в turn 2 |
|
||||
| `286dd904#1` | 1 (66-min verification) | — |
|
||||
|
||||
## Causal-chain candidates (analyzer `causalChains`)
|
||||
|
||||
| from | to | shared files |
|
||||
|---|---|---|
|
||||
| — | — | — |
|
||||
|
||||
Анализатор не нашёл «errored episode → fix episode на тех же файлах». 7 событий error (`tool_result reported is_error`) — это transient-сбои тулов внутри одного эпизода с retry-recovery, не межэпизодные.
|
||||
|
||||
## Observer health
|
||||
|
||||
- `observerErrorCount = 0` — за весь май **ни одного** `observer_error`-маркера. Парсер ни разу не сломался тихо.
|
||||
- `interrupts = 0` — заказчик ни разу не прерывал ход.
|
||||
- `errors = 7` (внутри 5 эпизодов) — все transient, retry-recovered.
|
||||
- `retries = 6` — корреспондируют ошибкам один-в-один (один retry бесплатный после restart-tooling).
|
||||
- `time_burn_total = 86 мин` — из них 66 мин — один эпизод `286dd904#2` (длинная verification-сессия Plan 2 экспорт).
|
||||
|
||||
## Canonical chains L1–L12 hit rate
|
||||
|
||||
Не считаем за май — нет атрибуции `chain_ref`; routing-таблица `docs/routing-off-phase.md` v1.2 ещё не интегрирована в primary_rationale. Кандидат на доработку парсера — см. ниже.
|
||||
|
||||
## Improvised chains (повторённые ≥2)
|
||||
|
||||
| node-set | times | candidate L13+? |
|
||||
|---|---|---|
|
||||
| direct → direct (continuation в одной сессии) | 14 | нет — это норма, не цепочка |
|
||||
|
||||
Других повторов нет.
|
||||
|
||||
## chain_divergence cases
|
||||
|
||||
Нет атрибуции — пропуск.
|
||||
|
||||
## Top error classes
|
||||
|
||||
| error class | count | recovery pattern |
|
||||
|---|---|---|
|
||||
| `tool_result reported is_error` (transient) | 7 | retry в том же эпизоде, без user-intervention |
|
||||
|
||||
## confusion_marker hot-spots
|
||||
|
||||
Нет таких маркеров в схеме v2 — пропуск.
|
||||
|
||||
---
|
||||
|
||||
## Candidates for owner review
|
||||
|
||||
### Candidate 1: уточнить analyzer для дельта-сравнения с предыдущим ретро
|
||||
|
||||
- **Type:** doc/skill enhancement, не нормативная правка.
|
||||
- **Evidence:** прошлое ретро (35fc31da#1) → `prompt_signal=correction` следующего хода (35fc31da#2). Анализатор корректно пометил `outcome=rework`, но в выводе нет указателя на номер прошлого ретро или diff vs предыдущий.
|
||||
- **Suggested action:** в `tools/brain-retro-analyzer.mjs` добавить опциональный аргумент `--since <ISO>` (срез по `started_at >= since`), чтобы можно было дёшево считать только дельту между ретро. Альтернатива: в шаблоне `.claude/skills/brain-retro/references/aggregation-template.md` добавить секцию «Delta vs prior retro» с явным diff'ом.
|
||||
- **Cost / risk:** низкий; чистый node-скрипт без побочек на JSONL. Сейчас процесс ручной (этот ретро diff делался руками).
|
||||
- **Rejection option:** заказчик может сказать «всегда срез — полный месяц», и тогда диффы не нужны.
|
||||
|
||||
### Candidate 2: атрибуция canonical chains L1–L12 в primary_rationale
|
||||
|
||||
- **Type:** observer schema extension (потребует amend ADR-011 / spec factor-analysis).
|
||||
- **Evidence:** ни один из 23 эпизодов не несёт ссылки на L1–L12 chain из `docs/routing-off-phase.md` v1.2 (а с finance-tooling — там уже L1–L13). «Canonical chains hit rate» — пустая таблица.
|
||||
- **Suggested action:** в `tools/observer-routing-detector.mjs` (или новый детектор) маппить выбранные узлы в L-цепочку и писать `primary_rationale.chain_ref: "L7"` (например). Только тогда можно отслеживать чистоту следования цепочкам.
|
||||
- **Cost / risk:** средний — нужен маппинг «node_chosen → L-chain», который сейчас живёт только в человеческом тексте routing-off-phase.md. Риск: дрейф маппинга между парсером и документом.
|
||||
- **Rejection option:** оставить L1–L13 как нормативное «чтение для человека», не пытаться формализовать.
|
||||
|
||||
### Candidate 3: проверить корректность аналитики прошлого ретро
|
||||
|
||||
- **Type:** ad-hoc review (не нормативка).
|
||||
- **Evidence:** rework-flag на 35fc31da#1 — единственный rework в выборке. Я (текущее ретро) дельту посчитал и нашёл, что предыдущее ретро отчиталось «17 эпизодов, 5 v1 пропущено» — это совпадает с записанным; коррекция была про что-то другое (содержание ретро, не структура). Без доступа к самой формулировке коррекции (`35fc31da#2` body) можно только сказать: коррекция структурно не была про observability, а про текст ретро.
|
||||
- **Suggested action:** заказчик при желании может прокомментировать «что именно правил после ретро 12:25», и записать урок в notes. **Не блокирующее.**
|
||||
- **Rejection option:** игнорировать — мелкая коррекция текста, не системный сигнал.
|
||||
|
||||
(Других кандидатов нет. Никаких removals/zombie nodes per memory `feedback_brain_unused_tools_not_problem`.)
|
||||
|
||||
---
|
||||
|
||||
## Informational metrics (NOT alerts)
|
||||
|
||||
- Узлов, использованных хотя бы раз за период (явно через `skill_invoked`): **5 / 63+** (superpowers TDD/debug/verify, claude-md-improver, brain-retro). Узел `direct` (=прямое исполнение) — отдельная категория, 19 эпизодов.
|
||||
- Узлов, ни разу не использованных с начала наблюдения: **большинство (≥55 из 63+)** — **не проблема** per [feedback_brain_unused_tools_not_problem](../../../../C:/Users/Administrator/.claude/projects/c---------------------crm-------------/memory/feedback_brain_unused_tools_not_problem.md). Capability-readiness — осознанная стратегия заказчика.
|
||||
- Параллельные сессии: 12 эпизодов из 23 (52%) с `parallel_session=true` — норма для текущего рабочего режима с §15 Pravila.
|
||||
- Длинные эпизоды (`time_burn` событие): 1 — `286dd904#2` 66 мин (Plan 2 экспорт verification). Все остальные — без time_burn-маркера (под 60 мин).
|
||||
|
||||
---
|
||||
|
||||
## Дельта vs прошлое ретро 2026-05-20 12:25 — итог
|
||||
|
||||
- Картина устойчива: improvised-доминанта, opus-4-7-only, 0 observer_error, rework на autonomous-выборах единичный.
|
||||
- Новый класс события: **`prompt_signal=correction` после skill-инвокации** (35fc31da#1 → #2). Раньше correction наблюдался только на autonomous direct-выборах; теперь видно, что brain-retro skill тоже не неприкосновенен — это здоровый сигнал.
|
||||
- Новый успешный кейс гибридной модели: **`user_chose_from_options` → regulated path** (286dd904#2, 66 мин, 133 tool_calls, через superpowers:verification-before-completion). Это первое подтверждение, что collaborative-choice + skill-маршрутизация даёт длинные продуктивные эпизоды без interruptions.
|
||||
- Никаких рекомендаций править Pravila / PSR_v1 / Tooling / CLAUDE.md — выборка (23 эпизода / 2 дня) слишком мала.
|
||||
@@ -0,0 +1,820 @@
|
||||
# Observer Canonical Chain Attribution (L1–L13) Implementation Plan
|
||||
|
||||
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
|
||||
|
||||
**Goal:** Добавить опциональное поле `chain_ref` в эпизоды наблюдателя, связывающее `node_chosen` с каноническими цепочками L1–L13, чтобы `/brain-retro` мог считать «hit rate цепочек».
|
||||
|
||||
**Architecture:** Один новый слой над наблюдателем — статический JSON-маппинг `node_chosen → [LN]`, чистая функция-детектор, врезка в парсер транскрипта, контролёр C6 сверки JSON↔`routing-off-phase.md` (в lefthook pre-commit + Vitest), одноразовый ретрофилл существующих эпизодов, агрегация в анализаторе. Всё детерминированно — 0 LLM-вызовов.
|
||||
|
||||
**Tech Stack:** Node 20+ ESM, Vitest 4.1.5, pure fs/regex (Security Guidance #40 — никаких shell-вызовов в parser/hook). Раннер: `npm run test:tools` (`cd app && npx vitest run --config vitest.config.tools.mjs`). Тесты лежат рядом с модулями: `tools/<name>.test.mjs`.
|
||||
|
||||
**Spec:** [docs/superpowers/specs/2026-05-20-observer-chain-attribution-design.md](../specs/2026-05-20-observer-chain-attribution-design.md).
|
||||
|
||||
---
|
||||
|
||||
## ⚠️ Внешняя зависимость и порядок (читать ПЕРВЫМ)
|
||||
|
||||
**Этот план НЕ исполняется немедленно.** Жёсткая зависимость от epic-плана [2026-05-20-observer-instrument-expansion.md](2026-05-20-observer-instrument-expansion.md) (20 task), который правит тот же `tools/observer-transcript-parser.mjs`.
|
||||
|
||||
**Процедура старта (Task 0 ниже):**
|
||||
|
||||
1. Дождаться сообщения «epic 20-task закрыт и push'нут на origin/main».
|
||||
2. `git fetch origin && git log HEAD..origin/main --oneline` — убедиться, что 20 task влиты (Pravila §15.2 pre-flight).
|
||||
3. Создать свежий worktree off `origin/main` (Task 0).
|
||||
4. Исполнять Tasks 1–9 по порядку.
|
||||
|
||||
---
|
||||
|
||||
## Уточнения к spec (раскрыты при детализации writing-plans)
|
||||
|
||||
1. **Расположение тестов.** Spec §5/§8 называл `tests/observer-chain-*.test.mjs`. Реальный паттерн репозитория — тест рядом с модулем: `tools/observer-chain-detector.test.mjs`. План использует реальный паттерн.
|
||||
|
||||
2. **Семантика контролёра C6 — по L-номерам, не по именам узлов.** Имена в таблице `routing-off-phase.md` — человеческие display-names (`Boost MCP`, `Trail of Bits`, `Semgrep MCP`), а `node_chosen` в эпизодах — технические skill-id (`superpowers:test-driven-development`, `claude-md-management:claude-md-improver`, `direct`). Прямая построчная сверка имён хрупкая. Поэтому C6 v1 сверяет **множества L-номеров**:
|
||||
- каждый `LN`, упомянутый в JSON, существует в `.md` (нет ссылок на несуществующую цепочку);
|
||||
- каждый `LN` из таблицы `.md` присутствует хотя бы в одной записи JSON (цепочка не «потеряна» при добавлении новой L в .md).
|
||||
|
||||
Это ловит главный класс дрейфа («добавили L14 в .md — JSON про неё не знает»). Точечная сверка «узел X в L7» через display-name-алиасы — out of scope v1 (можно как future-слой).
|
||||
|
||||
3. **Точка врезки в parser.** На `origin/main` это `return { … node_chosen: skills.length > 0 ? skills[0] : 'direct', … }` (≈строка 696). После epic-плана номер строки сдвинется — **искать по grep-маркеру** `node_chosen: skills.length > 0 ? skills[0] : 'direct'`, не по номеру.
|
||||
|
||||
---
|
||||
|
||||
## File Structure
|
||||
|
||||
| Файл | Тип | Ответственность |
|
||||
|---|---|---|
|
||||
| `tools/observer-chain-map.json` | новый (data) | Маппинг `node_chosen` (реальное значение) → массив `["LN"]`. Только узлы, входящие в L1–L13 |
|
||||
| `tools/observer-chain-detector.mjs` | новый | `loadChainMap(path)` + чистая `chainsFor(node, map)` → массив \| `null` |
|
||||
| `tools/observer-chain-detector.test.mjs` | новый | Юнит-тесты `chainsFor` |
|
||||
| `tools/observer-transcript-parser.mjs` | edit | Врезка `chain_ref` в `primary_rationale` |
|
||||
| `tools/observer-transcript-parser.test.mjs` | edit | +тест что эпизод несёт `chain_ref` |
|
||||
| `tools/observer-chain-map-checker.mjs` | новый | C6: `parseChainsFromMd()` + `checkSync()` + CLI |
|
||||
| `tools/observer-chain-map-checker.test.mjs` | новый | Тесты парсера .md + sync-сверки |
|
||||
| `lefthook.yml` | edit | Job 16 `observer-chain-map-checker` в pre-commit |
|
||||
| `tools/observer-retrofill-chain-ref.mjs` | новый | Одноразовый ретрофилл `chain_ref` в JSONL |
|
||||
| `tools/observer-retrofill-chain-ref.test.mjs` | новый | Тесты идемпотентности + dry-run |
|
||||
| `tools/brain-retro-analyzer.mjs` | edit | `factorMatrix.chain_ref` + `chainHitRate` |
|
||||
| `tools/brain-retro-analyzer.test.mjs` | edit | +тест агрегации chain_ref |
|
||||
| `tools/status-md-generator.mjs` | edit | Строка «C6 Chain map sync» |
|
||||
| `tools/status-md-generator.test.mjs` | edit | +тест строки C6 |
|
||||
| `.claude/skills/brain-retro/references/aggregation-template.md` | edit | Заполнить секцию «L1–L13 hit rate» |
|
||||
|
||||
---
|
||||
|
||||
## Task 0: Pre-flight + worktree (организационный, не код)
|
||||
|
||||
**Files:** нет правок кода.
|
||||
|
||||
- [ ] **Step 1: Убедиться, что epic-план влит**
|
||||
|
||||
Run:
|
||||
|
||||
```bash
|
||||
git fetch origin
|
||||
git log --oneline -5 origin/main
|
||||
git log HEAD..origin/main --oneline | grep -i "observer-instrument-expansion\|Task 21\|Task 20" || echo "epic не найден — НЕ СТАРТОВАТЬ"
|
||||
```
|
||||
|
||||
Expected: видны коммиты закрытия epic 20-task на origin/main. Если нет — остановиться, сообщить владельцу.
|
||||
|
||||
- [ ] **Step 2: Создать worktree off origin/main**
|
||||
|
||||
Использовать `superpowers:using-git-worktrees`. Целевая база — `origin/main` (свежий, после epic). Ветка `feat/observer-chain-attribution`.
|
||||
|
||||
- [ ] **Step 3: Verify базовая регрессия зелёная**
|
||||
|
||||
Run: `npm run test:tools`
|
||||
Expected: PASS (≥ baseline после epic, например 350+/350+). Записать число baseline для финальной сверки.
|
||||
|
||||
---
|
||||
|
||||
## Task 1: chain-map JSON + детектор `chainsFor`
|
||||
|
||||
**Files:**
|
||||
|
||||
- Create: `tools/observer-chain-map.json`
|
||||
- Create: `tools/observer-chain-detector.mjs`
|
||||
- Test: `tools/observer-chain-detector.test.mjs`
|
||||
|
||||
- [ ] **Step 1: Создать JSON-маппинг**
|
||||
|
||||
`tools/observer-chain-map.json` — только узлы, входящие в L1–L13. Имена ключей = реальные значения `node_chosen` (skill-id). NB: значения `node_chosen` берутся из первого `skill_invoked` (`skills[0]`); для skill-узлов это `plugin:skill` или `skill`. MCP-узлы (Boost/Sentry/Redis) в `node_chosen` не появляются (они не skill_invoked) — но включены в маппинг на будущее, если детектор узлов расширится.
|
||||
|
||||
```json
|
||||
{
|
||||
"_note": "node_chosen -> L-цепочки. Только узлы, входящие хотя бы в одну L1-L13. Узлы вне цепочек (direct, прочее) НЕ включаются -> chainsFor вернёт null. Имена ключей = реальные значения primary_rationale.node_chosen. Синхронизируется с docs/routing-off-phase.md через контролёр C6 (tools/observer-chain-map-checker.mjs).",
|
||||
"discovery-interview": ["L1", "L2"],
|
||||
"superpowers:brainstorming": ["L1"],
|
||||
"superpowers:writing-plans": ["L1"],
|
||||
"superpowers:subagent-driven-development": ["L1"],
|
||||
"audit-portal": ["L2"],
|
||||
"process-analysis": ["L3"],
|
||||
"process-modeling": ["L3", "L4"],
|
||||
"mermaid": ["L4"],
|
||||
"adr-kit:adr": ["L4", "L5"],
|
||||
"adr-kit:judge": ["L5"],
|
||||
"operations": ["L4"],
|
||||
"architecture-patterns:architecture-patterns": ["L5"],
|
||||
"deptrac": ["L5"],
|
||||
"security-review": ["L6"],
|
||||
"superpowers:systematic-debugging": ["L8"],
|
||||
"ccpm": ["L9"],
|
||||
"product-management:brainstorm": ["L9"],
|
||||
"promptfoo": ["L10"],
|
||||
"data-scientist": ["L10"],
|
||||
"claude-api": ["L10"],
|
||||
"skill-creator:skill-creator": ["L11"],
|
||||
"hookify:hookify": ["L11"],
|
||||
"plugin-dev:create-plugin": ["L11"],
|
||||
"claude-md-management:claude-md-improver": ["L12"],
|
||||
"claude-md-management:revise-claude-md": ["L12"],
|
||||
"billing-audit": ["L13"],
|
||||
"ru-tax-accounting": ["L13"]
|
||||
}
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Написать failing-тест детектора**
|
||||
|
||||
`tools/observer-chain-detector.test.mjs`:
|
||||
|
||||
```javascript
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { loadChainMap, chainsFor } from './observer-chain-detector.mjs';
|
||||
|
||||
const map = loadChainMap();
|
||||
|
||||
describe('chainsFor', () => {
|
||||
it('returns chain array for a single-chain node', () => {
|
||||
expect(chainsFor('billing-audit', map)).toEqual(['L13']);
|
||||
});
|
||||
|
||||
it('returns all chains for a multi-chain node', () => {
|
||||
expect(chainsFor('discovery-interview', map)).toEqual(['L1', 'L2']);
|
||||
});
|
||||
|
||||
it('returns null for direct', () => {
|
||||
expect(chainsFor('direct', map)).toBeNull();
|
||||
});
|
||||
|
||||
it('returns null for an unknown node', () => {
|
||||
expect(chainsFor('totally-unknown-xyz', map)).toBeNull();
|
||||
});
|
||||
|
||||
it('returns null for empty/null/undefined', () => {
|
||||
expect(chainsFor('', map)).toBeNull();
|
||||
expect(chainsFor(null, map)).toBeNull();
|
||||
expect(chainsFor(undefined, map)).toBeNull();
|
||||
});
|
||||
|
||||
it('ignores the _note metadata key', () => {
|
||||
expect(chainsFor('_note', map)).toBeNull();
|
||||
});
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 3: Запустить тест — убедиться, что падает**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-chain-detector.test.mjs`
|
||||
Expected: FAIL — `loadChainMap is not a function` / module not found.
|
||||
|
||||
- [ ] **Step 4: Реализовать детектор**
|
||||
|
||||
`tools/observer-chain-detector.mjs`:
|
||||
|
||||
```javascript
|
||||
import { readFileSync } from 'node:fs';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
|
||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
||||
const DEFAULT_MAP_PATH = join(__dirname, 'observer-chain-map.json');
|
||||
|
||||
/** Load the node->chains map. Throws on missing/invalid JSON (caller handles). */
|
||||
export function loadChainMap(path = DEFAULT_MAP_PATH) {
|
||||
const raw = JSON.parse(readFileSync(path, 'utf8'));
|
||||
const map = new Map();
|
||||
for (const [node, chains] of Object.entries(raw)) {
|
||||
if (node === '_note') continue;
|
||||
if (Array.isArray(chains) && chains.length > 0) map.set(node, chains);
|
||||
}
|
||||
return map;
|
||||
}
|
||||
|
||||
/** node_chosen -> array of L-chains, or null if not in any chain. */
|
||||
export function chainsFor(node, map) {
|
||||
if (!node || typeof node !== 'string') return null;
|
||||
const chains = map.get(node);
|
||||
return chains && chains.length > 0 ? chains : null;
|
||||
}
|
||||
```
|
||||
|
||||
- [ ] **Step 5: Запустить тест — убедиться, что проходит**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-chain-detector.test.mjs`
|
||||
Expected: PASS (6 tests).
|
||||
|
||||
- [ ] **Step 6: Commit**
|
||||
|
||||
```bash
|
||||
git add tools/observer-chain-map.json tools/observer-chain-detector.mjs tools/observer-chain-detector.test.mjs
|
||||
git commit -m "feat(observer): chain-map JSON + chainsFor detector (L1-L13 attribution)"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 2: Врезка `chain_ref` в парсер транскрипта
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `tools/observer-transcript-parser.mjs` (grep-маркер `node_chosen: skills.length > 0 ? skills[0] : 'direct'`)
|
||||
- Test: `tools/observer-transcript-parser.test.mjs`
|
||||
|
||||
- [ ] **Step 1: Написать failing-тест**
|
||||
|
||||
Добавить в `tools/observer-transcript-parser.test.mjs` (в подходящий describe для primary_rationale; адаптировать фабрику транскрипта под существующие хелперы файла):
|
||||
|
||||
```javascript
|
||||
it('attaches chain_ref for a node that belongs to a chain', () => {
|
||||
// транскрипт с skill_invoked = 'billing-audit' (адаптировать под фабрику файла)
|
||||
const episode = parseTranscript(transcriptWithSkill('billing-audit'));
|
||||
expect(episode.primary_rationale.chain_ref).toEqual(['L13']);
|
||||
});
|
||||
|
||||
it('sets chain_ref null for a direct episode', () => {
|
||||
const episode = parseTranscript(transcriptWithNoSkill());
|
||||
expect(episode.primary_rationale.chain_ref).toBeNull();
|
||||
});
|
||||
```
|
||||
|
||||
NB: точные имена хелперов (`parseTranscript` / фабрики) взять из существующего теста — не выдумывать. Если фабрики нет — собрать минимальный transcript-объект вручную по образцу соседних тестов.
|
||||
|
||||
- [ ] **Step 2: Запустить — убедиться, что падает**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-transcript-parser.test.mjs -t chain_ref`
|
||||
Expected: FAIL — `chain_ref` undefined.
|
||||
|
||||
- [ ] **Step 3: Реализовать врезку**
|
||||
|
||||
В `tools/observer-transcript-parser.mjs`:
|
||||
|
||||
1. Вверху файла добавить импорт:
|
||||
|
||||
```javascript
|
||||
import { loadChainMap, chainsFor } from './observer-chain-detector.mjs';
|
||||
```
|
||||
|
||||
1. Один раз модульно загрузить карту с защитой от битого JSON:
|
||||
|
||||
```javascript
|
||||
let CHAIN_MAP = null;
|
||||
try {
|
||||
CHAIN_MAP = loadChainMap();
|
||||
} catch {
|
||||
CHAIN_MAP = new Map(); // битый/отсутствующий JSON -> chainsFor вернёт null, observer не падает
|
||||
}
|
||||
```
|
||||
|
||||
1. Найти grep-маркер `node_chosen: skills.length > 0 ? skills[0] : 'direct'` и добавить рядом строку. Должно получиться:
|
||||
|
||||
```javascript
|
||||
node_chosen: skills.length > 0 ? skills[0] : 'direct',
|
||||
chain_ref: chainsFor(skills.length > 0 ? skills[0] : 'direct', CHAIN_MAP),
|
||||
```
|
||||
|
||||
- [ ] **Step 4: Запустить — убедиться, что проходит**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-transcript-parser.test.mjs`
|
||||
Expected: PASS (включая существующие тесты файла — ни один не сломан).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add tools/observer-transcript-parser.mjs tools/observer-transcript-parser.test.mjs
|
||||
git commit -m "feat(observer): emit chain_ref in primary_rationale"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 3: Контролёр C6 — sync JSON ↔ routing-off-phase.md
|
||||
|
||||
**Files:**
|
||||
|
||||
- Create: `tools/observer-chain-map-checker.mjs`
|
||||
- Test: `tools/observer-chain-map-checker.test.mjs`
|
||||
|
||||
- [ ] **Step 1: Написать failing-тест**
|
||||
|
||||
`tools/observer-chain-map-checker.test.mjs`:
|
||||
|
||||
```javascript
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { parseChainsFromMd, checkSync } from './observer-chain-map-checker.mjs';
|
||||
|
||||
const SAMPLE_MD = [
|
||||
'| # | Цепочка | Зачем |',
|
||||
'|---|---|---|',
|
||||
'| L1 | `discovery-interview` (FEATURE) → `brainstorming` | text |',
|
||||
'| L2 | `audit-portal` | text |',
|
||||
'| L13 | `billing-audit` (#62) + `Pest` | text |',
|
||||
].join('\n');
|
||||
|
||||
describe('parseChainsFromMd', () => {
|
||||
it('extracts the set of L-numbers from the table', () => {
|
||||
expect(parseChainsFromMd(SAMPLE_MD)).toEqual(new Set(['L1', 'L2', 'L13']));
|
||||
});
|
||||
});
|
||||
|
||||
describe('checkSync', () => {
|
||||
it('passes when JSON L-numbers subset of md and md subset of json-union', () => {
|
||||
const mdSet = new Set(['L1', 'L2', 'L13']);
|
||||
const jsonMap = { a: ['L1'], b: ['L2'], c: ['L13'] };
|
||||
expect(checkSync(jsonMap, mdSet).ok).toBe(true);
|
||||
});
|
||||
|
||||
it('fails when JSON references a chain absent from md', () => {
|
||||
const mdSet = new Set(['L1', 'L2']);
|
||||
const jsonMap = { a: ['L1'], b: ['L99'] };
|
||||
const res = checkSync(jsonMap, mdSet);
|
||||
expect(res.ok).toBe(false);
|
||||
expect(res.jsonOnly).toContain('L99');
|
||||
});
|
||||
|
||||
it('fails when md has a chain not covered by any JSON entry', () => {
|
||||
const mdSet = new Set(['L1', 'L2', 'L14']);
|
||||
const jsonMap = { a: ['L1'], b: ['L2'] };
|
||||
const res = checkSync(jsonMap, mdSet);
|
||||
expect(res.ok).toBe(false);
|
||||
expect(res.mdOnly).toContain('L14');
|
||||
});
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — убедиться, что падает**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-chain-map-checker.test.mjs`
|
||||
Expected: FAIL — module not found.
|
||||
|
||||
- [ ] **Step 3: Реализовать чекер**
|
||||
|
||||
`tools/observer-chain-map-checker.mjs`:
|
||||
|
||||
```javascript
|
||||
import { readFileSync } from 'node:fs';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
|
||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
||||
const MD_PATH = join(__dirname, '..', 'docs', 'routing-off-phase.md');
|
||||
const JSON_PATH = join(__dirname, 'observer-chain-map.json');
|
||||
|
||||
/** Extract the set of L-numbers ("L1".."L13") from the routing-off-phase.md table. */
|
||||
export function parseChainsFromMd(md) {
|
||||
const set = new Set();
|
||||
for (const line of md.split(/\r?\n/)) {
|
||||
const m = /^\|\s*(L\d+)\s*\|/.exec(line.trim());
|
||||
if (m) set.add(m[1]);
|
||||
}
|
||||
return set;
|
||||
}
|
||||
|
||||
/** Compare JSON L-numbers against the md set, both directions. */
|
||||
export function checkSync(jsonMap, mdSet) {
|
||||
const jsonSet = new Set();
|
||||
for (const [node, chains] of Object.entries(jsonMap)) {
|
||||
if (node === '_note') continue;
|
||||
if (Array.isArray(chains)) for (const c of chains) jsonSet.add(c);
|
||||
}
|
||||
const jsonOnly = [...jsonSet].filter((c) => !mdSet.has(c)); // ссылки на несуществующие L
|
||||
const mdOnly = [...mdSet].filter((c) => !jsonSet.has(c)); // потерянные цепочки
|
||||
return { ok: jsonOnly.length === 0 && mdOnly.length === 0, jsonOnly, mdOnly };
|
||||
}
|
||||
|
||||
/** CLI entry — exit 1 on drift with a human-readable message. */
|
||||
function main() {
|
||||
const md = readFileSync(MD_PATH, 'utf8');
|
||||
const jsonMap = JSON.parse(readFileSync(JSON_PATH, 'utf8'));
|
||||
const mdSet = parseChainsFromMd(md);
|
||||
if (mdSet.size === 0) {
|
||||
console.error('[chain-map-checker] не нашёл ни одной L-строки в routing-off-phase.md — формат таблицы изменился?');
|
||||
process.exit(1);
|
||||
}
|
||||
const res = checkSync(jsonMap, mdSet);
|
||||
if (res.ok) {
|
||||
console.log(`[chain-map-checker] OK — ${mdSet.size} chains in sync`);
|
||||
process.exit(0);
|
||||
}
|
||||
console.error('[chain-map-checker] дрейф маппинга chain-map <-> routing-off-phase.md:');
|
||||
if (res.jsonOnly.length) console.error(` JSON ссылается на отсутствующие в .md цепочки: ${res.jsonOnly.join(', ')}`);
|
||||
if (res.mdOnly.length) console.error(` В .md есть цепочки без записи в JSON: ${res.mdOnly.join(', ')} — добавьте узлы в tools/observer-chain-map.json`);
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
if (process.argv[1]?.endsWith('observer-chain-map-checker.mjs')) {
|
||||
main();
|
||||
}
|
||||
```
|
||||
|
||||
- [ ] **Step 4: Запустить — убедиться, что проходит**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-chain-map-checker.test.mjs`
|
||||
Expected: PASS (4 tests).
|
||||
|
||||
- [ ] **Step 5: Smoke CLI на реальных файлах**
|
||||
|
||||
Run: `node tools/observer-chain-map-checker.mjs`
|
||||
Expected: `[chain-map-checker] OK — 13 chains in sync` (exit 0). Если FAIL — привести JSON (Task 1) в соответствие с реальной таблицей `.md` (это раскроет реальные расхождения первого маппинга).
|
||||
|
||||
- [ ] **Step 6: Commit**
|
||||
|
||||
```bash
|
||||
git add tools/observer-chain-map-checker.mjs tools/observer-chain-map-checker.test.mjs
|
||||
git commit -m "feat(observer): C6 chain-map-checker (JSON vs routing-off-phase.md sync)"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 4: lefthook job 16 + red-green smoke
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `lefthook.yml` (после job 15 observer-coverage-checker)
|
||||
|
||||
- [ ] **Step 1: Добавить job**
|
||||
|
||||
В `lefthook.yml`, в секцию pre-commit после job 15:
|
||||
|
||||
```yaml
|
||||
# 16. observer-chain-map-checker — brain governance C6 (chain attribution).
|
||||
# Сверяет tools/observer-chain-map.json с таблицей L1-L13 в
|
||||
# docs/routing-off-phase.md. Падает при дрейфе (несуществующая L в JSON
|
||||
# или потерянная цепочка из .md).
|
||||
- name: observer-chain-map-checker
|
||||
run: node tools/observer-chain-map-checker.mjs
|
||||
fail_text: |
|
||||
observer-chain-map-checker: дрейф chain-map <-> routing-off-phase.md.
|
||||
Обновите tools/observer-chain-map.json под таблицу L1-L13.
|
||||
```
|
||||
|
||||
NB: точный синтаксис (`fail_text` vs `interactive` vs `|| true`) скопировать с соседних observer-job'ов (11–15) — формат должен совпасть.
|
||||
|
||||
- [ ] **Step 2: Red-green smoke**
|
||||
|
||||
Run (намеренная рассинхронизация):
|
||||
|
||||
```bash
|
||||
# временно добавить несуществующую цепочку в JSON
|
||||
node -e "const f='tools/observer-chain-map.json';const fs=require('fs');const j=JSON.parse(fs.readFileSync(f,'utf8'));j['__test_drift__']=['L99'];fs.writeFileSync(f,JSON.stringify(j,null,2));"
|
||||
node tools/observer-chain-map-checker.mjs; echo "exit=$?"
|
||||
```
|
||||
|
||||
Expected: exit=1, сообщение про `L99`.
|
||||
|
||||
```bash
|
||||
# откатить
|
||||
git checkout tools/observer-chain-map.json
|
||||
node tools/observer-chain-map-checker.mjs; echo "exit=$?"
|
||||
```
|
||||
|
||||
Expected: exit=0, `OK — 13 chains in sync`.
|
||||
|
||||
- [ ] **Step 3: Commit**
|
||||
|
||||
```bash
|
||||
git add lefthook.yml
|
||||
git commit -m "chore(lefthook): wire C6 observer-chain-map-checker (job 16)"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 5: Ретрофилл существующих эпизодов
|
||||
|
||||
**Files:**
|
||||
|
||||
- Create: `tools/observer-retrofill-chain-ref.mjs`
|
||||
- Test: `tools/observer-retrofill-chain-ref.test.mjs`
|
||||
|
||||
- [ ] **Step 1: Написать failing-тест**
|
||||
|
||||
`tools/observer-retrofill-chain-ref.test.mjs`:
|
||||
|
||||
```javascript
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { retrofillLine } from './observer-retrofill-chain-ref.mjs';
|
||||
import { loadChainMap } from './observer-chain-detector.mjs';
|
||||
|
||||
const map = loadChainMap();
|
||||
|
||||
describe('retrofillLine', () => {
|
||||
it('adds chain_ref to a v2 episode with a known node', () => {
|
||||
const ep = { schema_version: 2, primary_rationale: { node_chosen: 'billing-audit' } };
|
||||
const out = retrofillLine(ep, map);
|
||||
expect(out.primary_rationale.chain_ref).toEqual(['L13']);
|
||||
});
|
||||
|
||||
it('sets chain_ref null for a direct v2 episode', () => {
|
||||
const ep = { schema_version: 2, primary_rationale: { node_chosen: 'direct' } };
|
||||
expect(retrofillLine(ep, map).primary_rationale.chain_ref).toBeNull();
|
||||
});
|
||||
|
||||
it('is idempotent — does not overwrite existing chain_ref', () => {
|
||||
const ep = { schema_version: 2, primary_rationale: { node_chosen: 'direct', chain_ref: ['L1'] } };
|
||||
expect(retrofillLine(ep, map).primary_rationale.chain_ref).toEqual(['L1']);
|
||||
});
|
||||
|
||||
it('skips v1 episodes (no schema_version 2)', () => {
|
||||
const ep = { foo: 'bar' };
|
||||
expect(retrofillLine(ep, map)).toEqual({ foo: 'bar' });
|
||||
});
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — убедиться, что падает**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-retrofill-chain-ref.test.mjs`
|
||||
Expected: FAIL — module not found.
|
||||
|
||||
- [ ] **Step 3: Реализовать**
|
||||
|
||||
`tools/observer-retrofill-chain-ref.mjs`:
|
||||
|
||||
```javascript
|
||||
import { readFileSync, writeFileSync, renameSync, readdirSync } from 'node:fs';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
import { loadChainMap, chainsFor } from './observer-chain-detector.mjs';
|
||||
|
||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
||||
const OBS_DIR = join(__dirname, '..', 'docs', 'observer');
|
||||
|
||||
/** Add chain_ref to a single parsed episode object (pure). Idempotent. */
|
||||
export function retrofillLine(ep, map) {
|
||||
if (!ep || ep.schema_version !== 2 || !ep.primary_rationale) return ep;
|
||||
if ('chain_ref' in ep.primary_rationale) return ep; // idempotent
|
||||
ep.primary_rationale.chain_ref = chainsFor(ep.primary_rationale.node_chosen, map);
|
||||
return ep;
|
||||
}
|
||||
|
||||
/** Process one JSONL file atomically (tmp + rename). Returns {changed, total}. */
|
||||
export function retrofillFile(path, map, { dryRun = false } = {}) {
|
||||
const lines = readFileSync(path, 'utf8').split(/\r?\n/);
|
||||
let changed = 0, total = 0;
|
||||
const out = lines.map((line) => {
|
||||
if (!line.trim()) return line;
|
||||
total++;
|
||||
const ep = JSON.parse(line);
|
||||
const before = ep.primary_rationale && 'chain_ref' in ep.primary_rationale;
|
||||
const next = retrofillLine(ep, map);
|
||||
const after = next.primary_rationale && 'chain_ref' in next.primary_rationale;
|
||||
if (!before && after) changed++;
|
||||
return JSON.stringify(next);
|
||||
});
|
||||
if (!dryRun && changed > 0) {
|
||||
const tmp = `${path}.tmp`;
|
||||
writeFileSync(tmp, out.join('\n'), 'utf8');
|
||||
renameSync(tmp, path);
|
||||
}
|
||||
return { changed, total };
|
||||
}
|
||||
|
||||
function main() {
|
||||
const dryRun = process.argv.includes('--dry-run');
|
||||
const map = loadChainMap();
|
||||
const files = readdirSync(OBS_DIR).filter((f) => /^episodes-\d{4}-\d{2}\.jsonl$/.test(f));
|
||||
for (const f of files) {
|
||||
const { changed, total } = retrofillFile(join(OBS_DIR, f), map, { dryRun });
|
||||
console.log(`${dryRun ? '[dry-run] ' : ''}${f}: ${changed}/${total} lines would get chain_ref`);
|
||||
}
|
||||
}
|
||||
|
||||
if (process.argv[1]?.endsWith('observer-retrofill-chain-ref.mjs')) main();
|
||||
```
|
||||
|
||||
- [ ] **Step 4: Запустить — убедиться, что проходит**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/observer-retrofill-chain-ref.test.mjs`
|
||||
Expected: PASS (4 tests).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add tools/observer-retrofill-chain-ref.mjs tools/observer-retrofill-chain-ref.test.mjs
|
||||
git commit -m "feat(observer): one-shot chain_ref retrofill script (idempotent, atomic)"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 6: Агрегация в brain-retro-analyzer
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `tools/brain-retro-analyzer.mjs` (в формирование `factorMatrix`)
|
||||
- Test: `tools/brain-retro-analyzer.test.mjs`
|
||||
|
||||
- [ ] **Step 1: Написать failing-тест**
|
||||
|
||||
Добавить в `tools/brain-retro-analyzer.test.mjs`:
|
||||
|
||||
```javascript
|
||||
it('aggregates chain_ref into factorMatrix (multi-chain counted in each)', () => {
|
||||
const episodes = [
|
||||
{ schema_version: 2, primary_rationale: { node_chosen: 'discovery-interview', chain_ref: ['L1','L2'] } /* + поля, нужные analyzer */ },
|
||||
{ schema_version: 2, primary_rationale: { node_chosen: 'direct', chain_ref: null } },
|
||||
];
|
||||
const result = analyze(episodes); // имя функции взять из существующего теста
|
||||
expect(result.factorMatrix.chain_ref.L1).toBeDefined();
|
||||
expect(result.factorMatrix.chain_ref.L2).toBeDefined();
|
||||
expect(result.factorMatrix.chain_ref.null).toBeDefined();
|
||||
});
|
||||
```
|
||||
|
||||
NB: имена `analyze` / shape входа подогнать под существующий тест файла (там уже есть фикстуры эпизодов — переиспользовать форму).
|
||||
|
||||
- [ ] **Step 2: Запустить — убедиться, что падает**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/brain-retro-analyzer.test.mjs -t chain_ref`
|
||||
Expected: FAIL — `factorMatrix.chain_ref` undefined.
|
||||
|
||||
- [ ] **Step 3: Реализовать**
|
||||
|
||||
В `tools/brain-retro-analyzer.mjs`, где строится `factorMatrix`, добавить ось `chain_ref`. Multi-chain эпизод инкрементит каждую L; `null` → ключ `"null"`:
|
||||
|
||||
```javascript
|
||||
// внутри построения factorMatrix, по аналогии с другими осями:
|
||||
matrix.chain_ref = {};
|
||||
for (const ep of v2Episodes) {
|
||||
const cr = ep.primary_rationale?.chain_ref;
|
||||
const outcome = ep._inferredOutcome ?? 'unknown';
|
||||
const keys = Array.isArray(cr) && cr.length ? cr : ['null'];
|
||||
for (const k of keys) {
|
||||
matrix.chain_ref[k] = matrix.chain_ref[k] || {};
|
||||
matrix.chain_ref[k][outcome] = (matrix.chain_ref[k][outcome] || 0) + 1;
|
||||
}
|
||||
}
|
||||
```
|
||||
|
||||
NB: точные имена переменных (`matrix`, `v2Episodes`, поле inferred outcome) взять из реального кода — он уже строит другие оси, скопировать паттерн.
|
||||
|
||||
- [ ] **Step 4: Запустить — убедиться, что проходит**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/brain-retro-analyzer.test.mjs`
|
||||
Expected: PASS (включая существующие тесты).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add tools/brain-retro-analyzer.mjs tools/brain-retro-analyzer.test.mjs
|
||||
git commit -m "feat(brain-retro): aggregate chain_ref into factorMatrix"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 7: STATUS.md строка C6
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `tools/status-md-generator.mjs`
|
||||
- Test: `tools/status-md-generator.test.mjs`
|
||||
|
||||
- [ ] **Step 1: Написать failing-тест**
|
||||
|
||||
Добавить в `tools/status-md-generator.test.mjs`:
|
||||
|
||||
```javascript
|
||||
it('includes a C6 chain-map row', () => {
|
||||
const md = generateStatus(/* фикстура как в существующих тестах */);
|
||||
expect(md).toMatch(/C6 Chain map sync/);
|
||||
});
|
||||
```
|
||||
|
||||
NB: имя `generateStatus` и форму входа взять из существующего теста.
|
||||
|
||||
- [ ] **Step 2: Запустить — убедиться, что падает**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/status-md-generator.test.mjs -t C6`
|
||||
Expected: FAIL.
|
||||
|
||||
- [ ] **Step 3: Реализовать**
|
||||
|
||||
В `tools/status-md-generator.mjs`, в таблицу контролёров добавить строку C6 (по аналогии с C5). Поскольку чекер запускается в lefthook, статус в STATUS.md — информационный: вызвать `checkSync` через импорт и отразить ok/drift:
|
||||
|
||||
```javascript
|
||||
import { parseChainsFromMd, checkSync } from './observer-chain-map-checker.mjs';
|
||||
// ... при сборке таблицы контролёров:
|
||||
let c6 = '✅';
|
||||
let c6detail = '[chain-map-checker] OK';
|
||||
try {
|
||||
const md = readFileSync(join(OBS_ROOT, '..', 'routing-off-phase.md'), 'utf8'); // путь привести к реальному
|
||||
const jsonMap = JSON.parse(readFileSync(CHAIN_MAP_PATH, 'utf8'));
|
||||
const res = checkSync(jsonMap, parseChainsFromMd(md));
|
||||
if (!res.ok) { c6 = '🔴'; c6detail = `drift: ${[...res.jsonOnly, ...res.mdOnly].join(', ')}`; }
|
||||
} catch (e) { c6 = '⚠️'; c6detail = `checker error: ${e.message}`; }
|
||||
// добавить строку в таблицу: | C6 Chain map sync | ${c6} | ${c6detail} |
|
||||
```
|
||||
|
||||
NB: пути (`OBS_ROOT`, `CHAIN_MAP_PATH`) и формат строки таблицы взять из реального кода генератора.
|
||||
|
||||
- [ ] **Step 4: Запустить — убедиться, что проходит**
|
||||
|
||||
Run: `cd app && npx vitest run --config vitest.config.tools.mjs ../tools/status-md-generator.test.mjs`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add tools/status-md-generator.mjs tools/status-md-generator.test.mjs
|
||||
git commit -m "feat(status-md): surface C6 chain-map sync row"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 8: aggregation-template секция hit rate
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `.claude/skills/brain-retro/references/aggregation-template.md`
|
||||
|
||||
- [ ] **Step 1: Заменить пустую секцию**
|
||||
|
||||
Найти `## Canonical chains L1–L12 hit rate` и заменить на:
|
||||
|
||||
```markdown
|
||||
## Canonical chains L1–L13 hit rate (from analyzer `factorMatrix.chain_ref`)
|
||||
|
||||
| chain | times | outcome split | notes |
|
||||
|---|---|---|---|
|
||||
|
||||
Каждый узел может входить в несколько L (multi-chain эпизод засчитан в каждую).
|
||||
`null` = эпизоды вне цепочек (direct + узлы вне L1-L13) — **не проблема** per
|
||||
`memory/feedback_brain_unused_tools_not_problem`.
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Commit**
|
||||
|
||||
```bash
|
||||
git add .claude/skills/brain-retro/references/aggregation-template.md
|
||||
git commit -m "docs(brain-retro): fill L1-L13 hit rate template section"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 9: Финальная регрессия + ретрофилл + verification
|
||||
|
||||
**Files:** нет правок кода.
|
||||
|
||||
- [ ] **Step 1: Полная регрессия tools**
|
||||
|
||||
Run: `npm run test:tools`
|
||||
Expected: PASS, число ≥ baseline (Task 0 Step 3) + новые тесты (≈ +16). 0 сломанных существующих.
|
||||
|
||||
- [ ] **Step 2: Ретрофилл dry-run**
|
||||
|
||||
Run: `node tools/observer-retrofill-chain-ref.mjs --dry-run`
|
||||
Expected: для `episodes-2026-05.jsonl` — N/M строк получат chain_ref (N = число v2-эпизодов с known node).
|
||||
|
||||
- [ ] **Step 3: Ретрофилл реальный + идемпотентность**
|
||||
|
||||
Run:
|
||||
|
||||
```bash
|
||||
node tools/observer-retrofill-chain-ref.mjs
|
||||
node tools/observer-retrofill-chain-ref.mjs
|
||||
```
|
||||
|
||||
Expected: первый — `changed > 0`; второй — `0/M` (идемпотентно).
|
||||
|
||||
- [ ] **Step 4: Commit ретрофилла данных**
|
||||
|
||||
```bash
|
||||
git add docs/observer/episodes-2026-05.jsonl
|
||||
git commit -m "chore(observer): retrofill chain_ref on existing May episodes"
|
||||
```
|
||||
|
||||
- [ ] **Step 5: Verification-before-completion**
|
||||
|
||||
Использовать `superpowers:verification-before-completion`. Проверить acceptance criteria spec §13:
|
||||
|
||||
- 6 новых файлов созданы, тесты зелёные;
|
||||
- lefthook job 16 red-green работает (Task 4);
|
||||
- ретрофилл идемпотентен (Step 3);
|
||||
- `node tools/observer-chain-map-checker.mjs` → OK;
|
||||
- STATUS.md содержит строку C6.
|
||||
|
||||
- [ ] **Step 6: Финальный push**
|
||||
|
||||
```bash
|
||||
git push origin feat/observer-chain-attribution:main
|
||||
```
|
||||
|
||||
NB: gitleaks pre-push + полная регрессия по политике §15; push-паттерн `<ветка>:main` (FF).
|
||||
|
||||
---
|
||||
|
||||
## Self-Review (выполнено при написании плана)
|
||||
|
||||
**Spec coverage:** §4 архитектура → Tasks 1–8; §5 компоненты 1-8 → Tasks 1–8 (по одному); §6 потоки A/B/C/D → Tasks 2/4/6/5; §7 ошибки → defensive try/catch в Task 2 Step 3 (битый JSON) + Task 3 (формат .md) + Task 5 (идемпотентность); §8 тесты → Tasks 1/3/5/6/7; §10 порядок → Task 0; §13 acceptance → Task 9 Step 5. Все секции покрыты.
|
||||
|
||||
**Placeholder scan:** код приведён во всех code-степах. Места «NB: имя взять из существующего теста» — намеренные адаптационные точки (фабрики транскриптов и имена функций analyzer/generateStatus зависят от финальной базы после epic-плана и не могут быть зафиксированы заранее), не placeholder-логика.
|
||||
|
||||
**Type consistency:** `loadChainMap()`/`chainsFor(node, map)` — единые сигнатуры в Tasks 1/2/5/6. `parseChainsFromMd()`/`checkSync(jsonMap, mdSet)` → `{ ok, jsonOnly, mdOnly }` — единые в Tasks 3/4/7. `retrofillLine(ep, map)`/`retrofillFile(path, map, opts)` — Task 5. `chain_ref` форма (массив \| null) консистентна везде.
|
||||
|
||||
**Раскрытые при детализации уточнения** (в разделе «Уточнения к spec» вверху): тесты в `tools/` не `tests/`; C6 по L-номерам не по именам узлов; врезка по grep-маркеру.
|
||||
File diff suppressed because it is too large
Load Diff
@@ -0,0 +1,746 @@
|
||||
# Project delete + source dedup + human errors — Implementation Plan
|
||||
|
||||
> **For agentic workers:** REQUIRED SUB-SKILL: Use superpowers:subagent-driven-development (recommended) or superpowers:executing-plans to implement this plan task-by-task. Steps use checkbox (`- [ ]`) syntax for tracking.
|
||||
|
||||
**Goal:** Заменить архивацию проектов настоящим удалением (с защитой по сделкам и корректной обработкой шеринга у поставщика), добавить дедуп источника внутри клиента и заменить сырые SQL-ошибки человеческими сообщениями.
|
||||
|
||||
**Architecture:** Бэкенд — вся логика в `ProjectService` (create/update/delete) + новый `DeleteSupplierProjectJob` для удаления/пере-синка донора у поставщика с учётом шеринга + глобальный handler `QueryException`. Архивация (`archived_at`) убирается полностью (миграция-снос колонки). Фронтенд — «Архивировать»→«Удалить», убрать фильтр «Архивные».
|
||||
|
||||
**Tech Stack:** PHP 8.3 / Laravel 13, Pest 4; Vue 3 + Vuetify 3 + Pinia, Vitest; PostgreSQL 16.
|
||||
|
||||
**Спека:** `docs/superpowers/specs/2026-05-21-project-delete-dedup-errors-design.md`
|
||||
|
||||
**Команды (из `app/`):** `C:/tools/php83/php.exe artisan test --filter=<name>`, `composer pint`, `composer stan`, `npm run test:vue`.
|
||||
|
||||
**Ключевые факты (разведка):**
|
||||
|
||||
- `Project` — БЕЗ SoftDeletes → `$project->delete()` = hard delete. У `projects` нет `deleted_at` (только `archived_at` custom + `is_active`).
|
||||
- `Deal` — С SoftDeletes (`deals.deleted_at`). Guard считает ВСЕ сделки через `DB::table('deals')` (минует scope).
|
||||
- `deals.project_id` без FK; cascade на `projects(id)` только у служебных таблиц.
|
||||
- `supplier_projects.supplier_external_id` VARCHAR(64) — id донора у поставщика (числовой; cast к int для `deleteProject(int)`).
|
||||
- `project_supplier_links` — pivot (project_id, supplier_project_id), ON DELETE CASCADE на оба.
|
||||
- Источник: `signal_identifier` (call/site) либо `sms_senders[]`+`sms_keyword` (sms).
|
||||
|
||||
---
|
||||
|
||||
## Task 1: Дедуп источника + имени в ProjectService::create()
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `app/app/Services/Project/ProjectService.php` (метод `create`, +helper `assertSourceUnique`)
|
||||
- Test: `app/tests/Feature/Project/ProjectCreateDedupTest.php` (create)
|
||||
|
||||
- [ ] **Step 1: Написать падающие тесты**
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
|
||||
beforeEach(function () {
|
||||
$this->tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
});
|
||||
|
||||
function makeCall(array $over = []): array {
|
||||
return array_merge([
|
||||
'name' => 'Проект A', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
||||
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
||||
], $over);
|
||||
}
|
||||
|
||||
it('blocks duplicate source within tenant with human message', function () {
|
||||
app(\App\Services\Project\ProjectService::class)->create($this->tenant, makeCall());
|
||||
expect(fn () => app(\App\Services\Project\ProjectService::class)
|
||||
->create($this->tenant, makeCall(['name' => 'Проект B'])))
|
||||
->toThrow(\Illuminate\Http\Exceptions\HttpResponseException::class);
|
||||
});
|
||||
|
||||
it('allows same source for a different tenant (sharing)', function () {
|
||||
$other = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
app(\App\Services\Project\ProjectService::class)->create($this->tenant, makeCall());
|
||||
$p = app(\App\Services\Project\ProjectService::class)->create($other, makeCall(['name' => 'Проект B']));
|
||||
expect($p)->toBeInstanceOf(Project::class);
|
||||
});
|
||||
|
||||
it('blocks duplicate name within tenant with human message (not SQL)', function () {
|
||||
app(\App\Services\Project\ProjectService::class)->create($this->tenant, makeCall());
|
||||
try {
|
||||
app(\App\Services\Project\ProjectService::class)
|
||||
->create($this->tenant, makeCall(['name' => 'Проект A', 'signal_identifier' => '79992220000']));
|
||||
$this->fail('expected HttpResponseException');
|
||||
} catch (\Illuminate\Http\Exceptions\HttpResponseException $e) {
|
||||
$body = $e->getResponse()->getData(true);
|
||||
expect($body['errors']['name'][0] ?? '')->not->toContain('SQLSTATE');
|
||||
}
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — убедиться что падают**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=ProjectCreateDedupTest`
|
||||
Expected: FAIL (дедупа нет — второй create проходит / бьёт DB-констрейнт).
|
||||
|
||||
- [ ] **Step 3: Реализация в `ProjectService::create()`**
|
||||
|
||||
В начало `create()` (после получения `$tenant`, до `Project::create`) добавить вызовы и helper'ы:
|
||||
|
||||
```php
|
||||
// перед Project::create($data):
|
||||
$this->assertNameUnique($tenant->id, (string) $data['name']);
|
||||
$this->assertSourceUnique($tenant->id, $data);
|
||||
```
|
||||
|
||||
Добавить методы в класс:
|
||||
|
||||
```php
|
||||
private function assertNameUnique(int $tenantId, string $name, ?int $exceptId = null): void
|
||||
{
|
||||
$q = Project::where('tenant_id', $tenantId)->where('name', $name);
|
||||
if ($exceptId !== null) {
|
||||
$q->where('id', '!=', $exceptId);
|
||||
}
|
||||
if ($q->exists()) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'errors' => ['name' => ['Проект с таким названием у вас уже есть. Выберите другое название.']],
|
||||
], 422));
|
||||
}
|
||||
}
|
||||
|
||||
/** @param array<string,mixed> $data */
|
||||
private function assertSourceUnique(int $tenantId, array $data, ?int $exceptId = null): void
|
||||
{
|
||||
$signalType = $data['signal_type'] ?? null;
|
||||
$q = Project::where('tenant_id', $tenantId)->where('signal_type', $signalType);
|
||||
if ($exceptId !== null) {
|
||||
$q->where('id', '!=', $exceptId);
|
||||
}
|
||||
|
||||
if (in_array($signalType, ['call', 'site'], true)) {
|
||||
$identifier = (string) ($data['signal_identifier'] ?? '');
|
||||
if ($identifier === '') {
|
||||
return;
|
||||
}
|
||||
$q->where('signal_identifier', $identifier);
|
||||
} elseif ($signalType === 'sms') {
|
||||
$senders = (array) ($data['sms_senders'] ?? []);
|
||||
$norm = collect($senders)->map(fn ($s) => mb_strtolower(trim((string) $s)))->sort()->values()->all();
|
||||
if ($norm === []) {
|
||||
return;
|
||||
}
|
||||
// sms-источник идентичен, если совпадают набор отправителей и ключевое слово.
|
||||
$keyword = $data['sms_keyword'] ?? null;
|
||||
$q->where('sms_keyword', $keyword)
|
||||
->whereJsonContains('sms_senders', $norm)
|
||||
->whereRaw('jsonb_array_length(sms_senders::jsonb) = ?', [count($norm)]);
|
||||
} else {
|
||||
return;
|
||||
}
|
||||
|
||||
$existing = $q->first();
|
||||
if ($existing !== null) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'errors' => ['signal_identifier' => ["У вас уже есть проект с этим источником: «{$existing->name}»."]],
|
||||
], 422));
|
||||
}
|
||||
}
|
||||
```
|
||||
|
||||
Убедиться, что в шапке есть `use Illuminate\Http\Exceptions\HttpResponseException;` (уже есть).
|
||||
|
||||
- [ ] **Step 4: Запустить — PASS**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=ProjectCreateDedupTest`
|
||||
Expected: PASS (3 теста).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/app/Services/Project/ProjectService.php app/tests/Feature/Project/ProjectCreateDedupTest.php
|
||||
git commit -m "feat(projects): source+name dedup with human messages on create"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 2: Дедуп источника при смене источника (update)
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `app/app/Services/Project/ProjectService.php` (метод `update`)
|
||||
- Test: `app/tests/Feature/Project/ProjectUpdateDedupTest.php`
|
||||
|
||||
- [ ] **Step 1: Падающий тест**
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
|
||||
it('blocks update that collides source with another project of same tenant', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$svc = app(\App\Services\Project\ProjectService::class);
|
||||
$a = $svc->create($tenant, ['name' => 'A', 'signal_type' => 'call', 'signal_identifier' => '79991110000', 'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31]);
|
||||
$b = $svc->create($tenant, ['name' => 'B', 'signal_type' => 'call', 'signal_identifier' => '79992220000', 'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31]);
|
||||
|
||||
expect(fn () => $svc->update($b, ['signal_identifier' => '79991110000']))
|
||||
->toThrow(\Illuminate\Http\Exceptions\HttpResponseException::class);
|
||||
});
|
||||
|
||||
it('allows update keeping same source on the same project', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$svc = app(\App\Services\Project\ProjectService::class);
|
||||
$a = $svc->create($tenant, ['name' => 'A', 'signal_type' => 'call', 'signal_identifier' => '79991110000', 'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31]);
|
||||
$updated = $svc->update($a, ['signal_identifier' => '79991110000', 'daily_limit_target' => 7]);
|
||||
expect($updated->daily_limit_target)->toBe(7);
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — FAIL**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=ProjectUpdateDedupTest`
|
||||
Expected: FAIL (первый кейс не бросает).
|
||||
|
||||
- [ ] **Step 3: Реализация в `update()`**
|
||||
|
||||
После блока immutable-unset и до `$project->update($data)` добавить:
|
||||
|
||||
```php
|
||||
if (array_key_exists('signal_identifier', $data) || array_key_exists('sms_senders', $data) || array_key_exists('sms_keyword', $data)) {
|
||||
$this->assertSourceUnique($project->tenant_id, array_merge([
|
||||
'signal_type' => $project->signal_type,
|
||||
'signal_identifier' => $project->signal_identifier,
|
||||
'sms_senders' => $project->sms_senders,
|
||||
'sms_keyword' => $project->sms_keyword,
|
||||
], $data), exceptId: $project->id);
|
||||
}
|
||||
if (array_key_exists('name', $data)) {
|
||||
$this->assertNameUnique($project->tenant_id, (string) $data['name'], exceptId: $project->id);
|
||||
}
|
||||
```
|
||||
|
||||
- [ ] **Step 4: Запустить — PASS**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=ProjectUpdateDedupTest`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/app/Services/Project/ProjectService.php app/tests/Feature/Project/ProjectUpdateDedupTest.php
|
||||
git commit -m "feat(projects): source+name dedup on update"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 3: Глобальный handler QueryException (никакого SQL в UI)
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `app/bootstrap/app.php` (`withExceptions`)
|
||||
- Test: `app/tests/Feature/Project/QueryExceptionRenderTest.php`
|
||||
|
||||
- [ ] **Step 1: Падающий тест** (бьём прямой DB-констрейнт мимо app-проверок — два проекта с одинаковым именем через прямой insert невозможно из API после Task 1, поэтому тестируем рендер handler'а на искусственном маршруте)
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
use Illuminate\Support\Facades\Route;
|
||||
|
||||
it('renders QueryException as human JSON message, not SQLSTATE', function () {
|
||||
Route::get('/_test/boom-query', function () {
|
||||
throw new \Illuminate\Database\QueryException('pgsql', 'SELECT 1', [], new \Exception('SQLSTATE[23505] duplicate key'));
|
||||
});
|
||||
|
||||
$res = $this->getJson('/_test/boom-query');
|
||||
$res->assertStatus(422);
|
||||
expect($res->json('message'))->not->toContain('SQLSTATE');
|
||||
expect($res->json('message'))->toContain('Не удалось');
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — FAIL**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=QueryExceptionRenderTest`
|
||||
Expected: FAIL (по умолчанию 500 + SQL-текст в debug).
|
||||
|
||||
- [ ] **Step 3: Реализация в `bootstrap/app.php`**
|
||||
|
||||
Заменить тело `->withExceptions(function (Exceptions $exceptions): void { // });` на:
|
||||
|
||||
```php
|
||||
->withExceptions(function (Exceptions $exceptions): void {
|
||||
$exceptions->render(function (\Illuminate\Database\QueryException $e, \Illuminate\Http\Request $request) {
|
||||
\Illuminate\Support\Facades\Log::error('db.query_exception', [
|
||||
'message' => $e->getMessage(),
|
||||
'sql' => $e->getSql(),
|
||||
'path' => $request->path(),
|
||||
]);
|
||||
if ($request->expectsJson()) {
|
||||
return response()->json([
|
||||
'message' => 'Не удалось сохранить. Проверьте данные или попробуйте ещё раз.',
|
||||
], 422);
|
||||
}
|
||||
|
||||
return null; // дефолтный рендер для не-JSON
|
||||
});
|
||||
})
|
||||
```
|
||||
|
||||
- [ ] **Step 4: Запустить — PASS**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=QueryExceptionRenderTest`
|
||||
Expected: PASS.
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/bootstrap/app.php app/tests/Feature/Project/QueryExceptionRenderTest.php
|
||||
git commit -m "feat(errors): global QueryException handler returns human message"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 4: ProjectService::delete() с guard по сделкам (+ снос archive())
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `app/app/Services/Project/ProjectService.php` (+`delete()`, −`archive()`, bulk `archive`→`delete`)
|
||||
- Modify: `app/app/Http/Controllers/Api/ProjectController.php` (`destroy`→`delete`)
|
||||
- Modify: `app/app/Http/Requests/BulkProjectActionRequest.php` (`archive`→`delete`)
|
||||
- Test: `app/tests/Feature/Project/ProjectDeleteTest.php`
|
||||
|
||||
- [ ] **Step 1: Падающие тесты**
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Models\Project;
|
||||
use App\Models\Tenant;
|
||||
use App\Models\User;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
|
||||
it('hard-deletes an empty project', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$project = app(\App\Services\Project\ProjectService::class)->create($tenant, [
|
||||
'name' => 'Empty', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
||||
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
||||
]);
|
||||
|
||||
app(\App\Services\Project\ProjectService::class)->delete($project);
|
||||
|
||||
expect(Project::find($project->id))->toBeNull();
|
||||
});
|
||||
|
||||
it('blocks delete when project has deals', function () {
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$project = app(\App\Services\Project\ProjectService::class)->create($tenant, [
|
||||
'name' => 'WithDeals', 'signal_type' => 'call', 'signal_identifier' => '79991110000',
|
||||
'daily_limit_target' => 5, 'regions' => [], 'delivery_days_mask' => 31,
|
||||
]);
|
||||
DB::table('deals')->insert([
|
||||
'tenant_id' => $tenant->id, 'project_id' => $project->id, 'phone' => '79990001122',
|
||||
'status' => 'new', 'received_at' => now(), 'created_at' => now(),
|
||||
]);
|
||||
|
||||
expect(fn () => app(\App\Services\Project\ProjectService::class)->delete($project))
|
||||
->toThrow(\Illuminate\Http\Exceptions\HttpResponseException::class);
|
||||
expect(Project::find($project->id))->not->toBeNull();
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — FAIL**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=ProjectDeleteTest`
|
||||
Expected: FAIL (метода `delete()` нет).
|
||||
|
||||
- [ ] **Step 3: Реализация**
|
||||
|
||||
В `ProjectService` добавить `delete()` и удалить `archive()`:
|
||||
|
||||
```php
|
||||
public function delete(Project $project): void
|
||||
{
|
||||
$hasDeals = DB::table('deals')->where('project_id', $project->id)->exists();
|
||||
if ($hasDeals) {
|
||||
throw new HttpResponseException(response()->json([
|
||||
'errors' => ['project' => ['Нельзя удалить проект: по нему есть сделки. Поставьте приём на паузу, чтобы скрыть проект из работы.']],
|
||||
], 422));
|
||||
}
|
||||
|
||||
// Доноров фиксируем ДО удаления — pivot уйдёт каскадом.
|
||||
$supplierProjectIds = DB::table('project_supplier_links')
|
||||
->where('project_id', $project->id)
|
||||
->pluck('supplier_project_id')
|
||||
->all();
|
||||
|
||||
$project->delete(); // hard delete (Project без SoftDeletes); cascade чистит pivot + служебные.
|
||||
|
||||
if ($supplierProjectIds !== []) {
|
||||
\App\Jobs\Supplier\DeleteSupplierProjectJob::dispatch(array_map('intval', $supplierProjectIds));
|
||||
}
|
||||
}
|
||||
```
|
||||
|
||||
Добавить `use Illuminate\Support\Facades\DB;` в шапку. Удалить метод `archive()`. В `bulkAction()` строку `'archive' => ...` заменить на:
|
||||
|
||||
```php
|
||||
'delete' => $this->bulkDelete($query),
|
||||
```
|
||||
|
||||
Добавить `bulkDelete` (guard per-project, не роняет батч):
|
||||
|
||||
```php
|
||||
private function bulkDelete($query): array
|
||||
{
|
||||
$projects = (clone $query)->get(['id']);
|
||||
$deleted = 0; $skipped = [];
|
||||
foreach ($projects as $p) {
|
||||
$model = Project::find($p->id);
|
||||
if ($model === null) { continue; }
|
||||
try {
|
||||
$this->delete($model);
|
||||
$deleted++;
|
||||
} catch (HttpResponseException) {
|
||||
$skipped[] = ['id' => $p->id, 'reason' => 'has_deals'];
|
||||
}
|
||||
}
|
||||
|
||||
return ['updated' => $deleted, 'skipped' => $skipped, 'warnings' => []];
|
||||
}
|
||||
```
|
||||
|
||||
В `update()` убрать из unset строку `$data['archived_at'],` (колонка уходит в Task 6). В `resolveBulkScope()` ветку match `'archived' => ...` удалить; `'active'`/`'paused'` оставить без `whereNull('archived_at')` (см. Task 6).
|
||||
|
||||
В `ProjectController::destroy()` заменить `$this->projects->archive($project);` на `$this->projects->delete($project);` и docblock «soft-archive» → «hard delete (guard по сделкам)».
|
||||
|
||||
В `BulkProjectActionRequest`: в `Rule::in([...])` для action `'archive'` → `'delete'`; убрать `'archived'` из `Rule::in(['active','paused','archived'])`.
|
||||
|
||||
- [ ] **Step 4: Запустить — PASS** + регрессия bulk
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=ProjectDeleteTest`
|
||||
Then: `C:/tools/php83/php.exe artisan test --filter=Project`
|
||||
Expected: PASS; падений по `archive` нет (если есть старые тесты на archive — обновить на delete в этом же шаге).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/app/Services/Project/ProjectService.php app/app/Http/Controllers/Api/ProjectController.php app/app/Http/Requests/BulkProjectActionRequest.php app/tests/Feature/Project/ProjectDeleteTest.php
|
||||
git commit -m "feat(projects): hard delete with deals-guard, replace archive"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 5: DeleteSupplierProjectJob — удаление/пере-синк донора с учётом шеринга
|
||||
|
||||
**Files:**
|
||||
|
||||
- Create: `app/app/Jobs/Supplier/DeleteSupplierProjectJob.php`
|
||||
- Test: `app/tests/Feature/Supplier/DeleteSupplierProjectJobTest.php`
|
||||
|
||||
- [ ] **Step 1: Падающие тесты** (mock `SupplierPortalClient`)
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
use App\Jobs\Supplier\DeleteSupplierProjectJob;
|
||||
use App\Jobs\Supplier\SyncSupplierProjectsJob;
|
||||
use App\Models\Project;
|
||||
use App\Models\SupplierProject;
|
||||
use App\Models\Tenant;
|
||||
use App\Services\Supplier\SupplierPortalClient;
|
||||
use Illuminate\Support\Facades\Bus;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
|
||||
it('deletes donor at supplier when no consumers remain', function () {
|
||||
$sp = SupplierProject::create(['platform' => 'B1', 'signal_type' => 'call', 'unique_key' => '79991110000', 'supplier_external_id' => '555', 'current_limit' => 1]);
|
||||
|
||||
$mock = Mockery::mock(SupplierPortalClient::class);
|
||||
$mock->shouldReceive('deleteProject')->once()->with(555);
|
||||
app()->instance(SupplierPortalClient::class, $mock);
|
||||
|
||||
(new DeleteSupplierProjectJob([$sp->id]))->handle(app(SupplierPortalClient::class));
|
||||
|
||||
expect(SupplierProject::find($sp->id))->toBeNull();
|
||||
});
|
||||
|
||||
it('does NOT delete donor at supplier when other consumers remain; re-syncs', function () {
|
||||
Bus::fake([SyncSupplierProjectsJob::class]);
|
||||
$tenant = Tenant::factory()->create(['balance_leads' => 100]);
|
||||
$sp = SupplierProject::create(['platform' => 'B1', 'signal_type' => 'call', 'unique_key' => '79991110000', 'supplier_external_id' => '555', 'current_limit' => 1]);
|
||||
$other = Project::factory()->create(['tenant_id' => $tenant->id]);
|
||||
DB::table('project_supplier_links')->insert(['project_id' => $other->id, 'supplier_project_id' => $sp->id, 'subject_code' => null]);
|
||||
|
||||
$mock = Mockery::mock(SupplierPortalClient::class);
|
||||
$mock->shouldNotReceive('deleteProject');
|
||||
app()->instance(SupplierPortalClient::class, $mock);
|
||||
|
||||
(new DeleteSupplierProjectJob([$sp->id]))->handle(app(SupplierPortalClient::class));
|
||||
|
||||
expect(SupplierProject::find($sp->id))->not->toBeNull();
|
||||
Bus::assertDispatched(SyncSupplierProjectsJob::class);
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — FAIL**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=DeleteSupplierProjectJobTest`
|
||||
Expected: FAIL (класса нет).
|
||||
|
||||
- [ ] **Step 3: Реализация джоба**
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
namespace App\Jobs\Supplier;
|
||||
|
||||
use App\Models\SupplierProject;
|
||||
use App\Services\Supplier\SupplierPortalClient;
|
||||
use Illuminate\Bus\Queueable;
|
||||
use Illuminate\Contracts\Queue\ShouldQueue;
|
||||
use Illuminate\Foundation\Bus\Dispatchable;
|
||||
use Illuminate\Queue\InteractsWithQueue;
|
||||
use Illuminate\Queue\SerializesModels;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
use Illuminate\Support\Facades\Log;
|
||||
|
||||
/**
|
||||
* Удаление/пере-синк доноров у поставщика после удаления Лидерра-проекта.
|
||||
*
|
||||
* Для каждого supplier_project S (донора), к которому был привязан удалённый проект:
|
||||
* - остались другие потребители (project_supplier_links) → донор нужен другим клиентам:
|
||||
* НЕ удаляем у поставщика, пере-синкаем агрегат (SyncSupplierProjectsJob).
|
||||
* - потребителей не осталось → удаляем у поставщика (deleteProject) + локальную запись S.
|
||||
*
|
||||
* Spec: docs/superpowers/specs/2026-05-21-project-delete-dedup-errors-design.md §Решение 2.
|
||||
*/
|
||||
class DeleteSupplierProjectJob implements ShouldQueue
|
||||
{
|
||||
use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;
|
||||
|
||||
public int $tries = 3;
|
||||
|
||||
public int $backoff = 60;
|
||||
|
||||
public const DB_CONNECTION = 'pgsql_supplier';
|
||||
|
||||
/** @param array<int,int> $supplierProjectIds */
|
||||
public function __construct(public array $supplierProjectIds) {}
|
||||
|
||||
public function handle(SupplierPortalClient $client): void
|
||||
{
|
||||
$needsResync = false;
|
||||
|
||||
foreach ($this->supplierProjectIds as $id) {
|
||||
$sp = SupplierProject::on(self::DB_CONNECTION)->find($id);
|
||||
if ($sp === null) {
|
||||
continue;
|
||||
}
|
||||
|
||||
$remaining = DB::connection(self::DB_CONNECTION)
|
||||
->table('project_supplier_links')
|
||||
->where('supplier_project_id', $id)
|
||||
->count();
|
||||
|
||||
if ($remaining > 0) {
|
||||
$needsResync = true;
|
||||
continue;
|
||||
}
|
||||
|
||||
if ($sp->supplier_external_id !== null && $sp->supplier_external_id !== '') {
|
||||
try {
|
||||
$client->deleteProject((int) $sp->supplier_external_id);
|
||||
} catch (\Throwable $e) {
|
||||
Log::warning('supplier.delete_donor_failed', ['supplier_project_id' => $id, 'error' => $e->getMessage()]);
|
||||
throw $e; // ретрай джоба
|
||||
}
|
||||
}
|
||||
$sp->delete();
|
||||
}
|
||||
|
||||
if ($needsResync) {
|
||||
SyncSupplierProjectsJob::dispatch();
|
||||
}
|
||||
}
|
||||
}
|
||||
```
|
||||
|
||||
- [ ] **Step 4: Запустить — PASS**
|
||||
|
||||
Run: `C:/tools/php83/php.exe artisan test --filter=DeleteSupplierProjectJobTest`
|
||||
Expected: PASS (2 теста).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/app/Jobs/Supplier/DeleteSupplierProjectJob.php app/tests/Feature/Supplier/DeleteSupplierProjectJobTest.php
|
||||
git commit -m "feat(supplier): delete/re-sync donor on project delete respecting sharing"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 6: Снос archived_at — модель/ресурс/синк/дашборд/контроллер + миграция
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `app/app/Models/Project.php` (−scopeArchived, scopeActive, fillable+cast `archived_at`)
|
||||
- Modify: `app/app/Http/Resources/ProjectResource.php` (−`archived_at`)
|
||||
- Modify: `app/app/Http/Controllers/Api/ProjectController.php` (index `status=archived`/`active()`)
|
||||
- Modify: `app/app/Http/Controllers/Api/DashboardController.php` (−`whereNull('archived_at')`)
|
||||
- Modify: `app/app/Jobs/Supplier/SyncSupplierProjectsJob.php` (−`whereNull('archived_at')`)
|
||||
- Create: `app/database/migrations/2026_05_21_000000_drop_projects_archived_at.php`
|
||||
- Modify: `db/schema.sql` (убрать строку `archived_at` из projects + header v8.27) + `db/CHANGELOG_schema.md`
|
||||
- Test: запуск всей backend-регрессии
|
||||
|
||||
- [ ] **Step 1: Миграция**
|
||||
|
||||
```php
|
||||
<?php
|
||||
declare(strict_types=1);
|
||||
|
||||
use Illuminate\Database\Migrations\Migration;
|
||||
use Illuminate\Support\Facades\DB;
|
||||
|
||||
return new class extends Migration {
|
||||
public function up(): void
|
||||
{
|
||||
DB::statement('ALTER TABLE projects DROP COLUMN IF EXISTS archived_at');
|
||||
}
|
||||
|
||||
public function down(): void
|
||||
{
|
||||
DB::statement('ALTER TABLE projects ADD COLUMN archived_at TIMESTAMPTZ NULL');
|
||||
}
|
||||
};
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Чистка кода**
|
||||
|
||||
- `Project.php`: удалить `'archived_at'` из `$fillable` и из casts; удалить методы `scopeArchived` и `scopeActive` (scope `scopeActiveOnDay` — НЕ трогать, это про день недели). После удаления заменить всех вызывающих `->active()` на чистый query:
|
||||
- `ProjectService::create()` лимит-проверка: `Project::where('tenant_id', $tenant->id)->active()->count()` → `Project::where('tenant_id', $tenant->id)->count()` (после сноса архива «активные» = все проекты тенанта).
|
||||
- `ProjectController::index()` — см. ниже.
|
||||
Проверить `grep -rn "->active(" app/app` после правок (должны остаться только `scopeActiveOnDay`/`PricingTier`/`SupplierProject`).
|
||||
- `ProjectResource.php`: удалить строку `'archived_at' => ...`.
|
||||
- `ProjectController::index()`: удалить ветку `if ($status === 'archived')`; для `active`/`paused`/default убрать вызовы `->active()`/`->archived()` (фильтрация только по `is_active`).
|
||||
- `DashboardController.php:77`: убрать `->whereNull('archived_at')`.
|
||||
- `SyncSupplierProjectsJob.php:89`: убрать `->whereNull('archived_at')`.
|
||||
|
||||
- [ ] **Step 3: schema.sql + CHANGELOG**
|
||||
|
||||
В `db/schema.sql` удалить строку `archived_at TIMESTAMPTZ NULL,` из `CREATE TABLE projects`; обновить header-комментарий → v8.27 (drop projects.archived_at). В `db/CHANGELOG_schema.md` добавить запись v8.27.
|
||||
|
||||
- [ ] **Step 4: Прогнать миграцию на dev + регрессия**
|
||||
|
||||
Run:
|
||||
|
||||
```
|
||||
C:/tools/php83/php.exe artisan migrate
|
||||
C:/tools/php83/php.exe artisan test --filter=Project
|
||||
composer stan
|
||||
```
|
||||
|
||||
Expected: миграция OK; тесты зелёные; Larastan 0 (или обновить baseline, если всплыло legacy).
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/app/Models/Project.php app/app/Http/Resources/ProjectResource.php app/app/Http/Controllers/Api/ProjectController.php app/app/Http/Controllers/Api/DashboardController.php app/app/Jobs/Supplier/SyncSupplierProjectsJob.php app/database/migrations/2026_05_21_000000_drop_projects_archived_at.php db/schema.sql db/CHANGELOG_schema.md
|
||||
git commit -m "refactor(projects): remove archive feature, drop archived_at column (schema v8.27)"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 7: Фронтенд — «Архивировать» → «Удалить»
|
||||
|
||||
**Files:**
|
||||
|
||||
- Modify: `app/resources/js/stores/projectsStore.ts` (`archive`→`del`, bulk type, `archived_at` из интерфейса)
|
||||
- Modify: `app/resources/js/components/projects/BulkActionsBar.vue`
|
||||
- Modify: `app/resources/js/components/projects/ProjectCard.vue`
|
||||
- Modify: `app/resources/js/components/projects/ProjectDetailsDrawer.vue`
|
||||
- Modify: `app/resources/js/views/ProjectsView.vue` (фильтр «Архивные», `@archive`→`@delete`)
|
||||
- Test: `app/resources/js/stores/projectsStore.spec.ts` (или существующий) + затронутые spec'и
|
||||
|
||||
- [ ] **Step 1: Падающий тест стора**
|
||||
|
||||
В spec проверить, что метод удаления дёргает `DELETE /api/projects/{id}`:
|
||||
|
||||
```ts
|
||||
it('delete() calls DELETE /api/projects/{id}', async () => {
|
||||
const store = useProjectsStore();
|
||||
vi.spyOn(axios, 'delete').mockResolvedValue({ data: null });
|
||||
vi.spyOn(axios, 'get').mockResolvedValue({ data: { data: [], meta: { total: 0 } } });
|
||||
await store.del(7);
|
||||
expect(axios.delete).toHaveBeenCalledWith('/api/projects/7');
|
||||
});
|
||||
```
|
||||
|
||||
- [ ] **Step 2: Запустить — FAIL**
|
||||
|
||||
Run: `npm --prefix app run test:vue -- projectsStore`
|
||||
Expected: FAIL (`del` не существует).
|
||||
|
||||
- [ ] **Step 3: Реализация фронта**
|
||||
|
||||
- `projectsStore.ts`: переименовать `archive`→`del` (метод + return); в `bulkAction`/`BulkPayload` тип `'archive'`→`'delete'`; убрать `archived_at` из `interface Project`.
|
||||
- `BulkActionsBar.vue`: кнопка «Архивировать»→«Удалить» (`data-testid="bulk-delete"`, иконка `mdi-delete`→Lucide `Trash2` через IconSet), confirm-текст про удаление; `confirmAndRun('delete')`; тип union `'pause'|'resume'|'delete'`.
|
||||
- `ProjectCard.vue`: пункт меню «Архивировать»→«Удалить» (иконка `mdi-delete`), `$emit('delete', project)`; emit-тип `delete: [project: Project]`.
|
||||
- `ProjectDetailsDrawer.vue`: «Архивировать проект?»→«Удалить проект? Действие необратимо.»; `store.del(props.project.id)`.
|
||||
- `ProjectsView.vue`: `@archive`→`@delete="(p) => store.del(p.id)"`; убрать `{ title: 'Архивные', value: 'archived' }` из фильтра статусов.
|
||||
|
||||
NB: иконки — через существующий IconSet mapping в `plugins/vuetify.ts` (`mdi-delete`→`Trash2`); если маппинга нет — добавить.
|
||||
|
||||
- [ ] **Step 4: Запустить — PASS + тип-чек + сборка**
|
||||
|
||||
Run:
|
||||
|
||||
```
|
||||
npm --prefix app run test:vue
|
||||
npm --prefix app run type-check
|
||||
npm --prefix app run build
|
||||
```
|
||||
|
||||
Expected: тесты зелёные (обновить spec'и, где был `archive`/`archived`); type-check 0; build OK.
|
||||
|
||||
- [ ] **Step 5: Commit**
|
||||
|
||||
```bash
|
||||
git add -- app/resources/js/stores/projectsStore.ts app/resources/js/components/projects/BulkActionsBar.vue app/resources/js/components/projects/ProjectCard.vue app/resources/js/components/projects/ProjectDetailsDrawer.vue app/resources/js/views/ProjectsView.vue
|
||||
git commit -m "feat(projects-ui): replace archive with delete, drop archived filter"
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
## Task 8: Живая проверка всех 4 задач + чистка тестовых данных
|
||||
|
||||
Среда: portal `serve` :8000 + `queue:work` (запустить, если не идёт). Демо tenant 1 (`admin@demo.local`/`password`).
|
||||
|
||||
- [ ] **Шаг 1 (задача 4 — человеческие ошибки):** создать проект с именем существующего → ожидать понятное 422-сообщение (не SQL). Создать проект с именем-дублем через UI/`tinker` POST → проверить тело ответа.
|
||||
- [ ] **Шаг 2 (задача 3 — дедуп источника):** в рамках tenant 1 создать 2-й проект на тот же `signal_identifier` → ожидать «У вас уже есть проект с этим источником…».
|
||||
- [ ] **Шаг 3 (задача 2 — шеринг):** под другим тенантом создать проект с тем же источником → проходит (демонстрация «ок»).
|
||||
- [ ] **Шаг 4 (задача 1 — удаление):** удалить пустой проект → исчез; удалить проект со сделкой → блок с сообщением, проект жив. Если есть привязка к донору и других потребителей нет → проверить, что `DeleteSupplierProjectJob` удалил донора у поставщика (или пере-синкнул при наличии других).
|
||||
- [ ] **Шаг 5 (чистка):** удалить все тестовые проекты/сделки/доноров, созданные в шагах 1–4; вернуть БД к чистому демо; зафиксировать в отчёте, что изменилось.
|
||||
|
||||
(Verify-skill: каждый шаг — реальный рантайм, capture результата; затем cleanup.)
|
||||
|
||||
---
|
||||
|
||||
## Финальная регрессия (после всех задач)
|
||||
|
||||
Run (из корня/`app/`):
|
||||
|
||||
```
|
||||
C:/tools/php83/php.exe artisan test
|
||||
npm --prefix app run test:vue
|
||||
npm --prefix app run type-check
|
||||
npm --prefix app run build
|
||||
composer pint && composer stan
|
||||
```
|
||||
|
||||
Все зелёные → готово к push (`git push origin <ветка>:main`) по решению заказчика.
|
||||
@@ -0,0 +1,274 @@
|
||||
# Observer Canonical Chain Attribution (L1–L13) — Design Spec
|
||||
|
||||
**Дата:** 2026-05-20
|
||||
**Автор:** controller Opus 4.7 (через `superpowers:brainstorming` skill, ответы заказчика → AskUserQuestion)
|
||||
**Базовая ветка:** `feat/project-migration-redesign` (формально) → реальное исполнение в свежем worktree off `origin/main` ПОСЛЕ закрытия epic-плана `2026-05-20-observer-instrument-expansion v1.1` (20 task).
|
||||
**Триггер:** [docs/observer/notes/2026-05-20-brain-retro-v2.md](../../observer/notes/2026-05-20-brain-retro-v2.md) — Candidate №2 «атрибуция canonical chains L1–L13 в primary_rationale».
|
||||
**Cross-refs:** [docs/routing-off-phase.md](../../routing-off-phase.md) v1.2 (L1–L13 таблица, строки 84–96); [docs/superpowers/specs/2026-05-19-observer-factor-analysis-design.md](2026-05-19-observer-factor-analysis-design.md) v1.2 (Layer 2 heuristic capture, `triggers_matched` уже извлекает routing-off-phase LN — но как trigger, не как attribute узла); ADR-011 (Brain governance anchor).
|
||||
**Schema impact:** `schema_version` остаётся `2` — `chain_ref` это **опциональное** поле. Backward-compatible с v1/v2-эпизодами без него.
|
||||
|
||||
---
|
||||
|
||||
## 1. Проблема
|
||||
|
||||
В эпизодах наблюдателя (`docs/observer/episodes-YYYY-MM.jsonl`, schema v2) поле `primary_rationale.node_chosen` фиксирует **какой узел** Claude выбрал (например, `"superpowers:verification-before-completion"` или `"laravel-boost"` или `"direct"`). Но **нет ссылки на канонические цепочки L1–L13** из [docs/routing-off-phase.md](../../routing-off-phase.md) v1.2.
|
||||
|
||||
Из-за этого `/brain-retro` не может ответить на вопрос: «насколько часто Claude действительно ходит по правильным маршрутам цепочек?». В ретро-ноте 2026-05-20 секция «Canonical chains L1–L12 hit rate» — **пустая**: «нет атрибуции `chain_ref`».
|
||||
|
||||
`extractTriggers` (Layer 2 heuristic из factor-analysis v1.2) **уже** извлекает упоминания `routing-off-phase LN` в `assistant.text` — но это маркер **триггера** (что Claude упомянул в обосновании), а не **атрибуция узла**: узел Boost #10 живёт в L7 + L13 независимо от того, упомянул ли Claude routing-таблицу в тексте.
|
||||
|
||||
## 2. Цель
|
||||
|
||||
Добавить **атрибут** `chain_ref` рядом с `node_chosen`, который связывает выбранный узел с одной или несколькими каноническими цепочками L1–L13. Это позволит `/brain-retro` строить гистограмму «L1: N раз, L2: M раз, …, вне цепочек: K раз» по любому периоду.
|
||||
|
||||
**Граница цели:** наблюдатель *регистрирует* атрибут, *не диктует* поведение. Если Claude выбрал узел вне L-таблицы — никаких блокировок (Pravila §16.4 «не использован ≠ проблема»).
|
||||
|
||||
## 3. Решения по архитектуре (приняты заказчиком через AskUserQuestion)
|
||||
|
||||
| # | Развилка | Выбор | Обоснование |
|
||||
|---|---|---|---|
|
||||
| 1 | Что для `direct`-эпизодов (узел не в L-таблице) | `chain_ref: null` | Честно: L1–L13 — routing-цепочки для сложных задач; простые правки в них не входят. |
|
||||
| 2 | Узел в нескольких L (Boost в L7+L13, Sentry в L8+L13, adr-kit в L4+L5) | Массив всех цепочек `["L7","L13"]` | Узел действительно в обеих; агрегатор считает обе. |
|
||||
| 3 | Где живёт маппинг узел → цепочки | Отдельный JSON-файл + **тест сверки с `routing-off-phase.md`, врезанный в lefthook pre-commit** (Вариант Б, не А) | Дрейф ловится агрессивно: коммит, расходящийся с .md, блокируется. Pre-commit + Vitest используют одну логику. |
|
||||
| 4 | Ретроактивность для 23 v2-эпизодов мая 2026 | Однократный ретрофилл-скрипт | ~30 строк, идемпотентный, даёт честную статистику с 19.05. |
|
||||
|
||||
## 4. Архитектура высокого уровня
|
||||
|
||||
К наблюдателю добавляется **один слой** — «chain attribution». Работает в трёх местах:
|
||||
|
||||
1. **При записи эпизода** (Stop-хук → парсер транскрипта): после выбора `node_chosen` парсер дополнительно вычисляет `chain_ref` через чистую функцию `chainsFor(node)`. Запись в JSONL — атомарная append-line, как и сейчас.
|
||||
2. **При коммите** (lefthook pre-commit): контролёр **C6** сверяет JSON-маппинг с таблицей L1–L13 в `routing-off-phase.md`. Расхождение → коммит блокируется с человеко-читаемым diff'ом.
|
||||
3. **При ретро** (`/brain-retro`): `brain-retro-analyzer.mjs` агрегирует гистограмму `factorMatrix.chain_ref` + `chainHitRate`. Шаблон `aggregation-template.md` заполняет секцию «Canonical chains L1–L13 hit rate».
|
||||
|
||||
Существующие парсер/Stop-хук/routing-detector/choice-detector — **нетронуты**. Схема `schema_version: 2` сохраняется: `chain_ref` опционален; старые эпизоды без него по-прежнему валидны.
|
||||
|
||||
## 5. Компоненты
|
||||
|
||||
| # | Файл | Роль | Тип | Объём |
|
||||
|---|---|---|---|---|
|
||||
| 1 | `tools/observer-chain-map.json` | Таблица узел → массив цепочек, например `{"laravel-boost":["L7","L13"], "superpowers:verification-before-completion":[]}`. Человеко-читаемый | новый | ~60 строк |
|
||||
| 2 | `tools/observer-chain-detector.mjs` | Чистая функция `chainsFor(nodeChosen)` → массив или `null`. Грузит JSON один раз, кэширует в Map | новый | ~40 строк |
|
||||
| 3 | `tools/observer-transcript-parser.mjs` | **Точка врезки:** в формировании `primary_rationale` (текущая строка 450) — добавить `chain_ref: chainsFor(node_chosen)` | edit existing | +2 строки |
|
||||
| 4 | `tools/observer-chain-map-checker.mjs` | Контролёр **C6**: парсит таблицу L1–L13 из `routing-off-phase.md`, сверяет с JSON-маппингом. Возвращает diff или OK | новый | ~80 строк |
|
||||
| 5 | `lefthook.yml` | Новый job `chain-map-sync` в pre-commit запускает контролёр C6. Блокирует коммит при расхождении | edit existing | +5 строк |
|
||||
| 6 | `tools/observer-retrofill-chain-ref.mjs` | Одноразовый скрипт: добавляет `chain_ref` к v2-эпизодам в `episodes-*.jsonl`. Атомарный (tmp + rename), идемпотентный | новый | ~30 строк |
|
||||
| 7 | `tools/brain-retro-analyzer.mjs` | Дополняется `factorMatrix.chain_ref` (гистограмма) и `chainHitRate` (массив с процентами) | edit existing | +20 строк |
|
||||
| 8 | `.claude/skills/brain-retro/references/aggregation-template.md` | Раздел «Canonical chains L1–L13 hit rate» заполняется реальными данными из аналитика | edit existing | +5 строк |
|
||||
|
||||
**Тесты (отдельно, 2 файла):**
|
||||
|
||||
- `tests/observer-chain-detector.test.mjs` — юнит-тесты `chainsFor`: известный узел / multi-chain узел / неизвестный узел / `direct` / `null` / `undefined` / пустая строка. ~6 тестов, ~50 строк.
|
||||
- `tests/observer-chain-map-sync.test.mjs` — интеграционный тест: запускает тот же C6-чекер, парсит `routing-off-phase.md`, сверяет с JSON. Та же логика что в pre-commit, один источник правды.
|
||||
|
||||
**Итого:** 4 новых файла в `tools/`, 2 новых теста, 4 точки правок в existing, +1 в `lefthook.yml`, +1 в SKILL-template.
|
||||
|
||||
## 6. Поток данных
|
||||
|
||||
### Поток A — запись эпизода (runtime, при каждом Stop-хуке)
|
||||
|
||||
```
|
||||
Stop-хук → observer-transcript-parser.mjs (existing)
|
||||
↓
|
||||
формирует primary_rationale (node_chosen, triggers_matched, …)
|
||||
↓
|
||||
вызывает chainsFor(node_chosen) ← из observer-chain-detector.mjs (NEW)
|
||||
↓ читает observer-chain-map.json (cached)
|
||||
primary_rationale.chain_ref = ["L7","L13"] | null
|
||||
↓
|
||||
append-line в docs/observer/episodes-YYYY-MM.jsonl
|
||||
```
|
||||
|
||||
Дополнительная задержка — ~1–2 мс (lookup в `Map<string, string[]>` в памяти). JSON загружается один раз при первом вызове.
|
||||
|
||||
### Поток B — pre-commit сверка (раз в коммит)
|
||||
|
||||
```
|
||||
git commit → lefthook → job chain-map-sync (NEW)
|
||||
↓
|
||||
node tools/observer-chain-map-checker.mjs
|
||||
↓
|
||||
parse routing-off-phase.md (таблица L1–L13, строки 84–96)
|
||||
↓
|
||||
load observer-chain-map.json
|
||||
↓
|
||||
diff: (узлы в .md без записи в JSON) ∪ (узлы в JSON без записи в .md) ∪ (L-список расходится)
|
||||
↓
|
||||
exit 0 (синхронно) | exit 1 (расхождение + human-readable сообщение с подсказкой)
|
||||
```
|
||||
|
||||
### Поток C — агрегация в /brain-retro (раз в спринт)
|
||||
|
||||
```
|
||||
node tools/brain-retro-analyzer.mjs episodes-*.jsonl
|
||||
↓
|
||||
читает все эпизоды (v2)
|
||||
↓
|
||||
группирует по chain_ref (multi-chain эпизоды засчитываются в каждую L)
|
||||
↓
|
||||
factorMatrix.chain_ref: {"L1":0, "L7":3, "L13":1, "null":19, …}
|
||||
↓
|
||||
chainHitRate: [{chain:"L7", times:3, percent:"13.0%"}, …]
|
||||
↓
|
||||
aggregation-template заполняет секцию «L1–L13 hit rate»
|
||||
```
|
||||
|
||||
### Поток D — однократный ретрофилл
|
||||
|
||||
```
|
||||
node tools/observer-retrofill-chain-ref.mjs [--dry-run]
|
||||
↓
|
||||
для каждого episodes-*.jsonl:
|
||||
↓ для каждой строки v2:
|
||||
если chain_ref уже есть → skip (idempotent)
|
||||
иначе: добавить chain_ref: chainsFor(node_chosen)
|
||||
↓
|
||||
атомарная перезапись (write tmp → rename)
|
||||
```
|
||||
|
||||
Запускается один раз вручную после внедрения. Повторный запуск — 0 changes.
|
||||
|
||||
**Что НЕ меняется:**
|
||||
|
||||
- `schema_version: 2` — `chain_ref` опциональное поле.
|
||||
- 5 v1-эпизодов мая 2026 — пропускаются (нет `schema_version: 2` и нет `primary_rationale`).
|
||||
- Stop-хук, parser-логика выбора `node_chosen`, routing-detector, choice-detector — не трогаем.
|
||||
- Никаких новых hard-blockers для Claude. Pre-commit блокирует только дрейф маппинга.
|
||||
|
||||
## 7. Обработка ошибок
|
||||
|
||||
| # | Сценарий | Поведение |
|
||||
|---|---|---|
|
||||
| 1 | Узел не найден в маппинге (`chainsFor("foo-bar-baz")`) | Возвращает `null`. Эпизод пишется. Нормальная ситуация для direct/новых skills/строкового шума |
|
||||
| 2 | `observer-chain-map.json` отсутствует или битый | Парсер ловит исключение, пишет `observer_error` маркер + `chain_ref: null`. Эпизод **всё равно записывается** — наблюдатель не падает. Контролёр C5 (observer-coverage-checker) увидит маркер и подсветит в STATUS.md |
|
||||
| 3 | `.md` правят, JSON не обновили (дрейф) | Pre-commit `chain-map-sync` падает с понятным diff'ом: «в .md есть `ru-tax-accounting → L13`, в JSON нет — добавьте». Узлы из новых L-цепочек до добавления в JSON не атрибутируются (см. п. 1) |
|
||||
| 4 | Формат таблицы в `.md` изменён (новая колонка, переименование L) | Парсер `chain-map-checker.mjs` падает с указанием строки: «не могу распарсить L7». Vitest даёт ту же ошибку до коммита |
|
||||
| 5 | Retrofill-скрипт прерван на середине | Запись атомарная (tmp + rename) — файл всегда консистентен. Идемпотентность — повторный запуск пропускает уже обработанные строки |
|
||||
| 6 | JSON ссылается на несуществующую L (например `L99`) | C6 ловит: «JSON ссылается на L99, в `.md` нет». exit 1 |
|
||||
| 7 | v1-эпизоды (5 строк мая) | Пропускаются — у них нет `schema_version: 2` и `primary_rationale` |
|
||||
|
||||
## 8. Тестирование
|
||||
|
||||
**Юнит-тесты (`tests/observer-chain-detector.test.mjs`):**
|
||||
|
||||
- `chainsFor("laravel-boost")` → `["L7","L13"]`
|
||||
- `chainsFor("superpowers:verification-before-completion")` → `null` (узел НЕ входит ни в одну L1–L13 → его нет в JSON-маппинге → `null`, как и для `direct`)
|
||||
- `chainsFor("direct")` → `null`
|
||||
- `chainsFor("unknown-node")` → `null`
|
||||
- `chainsFor("")`, `chainsFor(null)`, `chainsFor(undefined)` → `null`
|
||||
- ~6 тестов, ~50 строк.
|
||||
|
||||
**Единообразие формы (решено в self-review):** только две формы результата — непустой массив `["LN", …]` (узел есть в JSON) или `null` (узла нет в JSON: direct / узел вне цепочек / неизвестный / шум). Пустой массив `[]` НЕ используется — нет узлов «в маппинге, но без цепочек».
|
||||
|
||||
**Интеграционный тест синхронизации (`tests/observer-chain-map-sync.test.mjs`):**
|
||||
|
||||
- Использует тот же `chain-map-checker.mjs`, что и pre-commit.
|
||||
- Падает, если кто-то добавил строку в `.md` без обновления JSON.
|
||||
- Сообщение теста = сообщение pre-commit job.
|
||||
|
||||
**Сценарный тест парсера (опционально, если бюджет позволяет):**
|
||||
|
||||
- Подаёт фиктивный транскрипт с известным skill (`Boost`) → проверяет, что в результирующем эпизоде есть `chain_ref: ["L7","L13"]`.
|
||||
- Подаёт транскрипт без skill (direct) → `chain_ref: null`.
|
||||
- Подаёт с битым JSON → пишется `observer_error` + `chain_ref: null`.
|
||||
|
||||
**Smoke ретрофилла (ручной шаг после реализации):**
|
||||
|
||||
- `node tools/observer-retrofill-chain-ref.mjs --dry-run` → видим планируемые изменения.
|
||||
- Если OK — без `--dry-run`. Идемпотентность проверяется повторным запуском (должно быть 0 changes).
|
||||
|
||||
**Что НЕ тестируем:**
|
||||
|
||||
- Скорость (1–2 мс заведомо некритично для Stop-хука).
|
||||
- Все 60+ узлов руками — JSON + sync-тест уже это покрывают.
|
||||
|
||||
## 9. Initial JSON-маппинг (на момент дизайна, routing-off-phase.md v1.2)
|
||||
|
||||
Источник истины — таблица L1–L13 в [docs/routing-off-phase.md строки 84–96](../../routing-off-phase.md#L84). Извлечение узлов из ячейки «Цепочка» — однократное руками при первой реализации, дальше контролёр C6 синхронизирует.
|
||||
|
||||
Примерный shape (демонстрация формата; финальный JSON генерируется при реализации):
|
||||
|
||||
```json
|
||||
{
|
||||
"_note": "Только узлы, входящие хотя бы в одну L1-L13. Узлы вне цепочек (direct, verification-before-completion, прочие skills вне L) НЕ включаются — chainsFor вернёт null.",
|
||||
"discovery-interview": ["L1","L2"],
|
||||
"superpowers:brainstorming": ["L1"],
|
||||
"superpowers:writing-plans": ["L1"],
|
||||
"superpowers:subagent-driven-development": ["L1"],
|
||||
"audit-portal": ["L2"],
|
||||
"process-analysis": ["L3"],
|
||||
"process-modeling": ["L3","L4"],
|
||||
"mermaid": ["L4"],
|
||||
"adr-kit": ["L4","L5"],
|
||||
"operations": ["L4"],
|
||||
"architecture-patterns": ["L5"],
|
||||
"deptrac": ["L5"],
|
||||
"trail-of-bits": ["L6"],
|
||||
"semgrep-mcp": ["L6"],
|
||||
"security-guidance": ["L6"],
|
||||
"security-review": ["L6"],
|
||||
"openapi-mcp-server": ["L7"],
|
||||
"api-docs": ["L7"],
|
||||
"laravel-boost": ["L7","L13"],
|
||||
"superpowers:systematic-debugging": ["L8"],
|
||||
"sentry-mcp": ["L8","L13"],
|
||||
"redis-mcp": ["L8","L13"],
|
||||
"ccpm": ["L9"],
|
||||
"product-management": ["L9"],
|
||||
"github-mcp": ["L9"],
|
||||
"promptfoo": ["L10"],
|
||||
"data-scientist": ["L10"],
|
||||
"claude-api": ["L10"],
|
||||
"skill-creator": ["L11"],
|
||||
"hookify": ["L11"],
|
||||
"plugin-dev": ["L11"],
|
||||
"claude-md-management": ["L12"],
|
||||
"billing-audit": ["L13"],
|
||||
"pest": ["L13"],
|
||||
"ru-tax-accounting": ["L13"]
|
||||
}
|
||||
```
|
||||
|
||||
**Note для реализации:** имена узлов в JSON должны точно соответствовать значениям, которые парсер записывает в `node_chosen`. Если узел в `node_chosen` — это `"superpowers:verification-before-completion"`, то в JSON ключ такой же. Это валидируется C6 + sync-тестом.
|
||||
|
||||
## 10. Внешние зависимости и порядок исполнения
|
||||
|
||||
**Жёсткая зависимость:** этот spec исполняется **ПОСЛЕ** закрытия и push'а в `origin/main` эпик-плана [docs/superpowers/plans/2026-05-20-observer-instrument-expansion.md](../plans/2026-05-20-observer-instrument-expansion.md) v1.1 (20 атомарных коммитов).
|
||||
|
||||
**Причина:** epic 20-task правит `observer-transcript-parser.mjs` в Tasks #1, #2, #4, #6, #7, #8, #9, #12, #13. Моя врезка `chain_ref` тоже в этом файле. Параллельное исполнение → merge-конфликт почти гарантирован.
|
||||
|
||||
**Процедура запуска работы по этому spec:**
|
||||
|
||||
1. Дождаться push'а epic 20-task на `origin/main` (контролёр или владелец сообщает «epic закрыт, push сделан»).
|
||||
2. `git fetch origin && git log HEAD..origin/main --oneline` — убедиться, что 20 task реально влиты.
|
||||
3. Создать свежий worktree `.claude/worktrees/observer-chain-attribution/` off `origin/main` (новая ветка `feat/observer-chain-attribution`).
|
||||
4. Перейти к `superpowers:writing-plans` — детальный TDD-план по компонентам §5.
|
||||
5. Исполнить через `superpowers:subagent-driven-development` (Sonnet/Opus only per Pravila §15.1).
|
||||
6. Финальный push: `git push origin feat/observer-chain-attribution:main` (FF-merge).
|
||||
|
||||
## 11. Влияние на нормативку
|
||||
|
||||
**Не требует** правок Pravila / PSR_v1 / Tooling / CLAUDE.md / ADR.
|
||||
|
||||
- `chain_ref` — опциональное расширение schema v2, не нормативный сдвиг (как `task_cost` в epic-плане v1.1 — там тоже без правок Pravila).
|
||||
- L1–L13 уже формализованы в `routing-off-phase.md` v1.2 (правлено 20.05 при finance-tooling).
|
||||
- Контролёр C6 — добавление нового lefthook job, по аналогии с C1/C2/C3/C4/C5; в STATUS.md появится строка «C6 Chain map sync». Это организационная правка, не нормативная.
|
||||
|
||||
**Опционально:** при желании заказчика — micro-правка в Pravila §16.2 «Схема эпизода v2» с упоминанием `chain_ref` как опционального атрибута. Делается одним коммитом через `claude-md-management`. Не блокирует основную работу.
|
||||
|
||||
## 12. Из scope исключено (NOT this spec)
|
||||
|
||||
- **`chain_divergence` event** — заявлен в factor-analysis v1.2 §10 как phase-2 / agent-based (нужен LLM-судья «правильная ли цепочка»). Не в этом spec'е. `chain_ref` — это атрибутирование, а не суждение.
|
||||
- **`triggers_matched: routing-off-phase L7` heuristic** — уже реализуется в epic-плане v1.1 Task #6 (reasoning capture). `chain_ref` — отдельный атрибут параллельно, не дубль.
|
||||
- **Real-time блокировка «ушёл вне цепочки»** — спорная idea, противоречит Pravila §16.4 «не использован ≠ проблема». NOT this spec.
|
||||
- **Авто-правки нормативки по результатам hit rate** — фантазия v3+. NOT this spec.
|
||||
|
||||
## 13. Acceptance criteria
|
||||
|
||||
Spec считается реализованным когда:
|
||||
|
||||
1. Все 6 новых файлов из §5 созданы, тесты проходят локально и в CI.
|
||||
2. Pre-commit job `chain-map-sync` врезан в `lefthook.yml`, smoke-проверка red-green работает (намеренная рассинхронизация JSON ↔ .md → коммит блокируется; правильная синхронизация → проходит).
|
||||
3. `node tools/observer-retrofill-chain-ref.mjs --dry-run` показывает планируемые изменения для всех v2-эпизодов в `episodes-2026-05.jsonl`; запуск без `--dry-run` добавляет `chain_ref`; повторный запуск → 0 changes.
|
||||
4. `/brain-retro` следующего spring печатает непустую секцию «Canonical chains L1–L13 hit rate» с реальными цифрами.
|
||||
5. STATUS.md добавлена строка «C6 Chain map sync: ✅ — last sync OK».
|
||||
6. Финальная регрессия `npm run test:tools` ≥ 331 + N (где N — число новых тестов из §8) GREEN.
|
||||
@@ -0,0 +1,149 @@
|
||||
# Удаление проектов вместо архива + дедуп источника + человеческие ошибки
|
||||
|
||||
**Дата:** 2026-05-21
|
||||
**Статус:** утверждён (заказчик «ок делай»)
|
||||
**Ветка исполнения:** worktree от `feat/project-migration-redesign` (origin/main)
|
||||
|
||||
## Контекст и проблема
|
||||
|
||||
Заказчик при создании проекта получил сырой `SQLSTATE[23505] Unique violation ...
|
||||
projects_tenant_id_name_key` в UI. Разбор вскрыл 4 связанных задачи:
|
||||
|
||||
1. **Архивация бессмысленна** — если клиенту проект не нужен, хранить его незачем.
|
||||
Сейчас «удаление» (`DELETE /api/projects/{id}`) = мягкая архивация
|
||||
(`ProjectService::archive` → `archived_at`, `is_active=false`). Нужно настоящее удаление.
|
||||
2. **Шеринг между клиентами** — два разных клиента могут завести проекты с одинаковыми
|
||||
параметрами/источником. Это корректно (модель шеринга: один донор раздаётся ≤3 клиентам
|
||||
через `LeadRouter`). Менять не нужно — подтвердить.
|
||||
3. **Дедуп источника внутри клиента отсутствует** — один клиент может завести 2 проекта на
|
||||
один источник (номер/домен/SMS-отправитель). Нужен запрет.
|
||||
4. **Утечка SQL в UI** — `Project::create()` бьётся о DB-констрейнт без перехвата → сырой
|
||||
`SQLSTATE` рендерится пользователю. Нужны человеческие сообщения.
|
||||
|
||||
## Ключевые факты кодовой базы (разведка)
|
||||
|
||||
- `deals.project_id` — **без FK** (таблица партиционирована, partition-wise FK не
|
||||
поддерживается). Hard-delete проекта НЕ каскадит и НЕ блокируется сделками → они «повиснут».
|
||||
Поэтому удаление проекта со сделками опасно → блокируем (см. Решение 1).
|
||||
- `ON DELETE CASCADE` на `projects(id)` имеют только служебные таблицы:
|
||||
`project_supplier_links`, `supplier_manual_sync_queue`, `project_suppliers`,
|
||||
`project_user_assignments`, `project_limit_adjustments`. Их каскад при удалении — норма.
|
||||
- Уникальность `projects` на DB-уровне: `projects_tenant_id_name_key` = `(tenant_id, name)`.
|
||||
Дедупа по источнику на DB-уровне нет.
|
||||
- Источник проекта: `signal_identifier` (call=телефон `7\d{10}`, site=домен) либо
|
||||
`sms_senders[]` (+ `sms_keyword`) для sms.
|
||||
- Шеринг донора: `project_supplier_links` (M:N projects↔supplier_projects) связывает
|
||||
Лидерра-проекты РАЗНЫХ тенантов с одним `supplier_project` (донором). Outbound-синк
|
||||
(`SyncSupplierProjectsJob`) считает агрегатный лимит по всем клиентам источника
|
||||
(`computeOrder = max(max, ceil(Σ/3))`, cap=3).
|
||||
- Архив-ссылки (под снос): backend — `ProjectController` (destroy/index/bulk),
|
||||
`ProjectService` (archive/bulk/update/resolveBulkScope), `Project` (scopeActive/scopeArchived,
|
||||
fillable+cast `archived_at`), `ProjectResource`, `DashboardController`,
|
||||
`SyncSupplierProjectsJob`, `BulkProjectActionRequest`. Frontend — `BulkActionsBar`,
|
||||
`ProjectCard`, `ProjectDetailsDrawer`, `ProjectsView`, `projectsStore`.
|
||||
|
||||
## Решения (утверждены заказчиком)
|
||||
|
||||
### Решение 1 — удаление вместо архива, с защитой по сделкам
|
||||
|
||||
- `DELETE /api/projects/{id}` → **hard delete** через новый `ProjectService::delete()`.
|
||||
- **Guard:** если по проекту есть хоть одна `deals` (любой статус, включая `deleted_at`-soft)
|
||||
→ удаление блокируется HTTP 422 (`{errors:{...}}`/`message`, формат фронта) с сообщением:
|
||||
*«Нельзя удалить проект: по нему есть сделки. Остановите приём (пауза), чтобы скрыть из
|
||||
работы».* Пустой проект (0 сделок) → удаляется насовсем.
|
||||
- Архивация убирается **полностью**: код-пути `archive`, scope `archived`, фильтр «Архивные»,
|
||||
bulk-action `archive`. Колонка `archived_at` дропается миграцией (schema bump). Пауза
|
||||
(`is_active`) сохраняется — это отдельный механизм.
|
||||
- Bulk: action `archive` → `delete` (с тем же guard'ом per-project; проекты со сделками
|
||||
попадают в `skipped` с причиной, не роняют весь батч).
|
||||
|
||||
### Решение 2 — удаление у поставщика с учётом шеринга
|
||||
|
||||
При удалении Лидерра-проекта P (тенант T) по источнику X (один или несколько доноров
|
||||
B1/B2/B3 через `project_supplier_links`):
|
||||
|
||||
1. Удаляем P локально (его `project_supplier_links` уходят каскадом).
|
||||
2. Для каждого затронутого `supplier_project` S (донора источника X):
|
||||
- Считаем оставшиеся `project_supplier_links` на S (проекты ДРУГИХ тенантов).
|
||||
- **Остались** → донор нужен другим клиентам → **пере-синк** S (агрегатный лимит/регионы/дни
|
||||
без T) через outbound-синк. У поставщика проект НЕ удаляем.
|
||||
- **Не осталось** (T был последним) → у поставщика **удаляем** донора
|
||||
(`SupplierPortalClient::deleteProject(external_id)`) + удаляем локальную запись
|
||||
`supplier_projects` S.
|
||||
3. Внешние вызовы к поставщику — через job (resilience, retry), не inline в HTTP-запросе.
|
||||
|
||||
Граничные сценарии:
|
||||
|
||||
- P не привязан ни к одному донору (например, синк ещё не прошёл) → шаг 2 пропускается.
|
||||
- Несколько доноров (B1/B2/B3) у одного источника → шаг 2 для каждого независимо.
|
||||
- Падение удаления у поставщика → job ретраит; локальное удаление P уже выполнено
|
||||
(eventual consistency; «висячий» донор у поставщика подметёт следующий синк/ретрай).
|
||||
|
||||
### Решение 3 — дедуп источника внутри клиента
|
||||
|
||||
- При создании и при изменении источника: внутри `tenant_id` источник должен быть уникален
|
||||
среди проектов клиента.
|
||||
- «Источник» (source key):
|
||||
- call/site → `signal_identifier`;
|
||||
- sms → нормализованный `sms_senders` (сортировка+lower) + `sms_keyword`.
|
||||
- Enforcement: app-level проверка в `ProjectService::create()`/`update()` → 422 с сообщением
|
||||
*«У вас уже есть проект с этим источником: "<название>"»*. После сноса архива «существующие
|
||||
проекты» = все проекты клиента (soft-deleted проектов нет — мы их hard-delete'им).
|
||||
- DB-уровень: partial unique index как защита-эшелон (опционально, в той же миграции);
|
||||
его нарушение перехватывается общим обработчиком (Решение 4) и не утекает.
|
||||
|
||||
### Решение 4 — человеческие ошибки вместо SQL
|
||||
|
||||
- App-level pre-checks (до DB): уникальность `name` в рамках клиента + уникальность источника
|
||||
(Решение 3) → 422 `{errors: {field: [msg]}}` (формат, который уже понимает фронт).
|
||||
- Глобальный перехват `Illuminate\Database\QueryException` в `bootstrap/app.php`
|
||||
(`withExceptions`): в лог — полный текст; пользователю — generic
|
||||
*«Не удалось сохранить. Проверьте данные или попробуйте ещё раз»* (HTTP 422 для JSON-запросов).
|
||||
Никакой `SQLSTATE` в UI.
|
||||
- Фронт `NewProjectDialog`/`projectsStore` — убедиться, что 422 `errors`/`message`
|
||||
показываются как есть (уже умеет; правок минимум).
|
||||
|
||||
## Затрагиваемые компоненты
|
||||
|
||||
**Backend**
|
||||
|
||||
- `ProjectService`: +`delete()` (guard сделок + оркестрация шеринга), −`archive()`,
|
||||
bulk `archive`→`delete`, чистка `archived_at`/`archived` из update/resolveBulkScope,
|
||||
+дедуп источника в create/update.
|
||||
- `ProjectController`: `destroy()` → `delete()`, index `status=archived` убрать, bulk doc.
|
||||
- `BulkProjectActionRequest`: `archive`→`delete`, status `archived` убрать.
|
||||
- `Project` (модель): −scopeArchived, scopeActive упростить/убрать, −`archived_at` fillable+cast.
|
||||
- `ProjectResource`: −`archived_at`.
|
||||
- `DashboardController`, `SyncSupplierProjectsJob`: убрать `whereNull('archived_at')`.
|
||||
- Новый job `DeleteSupplierProjectJob` (или расширение существующего) — удаление донора у
|
||||
поставщика, когда источник остался без потребителей.
|
||||
- Миграция: `DROP COLUMN projects.archived_at` (+ опц. partial unique index источника) →
|
||||
schema bump v8.27 + `db/CHANGELOG_schema.md`.
|
||||
- `bootstrap/app.php`: глобальный handler `QueryException`.
|
||||
|
||||
**Frontend**
|
||||
|
||||
- `BulkActionsBar`: «Архивировать»→«Удалить» (+подтверждение/иконка).
|
||||
- `ProjectCard`, `ProjectDetailsDrawer`: «Архивировать»→«Удалить».
|
||||
- `ProjectsView`: `@archive`→`@delete`, убрать фильтр «Архивные».
|
||||
- `projectsStore`: `archive()`→`delete()`, bulk `archive`→`delete`, тип `archived_at` убрать.
|
||||
|
||||
## Тестирование (TDD)
|
||||
|
||||
- Backend (Pest): delete пустого проекта → 204 + строки нет; delete со сделками → 409/422,
|
||||
проект жив; шеринг — delete последнего потребителя → донор удалён у поставщика (mock client);
|
||||
delete при оставшихся потребителях → донор НЕ удалён, пере-синк; дедуп источника create/update
|
||||
→ 422; имя-дубль → 422 (не SQL); глобальный QueryException handler → generic message.
|
||||
- Frontend (Vitest): кнопки «Удалить» вместо «Архивировать»; нет фильтра «Архивные»;
|
||||
store.delete дёргает DELETE; ошибка сервера показывается человеческим текстом.
|
||||
- Live («проверь на практике» по каждой задаче, затем чистка тестовых данных):
|
||||
1) удаление пустого проекта + блок на проекте со сделками;
|
||||
2) два тенанта с одинаковым источником — создание проходит;
|
||||
3) попытка дубля источника у одного тенанта — отказ с понятным текстом;
|
||||
4) создание дубля имени — человеческое сообщение, не SQL.
|
||||
|
||||
## Вне scope (YAGNI)
|
||||
|
||||
- Restore/корзина удалённых проектов (удаление окончательное по решению заказчика).
|
||||
- Массовая авто-чистка уже-архивированных проектов dev-БД (разовая ручная операция).
|
||||
- `StatusPill 'archived'` mapping не трогаем (используется и для статусов сделок).
|
||||
@@ -176,6 +176,18 @@ pre-commit:
|
||||
cross-ref-checker detected version drift in §0 cross-refs.
|
||||
Update the offending file's cross-ref to match the target's header.
|
||||
|
||||
# 12b. extract-node-dormancy — регенерирует tools/.node-dormancy.json
|
||||
# из Tooling Прил.Н §3.5/§4.X (Pravila §16.4 v1.36, missed-activation
|
||||
# matcher). Учитывает два сигнала: dormant=true в строке атрибутов или
|
||||
# ключевое слово DEFERRED в колонке boundaries. Регенерированный JSON
|
||||
# авто-стейджится — попадает в тот же коммит, что и правки Tooling.
|
||||
- name: extract-node-dormancy
|
||||
glob: "docs/Tooling_v8_3.md"
|
||||
run: node tools/extract-node-dormancy.mjs && git add tools/.node-dormancy.json
|
||||
fail_text: |
|
||||
extract-node-dormancy failed.
|
||||
Проверьте формат 9-attribute table rows в docs/Tooling_v8_3.md.
|
||||
|
||||
# 13. observer-of-observer — счётчик чтений docs/observer/ + 54-week self-prune
|
||||
# (brain governance C3, ADR-011 spec §6.3). Скрипт всегда exit 0 (warn-only by
|
||||
# design). При observer infrastructure не используется >=54 недель — warn.
|
||||
@@ -196,6 +208,16 @@ pre-commit:
|
||||
observer-coverage-checker reports a gap (coverage or registration).
|
||||
See docs/observer/STATUS.md C5 row for details.
|
||||
|
||||
# 16. observer-chain-map-checker — brain governance C6 (chain attribution).
|
||||
# Сверяет tools/observer-chain-map.json с таблицей L1-L13 в
|
||||
# docs/routing-off-phase.md по множествам L-номеров (обе стороны). Блокирует
|
||||
# коммит при дрейфе: несуществующая L в JSON или потерянная цепочка из .md.
|
||||
- name: observer-chain-map-checker
|
||||
run: node tools/observer-chain-map-checker.mjs
|
||||
fail_text: |
|
||||
observer-chain-map-checker: дрейф chain-map <-> routing-off-phase.md.
|
||||
Обновите tools/observer-chain-map.json под таблицу L1-LN.
|
||||
|
||||
# Post-commit: regenerate STATUS.md dashboard (informational, not gate)
|
||||
post-commit:
|
||||
parallel: false
|
||||
|
||||
@@ -0,0 +1,69 @@
|
||||
{
|
||||
"#1": true,
|
||||
"#2": false,
|
||||
"#3": false,
|
||||
"#4": false,
|
||||
"#5": false,
|
||||
"#6": false,
|
||||
"#7": false,
|
||||
"#8": false,
|
||||
"#9": false,
|
||||
"#10": false,
|
||||
"#11": false,
|
||||
"#12": false,
|
||||
"#13": false,
|
||||
"#14": false,
|
||||
"#15": false,
|
||||
"#16": false,
|
||||
"#17": true,
|
||||
"#18": false,
|
||||
"#19": false,
|
||||
"#20": false,
|
||||
"#21": false,
|
||||
"#22": false,
|
||||
"#23": false,
|
||||
"#24": false,
|
||||
"#30": false,
|
||||
"#31": false,
|
||||
"#32": false,
|
||||
"#33": false,
|
||||
"#34": false,
|
||||
"#35": false,
|
||||
"#36": false,
|
||||
"#37": false,
|
||||
"#38": false,
|
||||
"#39": false,
|
||||
"#40": false,
|
||||
"#41": false,
|
||||
"#42": false,
|
||||
"#43": false,
|
||||
"#44": true,
|
||||
"#45": false,
|
||||
"#46": false,
|
||||
"#47": false,
|
||||
"#48": false,
|
||||
"#49": false,
|
||||
"#50": true,
|
||||
"#51": false,
|
||||
"#52": false,
|
||||
"#53": false,
|
||||
"#54": true,
|
||||
"#55": false,
|
||||
"#56": false,
|
||||
"#57": false,
|
||||
"#58": false,
|
||||
"#59": false,
|
||||
"#60": false,
|
||||
"#61": false,
|
||||
"#62": false,
|
||||
"#63": false,
|
||||
"#64": false,
|
||||
"#65": false,
|
||||
"#66": false,
|
||||
"#67": true,
|
||||
"#25": false,
|
||||
"#26": false,
|
||||
"#27": false,
|
||||
"#28": false,
|
||||
"#29": false
|
||||
}
|
||||
@@ -7,6 +7,7 @@
|
||||
* Security Guidance #40: pure parsing — no exec/execSync.
|
||||
*/
|
||||
import { readFileSync, existsSync } from 'fs';
|
||||
import { detectMissedActivations } from './missed-activations.mjs';
|
||||
|
||||
const SIZE_SMALL = 20;
|
||||
const SIZE_LARGE = 60;
|
||||
@@ -177,11 +178,23 @@ export function buildFactorMatrix(episodesWithOutcome) {
|
||||
matrix[fname][val][outcome] = (matrix[fname][val][outcome] || 0) + 1;
|
||||
}
|
||||
}
|
||||
// chain_ref is multi-value: a multi-chain episode counts once per chain;
|
||||
// null/absent → key "null". Handled outside FACTOR_FNS (single-value loop).
|
||||
matrix.chain_ref = {};
|
||||
for (const e of episodesWithOutcome) {
|
||||
const cr = (e.primary_rationale || {}).chain_ref;
|
||||
const outcome = e._inferredOutcome || 'unknown';
|
||||
const keys = Array.isArray(cr) && cr.length ? cr : ['null'];
|
||||
for (const k of keys) {
|
||||
matrix.chain_ref[k] = matrix.chain_ref[k] || {};
|
||||
matrix.chain_ref[k][outcome] = (matrix.chain_ref[k][outcome] || 0) + 1;
|
||||
}
|
||||
}
|
||||
return matrix;
|
||||
}
|
||||
|
||||
/** Full deterministic aggregation: dedup → infer outcomes → group → chains → matrix. */
|
||||
export function analyze(episodes) {
|
||||
/** Full deterministic aggregation: dedup → infer outcomes → group → chains → matrix → missed activations. */
|
||||
export function analyze(episodes, options = {}) {
|
||||
const deduped = dedupeEpisodes(episodes);
|
||||
const allNormal = deduped.filter((e) => !e.observer_error);
|
||||
// v1 episodes lack environment / prompt_signal / decision_provenance — they
|
||||
@@ -193,6 +206,8 @@ export function analyze(episodes) {
|
||||
episode._inferredOutcome = inferOutcome(episode, eps[i + 1]);
|
||||
});
|
||||
}
|
||||
const classificationMap = options.classificationMap || {};
|
||||
const dormancy = options.dormancy || {};
|
||||
return {
|
||||
episodeCount: normal.length,
|
||||
v1SkippedCount,
|
||||
@@ -200,6 +215,7 @@ export function analyze(episodes) {
|
||||
tasks: groupEpisodesToTasks(normal),
|
||||
causalChains: findCausalChains(normal),
|
||||
factorMatrix: buildFactorMatrix(normal),
|
||||
missedActivations: detectMissedActivations(normal, classificationMap, dormancy),
|
||||
};
|
||||
}
|
||||
|
||||
@@ -221,7 +237,17 @@ function loadEpisodes(files) {
|
||||
}
|
||||
|
||||
if (process.argv[1] && process.argv[1].replace(/\\/g, '/').endsWith('/brain-retro-analyzer.mjs')) {
|
||||
const result = analyze(loadEpisodes(process.argv.slice(2)));
|
||||
const classificationMap = (() => {
|
||||
try {
|
||||
return JSON.parse(readFileSync('tools/observer-classification-map.json', 'utf-8')).map || {};
|
||||
} catch { return {}; }
|
||||
})();
|
||||
const dormancy = (() => {
|
||||
try {
|
||||
return JSON.parse(readFileSync('tools/.node-dormancy.json', 'utf-8'));
|
||||
} catch { return {}; }
|
||||
})();
|
||||
const result = analyze(loadEpisodes(process.argv.slice(2)), { classificationMap, dormancy });
|
||||
console.log(JSON.stringify(result, null, 2));
|
||||
process.exit(0);
|
||||
}
|
||||
|
||||
@@ -230,6 +230,23 @@ describe('buildFactorMatrix — session_segment_turn axis rename (Task 14)', ()
|
||||
});
|
||||
});
|
||||
|
||||
describe('buildFactorMatrix — chain_ref axis (multi-chain)', () => {
|
||||
it('counts a multi-chain episode in each chain and null for direct', () => {
|
||||
const m = buildFactorMatrix([
|
||||
{ _inferredOutcome: 'success', primary_rationale: { node_chosen: 'discovery-interview', chain_ref: ['L1', 'L2'] } },
|
||||
{ _inferredOutcome: 'unknown', primary_rationale: { node_chosen: 'direct', chain_ref: null } },
|
||||
]);
|
||||
expect(m.chain_ref.L1).toEqual({ success: 1 });
|
||||
expect(m.chain_ref.L2).toEqual({ success: 1 });
|
||||
expect(m.chain_ref.null).toEqual({ unknown: 1 });
|
||||
});
|
||||
|
||||
it('chain_ref axis present via analyze()', () => {
|
||||
const result = analyze([ep({ primary_rationale: { node_chosen: 'billing-audit', chain_ref: ['L13'], task_classification: 'other' } })]);
|
||||
expect(result.factorMatrix).toHaveProperty('chain_ref');
|
||||
});
|
||||
});
|
||||
|
||||
describe('inferOutcome — neutral → soft_success (Task 16)', () => {
|
||||
it('returns soft_success when next prompt is neutral', () => {
|
||||
const a = { events: [] };
|
||||
@@ -246,3 +263,29 @@ describe('inferOutcome — neutral → soft_success (Task 16)', () => {
|
||||
expect(inferOutcome({ events: [] }, { prompt_signal: 'approval' })).toBe('success');
|
||||
});
|
||||
});
|
||||
|
||||
describe('analyze() — missedActivations integration', () => {
|
||||
it('includes missedActivations in the result', () => {
|
||||
const eps = [
|
||||
{
|
||||
schema_version: 2,
|
||||
task_id: 't1',
|
||||
timestamps: { started_at: '2026-05-21T00:00:00Z' },
|
||||
primary_rationale: { node_chosen: 'direct', task_classification: 'refactor' },
|
||||
events: [],
|
||||
},
|
||||
];
|
||||
const map = { refactor: ['#11'], other: [] };
|
||||
const dormancy = { '#11': false };
|
||||
const result = analyze(eps, { classificationMap: map, dormancy });
|
||||
expect(result.missedActivations).toBeDefined();
|
||||
expect(result.missedActivations.totalMissed).toBe(1);
|
||||
expect(result.missedActivations.byNode).toEqual({ '#11': 1 });
|
||||
});
|
||||
|
||||
it('returns missedActivations.totalMissed=0 when no map/dormancy provided', () => {
|
||||
const eps = [{ schema_version: 2, task_id: 't1', timestamps: { started_at: 'x' }, primary_rationale: { node_chosen: 'direct', task_classification: 'refactor' }, events: [] }];
|
||||
const result = analyze(eps);
|
||||
expect(result.missedActivations.totalMissed).toBe(0);
|
||||
});
|
||||
});
|
||||
|
||||
@@ -0,0 +1,39 @@
|
||||
#!/usr/bin/env node
|
||||
/**
|
||||
* Tooling Прил.Н dormancy extractor — emits {id: unavailable_bool} JSON for
|
||||
* the missed-activation matcher (Pravila §16.4 conditional rule).
|
||||
*
|
||||
* Two signals (either is sufficient) treat a node as effectively unavailable:
|
||||
* 1. `dormant: true` — Tooling-marked permanent dormancy (e.g. #17 pg_partman,
|
||||
* native Windows-PG cannot load the extension).
|
||||
* 2. `boundaries` column contains the word DEFERRED — node is registered
|
||||
* but not active (e.g. #44 Figma MCP "DEFERRED — нет Figma-аккаунта",
|
||||
* #50 Jupyter MCP, #54 n8n-mcp). The output key is still named "dormant"
|
||||
* for consumer simplicity — semantics: "node cannot be activated right
|
||||
* now, exclude from missed-activation counts".
|
||||
*
|
||||
* Parses 9-attribute table rows; ignores headers/separators/templates.
|
||||
*
|
||||
* Security Guidance #40: pure parsing — no exec/execSync.
|
||||
*/
|
||||
import { readFileSync, writeFileSync } from 'fs';
|
||||
|
||||
const ROW_RE = /^\|\s*#(\d+)\s*\|[^|]+\|[^|]+\|[^|]+\|[^|]+\|[^|]+\|([^|]+)\|\s*(true|false)\s*\|[^|]+\|$/gm;
|
||||
|
||||
export function extractDormancy(md) {
|
||||
const out = {};
|
||||
for (const m of md.matchAll(ROW_RE)) {
|
||||
const id = `#${m[1]}`;
|
||||
const boundaries = m[2];
|
||||
const tooledDormant = m[3] === 'true';
|
||||
out[id] = tooledDormant || /\bDEFERRED\b/.test(boundaries);
|
||||
}
|
||||
return out;
|
||||
}
|
||||
|
||||
if (process.argv[1] && process.argv[1].replace(/\\/g, '/').endsWith('/extract-node-dormancy.mjs')) {
|
||||
const src = readFileSync('docs/Tooling_v8_3.md', 'utf-8');
|
||||
const dormancy = extractDormancy(src);
|
||||
writeFileSync('tools/.node-dormancy.json', JSON.stringify(dormancy, null, 2) + '\n');
|
||||
console.log(`[extract-node-dormancy] OK — ${Object.keys(dormancy).length} nodes`);
|
||||
}
|
||||
@@ -0,0 +1,53 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { extractDormancy } from './extract-node-dormancy.mjs';
|
||||
|
||||
describe('extractDormancy', () => {
|
||||
it('returns false for a live row (dormant=false, no DEFERRED in boundaries)', () => {
|
||||
const md = [
|
||||
'#### #10 Laravel Boost',
|
||||
'',
|
||||
'**Атрибуты:**',
|
||||
'',
|
||||
'| id | name | kind | phase | subcategory | triggers | boundaries | dormant | last-touched |',
|
||||
'|---|---|---|---|---|---|---|---|---|',
|
||||
'| #10 | Laravel Boost | composer-dep | 1 | — | «SQL, Eloquent» | replaces #1 PG MCP | false | 2026-05-19 |',
|
||||
].join('\n');
|
||||
expect(extractDormancy(md)).toEqual({ '#10': false });
|
||||
});
|
||||
|
||||
it('returns true when Tooling marks dormant=true', () => {
|
||||
const md = '| #17 | pg_partman | binary-dep | 1 | — | «partition mgmt» | none | true | 2026-05-19 |';
|
||||
expect(extractDormancy(md)).toEqual({ '#17': true });
|
||||
});
|
||||
|
||||
it('returns true when boundaries contains DEFERRED (even if dormant=false)', () => {
|
||||
const md = '| #44 | Figma MCP | mcp | off-phase | design-tooling | «figma extract» | DEFERRED — нет Figma-аккаунта | false | 2026-05-19 |';
|
||||
expect(extractDormancy(md)).toEqual({ '#44': true });
|
||||
});
|
||||
|
||||
it('handles multiple nodes in one pass (mixed signals)', () => {
|
||||
const md = [
|
||||
'| #44 | Figma MCP | mcp | off-phase | design-tooling | «figma extract» | DEFERRED — нет Figma | false | 2026-05-17 |',
|
||||
'| #45 | Universal Icons MCP | mcp | off-phase | design-tooling | «svg search» | non-Lucide | false | 2026-05-17 |',
|
||||
].join('\n');
|
||||
expect(extractDormancy(md)).toEqual({ '#44': true, '#45': false });
|
||||
});
|
||||
|
||||
it('ignores header/separator rows', () => {
|
||||
const md = [
|
||||
'| id | name | kind | phase | subcategory | triggers | boundaries | dormant | last-touched |',
|
||||
'|---|---|---|---|---|---|---|---|---|',
|
||||
].join('\n');
|
||||
expect(extractDormancy(md)).toEqual({});
|
||||
});
|
||||
|
||||
it('ignores non-numeric ids (template placeholders)', () => {
|
||||
const md = '| #NN | <name> | <kind> | <phase> | <subcat or —> | «<triggers>» | <ADR-NNN or none> | false | 2026-05-19 |';
|
||||
expect(extractDormancy(md)).toEqual({});
|
||||
});
|
||||
|
||||
it('does NOT match the word DEFERRED inside a longer token (boundary check)', () => {
|
||||
const md = '| #99 | fake | mcp | off | tooling | «t» | NODEFERREDX prefix | false | 2026-05-19 |';
|
||||
expect(extractDormancy(md)).toEqual({ '#99': false });
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,46 @@
|
||||
#!/usr/bin/env node
|
||||
/**
|
||||
* Missed-activation matcher (Pravila §16.4 v1.36 conditional rule).
|
||||
* Pure deterministic — read-only, no exec, no fs.
|
||||
*
|
||||
* An episode is "missed" iff:
|
||||
* 1. schema_version === 2 (v1 lacks factor data)
|
||||
* 2. NOT observer_error
|
||||
* 3. primary_rationale.task_classification ∈ map AND map[c].length > 0
|
||||
* 4. primary_rationale.node_chosen === 'direct' (no explicit node)
|
||||
* 5. AT LEAST ONE recommended node is non-dormant
|
||||
*
|
||||
* Threshold: single episode (per Pravila §16.4 v1.36).
|
||||
* DEFERRED-узлы filtered via dormancy registry (dormancy[id] === true means
|
||||
* unavailable — covers both Tooling-marked dormant nodes and DEFERRED-in-
|
||||
* boundaries nodes, normalized by tools/extract-node-dormancy.mjs).
|
||||
*/
|
||||
|
||||
export function detectMissedActivations(episodes, classificationMap, dormancy) {
|
||||
const byNode = {};
|
||||
const byClassification = {};
|
||||
let totalMissed = 0;
|
||||
|
||||
for (const e of episodes) {
|
||||
if (!e || e.observer_error) continue;
|
||||
if (e.schema_version !== 2) continue;
|
||||
const pr = e.primary_rationale || {};
|
||||
const cls = pr.task_classification;
|
||||
const chosen = pr.node_chosen;
|
||||
if (!cls || chosen !== 'direct') continue;
|
||||
|
||||
const recommended = classificationMap[cls];
|
||||
if (!Array.isArray(recommended) || recommended.length === 0) continue;
|
||||
|
||||
const live = recommended.filter((id) => dormancy[id] === false);
|
||||
if (live.length === 0) continue;
|
||||
|
||||
totalMissed += 1;
|
||||
byClassification[cls] = (byClassification[cls] || 0) + 1;
|
||||
for (const id of live) {
|
||||
byNode[id] = (byNode[id] || 0) + 1;
|
||||
}
|
||||
}
|
||||
|
||||
return { totalMissed, byNode, byClassification };
|
||||
}
|
||||
@@ -0,0 +1,78 @@
|
||||
// tools/missed-activations.test.mjs
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { detectMissedActivations } from './missed-activations.mjs';
|
||||
|
||||
const map = {
|
||||
refactor: ['#11', '#12', '#43'],
|
||||
bugfix: ['#18', '#34'],
|
||||
feature: ['#19'],
|
||||
other: [],
|
||||
};
|
||||
const dormancy = { '#11': false, '#12': false, '#43': false, '#18': false, '#34': false, '#19': false };
|
||||
|
||||
function ep(classification, node_chosen) {
|
||||
return {
|
||||
schema_version: 2,
|
||||
primary_rationale: { task_classification: classification, node_chosen },
|
||||
};
|
||||
}
|
||||
|
||||
describe('detectMissedActivations', () => {
|
||||
it('counts an episode with profile classification + node_chosen=direct as missed', () => {
|
||||
const result = detectMissedActivations([ep('refactor', 'direct')], map, dormancy);
|
||||
expect(result.totalMissed).toBe(1);
|
||||
expect(result.byNode).toEqual({ '#11': 1, '#12': 1, '#43': 1 });
|
||||
});
|
||||
|
||||
it('does NOT count episode when the recommended node IS chosen', () => {
|
||||
const result = detectMissedActivations([ep('refactor', '#11')], map, dormancy);
|
||||
expect(result.totalMissed).toBe(0);
|
||||
});
|
||||
|
||||
it('does NOT count episode when classification=other (empty list)', () => {
|
||||
const result = detectMissedActivations([ep('other', 'direct')], map, dormancy);
|
||||
expect(result.totalMissed).toBe(0);
|
||||
});
|
||||
|
||||
it('excludes dormant (DEFERRED) nodes from recommendations', () => {
|
||||
const dorm = { ...dormancy, '#43': true };
|
||||
const result = detectMissedActivations([ep('refactor', 'direct')], map, dorm);
|
||||
expect(result.byNode).toEqual({ '#11': 1, '#12': 1 });
|
||||
expect(result.totalMissed).toBe(1);
|
||||
});
|
||||
|
||||
it('returns totalMissed=0 when ALL recommended nodes are dormant', () => {
|
||||
const dorm = { '#11': true, '#12': true, '#43': true };
|
||||
const result = detectMissedActivations([ep('refactor', 'direct')], map, dorm);
|
||||
expect(result.totalMissed).toBe(0);
|
||||
expect(result.byNode).toEqual({});
|
||||
});
|
||||
|
||||
it('ignores schema v1 episodes (no factor analysis)', () => {
|
||||
const v1 = { schema_version: 1, primary_rationale: { task_classification: 'refactor', node_chosen: 'direct' } };
|
||||
const result = detectMissedActivations([v1], map, dormancy);
|
||||
expect(result.totalMissed).toBe(0);
|
||||
});
|
||||
|
||||
it('ignores observer_error markers', () => {
|
||||
const err = { observer_error: true };
|
||||
const result = detectMissedActivations([err], map, dormancy);
|
||||
expect(result.totalMissed).toBe(0);
|
||||
});
|
||||
|
||||
it('ignores unknown classification (not in map)', () => {
|
||||
const result = detectMissedActivations([ep('unknown-bucket', 'direct')], map, dormancy);
|
||||
expect(result.totalMissed).toBe(0);
|
||||
});
|
||||
|
||||
it('aggregates byClassification breakdown for the report', () => {
|
||||
const eps = [
|
||||
ep('refactor', 'direct'),
|
||||
ep('refactor', 'direct'),
|
||||
ep('bugfix', 'direct'),
|
||||
];
|
||||
const result = detectMissedActivations(eps, map, dormancy);
|
||||
expect(result.byClassification).toEqual({ refactor: 2, bugfix: 1 });
|
||||
expect(result.totalMissed).toBe(3);
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,24 @@
|
||||
import { readFileSync } from 'node:fs';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
|
||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
||||
const DEFAULT_MAP_PATH = join(__dirname, 'observer-chain-map.json');
|
||||
|
||||
/** Load the node->chains map. Throws on missing/invalid JSON (caller handles). */
|
||||
export function loadChainMap(path = DEFAULT_MAP_PATH) {
|
||||
const raw = JSON.parse(readFileSync(path, 'utf8'));
|
||||
const map = new Map();
|
||||
for (const [node, chains] of Object.entries(raw)) {
|
||||
if (node === '_note') continue;
|
||||
if (Array.isArray(chains) && chains.length > 0) map.set(node, chains);
|
||||
}
|
||||
return map;
|
||||
}
|
||||
|
||||
/** node_chosen -> array of L-chains, or null if not in any chain. */
|
||||
export function chainsFor(node, map) {
|
||||
if (!node || typeof node !== 'string') return null;
|
||||
const chains = map.get(node);
|
||||
return chains && chains.length > 0 ? chains : null;
|
||||
}
|
||||
@@ -0,0 +1,32 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { loadChainMap, chainsFor } from './observer-chain-detector.mjs';
|
||||
|
||||
const map = loadChainMap();
|
||||
|
||||
describe('chainsFor', () => {
|
||||
it('returns chain array for a single-chain node', () => {
|
||||
expect(chainsFor('billing-audit', map)).toEqual(['L13']);
|
||||
});
|
||||
|
||||
it('returns all chains for a multi-chain node', () => {
|
||||
expect(chainsFor('discovery-interview', map)).toEqual(['L1', 'L2']);
|
||||
});
|
||||
|
||||
it('returns null for direct', () => {
|
||||
expect(chainsFor('direct', map)).toBeNull();
|
||||
});
|
||||
|
||||
it('returns null for an unknown node', () => {
|
||||
expect(chainsFor('totally-unknown-xyz', map)).toBeNull();
|
||||
});
|
||||
|
||||
it('returns null for empty/null/undefined', () => {
|
||||
expect(chainsFor('', map)).toBeNull();
|
||||
expect(chainsFor(null, map)).toBeNull();
|
||||
expect(chainsFor(undefined, map)).toBeNull();
|
||||
});
|
||||
|
||||
it('ignores the _note metadata key', () => {
|
||||
expect(chainsFor('_note', map)).toBeNull();
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,67 @@
|
||||
#!/usr/bin/env node
|
||||
/**
|
||||
* Brain governance controller C6 — chain-map sync checker.
|
||||
* Verifies tools/observer-chain-map.json against the L1-L13 table in
|
||||
* docs/routing-off-phase.md. Sync is checked by L-number sets (both
|
||||
* directions), not by node names — node_chosen values (skill-id) differ
|
||||
* from the human display names in the .md table. Pure fs/regex, no LLM.
|
||||
*/
|
||||
import { readFileSync } from 'node:fs';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
|
||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
||||
const MD_PATH = join(__dirname, '..', 'docs', 'routing-off-phase.md');
|
||||
const JSON_PATH = join(__dirname, 'observer-chain-map.json');
|
||||
|
||||
/** Extract the set of L-numbers ("L1".."L13") from the routing-off-phase.md table. */
|
||||
export function parseChainsFromMd(md) {
|
||||
const set = new Set();
|
||||
for (const line of md.split(/\r?\n/)) {
|
||||
const m = /^\|\s*(L\d+)\s*\|/.exec(line.trim());
|
||||
if (m) set.add(m[1]);
|
||||
}
|
||||
return set;
|
||||
}
|
||||
|
||||
/** Compare JSON L-numbers against the md set, both directions. */
|
||||
export function checkSync(jsonMap, mdSet) {
|
||||
const jsonSet = new Set();
|
||||
for (const [node, chains] of Object.entries(jsonMap)) {
|
||||
if (node === '_note') continue;
|
||||
if (Array.isArray(chains)) for (const c of chains) jsonSet.add(c);
|
||||
}
|
||||
const jsonOnly = [...jsonSet].filter((c) => !mdSet.has(c)); // ссылки на несуществующие L
|
||||
const mdOnly = [...mdSet].filter((c) => !jsonSet.has(c)); // потерянные цепочки
|
||||
return { ok: jsonOnly.length === 0 && mdOnly.length === 0, jsonOnly, mdOnly };
|
||||
}
|
||||
|
||||
/** CLI entry — exit 1 on drift with a human-readable message. */
|
||||
function main() {
|
||||
const md = readFileSync(MD_PATH, 'utf8');
|
||||
const jsonMap = JSON.parse(readFileSync(JSON_PATH, 'utf8'));
|
||||
const mdSet = parseChainsFromMd(md);
|
||||
if (mdSet.size === 0) {
|
||||
console.error(
|
||||
'[chain-map-checker] не нашёл ни одной L-строки в routing-off-phase.md — формат таблицы изменился?'
|
||||
);
|
||||
process.exit(1);
|
||||
}
|
||||
const res = checkSync(jsonMap, mdSet);
|
||||
if (res.ok) {
|
||||
console.log(`[chain-map-checker] OK — ${mdSet.size} chains in sync`);
|
||||
process.exit(0);
|
||||
}
|
||||
console.error('[chain-map-checker] дрейф маппинга chain-map <-> routing-off-phase.md:');
|
||||
if (res.jsonOnly.length)
|
||||
console.error(` JSON ссылается на отсутствующие в .md цепочки: ${res.jsonOnly.join(', ')}`);
|
||||
if (res.mdOnly.length)
|
||||
console.error(
|
||||
` В .md есть цепочки без записи в JSON: ${res.mdOnly.join(', ')} — добавьте узлы в tools/observer-chain-map.json`
|
||||
);
|
||||
process.exit(1);
|
||||
}
|
||||
|
||||
if (process.argv[1]?.endsWith('observer-chain-map-checker.mjs')) {
|
||||
main();
|
||||
}
|
||||
@@ -0,0 +1,46 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { parseChainsFromMd, checkSync } from './observer-chain-map-checker.mjs';
|
||||
|
||||
const SAMPLE_MD = [
|
||||
'| # | Цепочка | Зачем |',
|
||||
'|---|---|---|',
|
||||
'| L1 | `discovery-interview` (FEATURE) → `brainstorming` | text |',
|
||||
'| L2 | `audit-portal` | text |',
|
||||
'| L13 | `billing-audit` (#62) + `Pest` | text |',
|
||||
].join('\n');
|
||||
|
||||
describe('parseChainsFromMd', () => {
|
||||
it('extracts the set of L-numbers from the table', () => {
|
||||
expect(parseChainsFromMd(SAMPLE_MD)).toEqual(new Set(['L1', 'L2', 'L13']));
|
||||
});
|
||||
});
|
||||
|
||||
describe('checkSync', () => {
|
||||
it('passes when JSON L-numbers subset of md and md subset of json-union', () => {
|
||||
const mdSet = new Set(['L1', 'L2', 'L13']);
|
||||
const jsonMap = { a: ['L1'], b: ['L2'], c: ['L13'] };
|
||||
expect(checkSync(jsonMap, mdSet).ok).toBe(true);
|
||||
});
|
||||
|
||||
it('fails when JSON references a chain absent from md', () => {
|
||||
const mdSet = new Set(['L1', 'L2']);
|
||||
const jsonMap = { a: ['L1'], b: ['L99'] };
|
||||
const res = checkSync(jsonMap, mdSet);
|
||||
expect(res.ok).toBe(false);
|
||||
expect(res.jsonOnly).toContain('L99');
|
||||
});
|
||||
|
||||
it('fails when md has a chain not covered by any JSON entry', () => {
|
||||
const mdSet = new Set(['L1', 'L2', 'L14']);
|
||||
const jsonMap = { a: ['L1'], b: ['L2'] };
|
||||
const res = checkSync(jsonMap, mdSet);
|
||||
expect(res.ok).toBe(false);
|
||||
expect(res.mdOnly).toContain('L14');
|
||||
});
|
||||
|
||||
it('ignores the _note metadata key in the JSON map', () => {
|
||||
const mdSet = new Set(['L1']);
|
||||
const jsonMap = { _note: 'meta', a: ['L1'] };
|
||||
expect(checkSync(jsonMap, mdSet).ok).toBe(true);
|
||||
});
|
||||
});
|
||||
@@ -0,0 +1,41 @@
|
||||
{
|
||||
"_note": "node_chosen -> L-цепочки. Только узлы, входящие хотя бы в одну L1-L13. Узлы вне цепочек (direct, прочее) НЕ включаются -> chainsFor вернёт null. Имена ключей = реальные значения primary_rationale.node_chosen (skill-id из skill_invoked). MCP/agent-узлы (laravel-boost, openapi-mcp-server, api-docs, sentry-mcp, redis-mcp, pest, github-mcp) в node_chosen не появляются, но включены для полноты покрытия цепочек L1-L13 (контролёр C6 требует, чтобы каждая L из routing-off-phase.md была покрыта). Синхронизируется с docs/routing-off-phase.md через tools/observer-chain-map-checker.mjs.",
|
||||
"discovery-interview": ["L1", "L2"],
|
||||
"superpowers:brainstorming": ["L1"],
|
||||
"superpowers:writing-plans": ["L1"],
|
||||
"superpowers:subagent-driven-development": ["L1"],
|
||||
"audit-portal": ["L2"],
|
||||
"process-analysis": ["L3"],
|
||||
"process-modeling": ["L3", "L4"],
|
||||
"mermaid": ["L4"],
|
||||
"adr-kit:adr": ["L4", "L5"],
|
||||
"adr-kit:judge": ["L5"],
|
||||
"operations": ["L4"],
|
||||
"architecture-patterns:architecture-patterns": ["L5"],
|
||||
"deptrac": ["L5", "L14"],
|
||||
"rector": ["L14"],
|
||||
"php-insights": ["L14"],
|
||||
"larastan": ["L14"],
|
||||
"laravel-backend-patterns": ["L14"],
|
||||
"security-review": ["L6"],
|
||||
"openapi-mcp-server": ["L7"],
|
||||
"api-docs": ["L7"],
|
||||
"laravel-boost": ["L7", "L13"],
|
||||
"superpowers:systematic-debugging": ["L8"],
|
||||
"sentry-mcp": ["L8", "L13"],
|
||||
"redis-mcp": ["L8", "L13"],
|
||||
"ccpm": ["L9"],
|
||||
"product-management:brainstorm": ["L9"],
|
||||
"github-mcp": ["L9"],
|
||||
"promptfoo": ["L10"],
|
||||
"data-scientist": ["L10"],
|
||||
"claude-api": ["L10"],
|
||||
"skill-creator:skill-creator": ["L11"],
|
||||
"hookify:hookify": ["L11"],
|
||||
"plugin-dev:create-plugin": ["L11"],
|
||||
"claude-md-management:claude-md-improver": ["L12"],
|
||||
"claude-md-management:revise-claude-md": ["L12"],
|
||||
"billing-audit": ["L13"],
|
||||
"pest": ["L13"],
|
||||
"ru-tax-accounting": ["L13"]
|
||||
}
|
||||
@@ -0,0 +1,16 @@
|
||||
{
|
||||
"$schema_version": 1,
|
||||
"description": "Mapping from observer transcript-parser task_classification values to recommended Tooling Прил.Н node IDs. Source of truth for missed-activation detection (Pravila §16.4 conditional rule). 'other' deliberately empty — no recommendation, never counts as missed. DEFERRED-узлы filtered out by .node-dormancy.json at runtime.",
|
||||
"map": {
|
||||
"refactor": ["#11", "#12", "#43", "#64", "#65"],
|
||||
"bugfix": ["#18", "#34"],
|
||||
"feature": ["#19"],
|
||||
"planning": ["#19", "#41", "#42"],
|
||||
"memory-sync": ["#33"],
|
||||
"monitoring": ["#34", "#35"],
|
||||
"analysis": ["#25", "#39", "#53"],
|
||||
"cleanup": ["#11", "#12"],
|
||||
"question": ["#60"],
|
||||
"other": []
|
||||
}
|
||||
}
|
||||
@@ -17,6 +17,8 @@
|
||||
*/
|
||||
import { readFileSync, existsSync } from 'fs';
|
||||
import { join } from 'path';
|
||||
import { detectMissedActivations } from './missed-activations.mjs';
|
||||
import { dedupeEpisodes } from './brain-retro-analyzer.mjs';
|
||||
|
||||
/**
|
||||
* @param {number} episodeCount - episodes in the current month JSONL
|
||||
@@ -59,6 +61,31 @@ function countEpisodes(root) {
|
||||
return readFileSync(file, 'utf-8').trim().split('\n').filter(Boolean).length;
|
||||
}
|
||||
|
||||
function loadEpisodes(root) {
|
||||
const month = new Date().toISOString().slice(0, 7);
|
||||
const file = join(root, 'docs', 'observer', `episodes-${month}.jsonl`);
|
||||
if (!existsSync(file)) return [];
|
||||
const out = [];
|
||||
for (const line of readFileSync(file, 'utf-8').split('\n')) {
|
||||
const t = line.trim();
|
||||
if (!t) continue;
|
||||
try { out.push(JSON.parse(t)); } catch { /* skip */ }
|
||||
}
|
||||
return out;
|
||||
}
|
||||
|
||||
function loadClassificationMap(root) {
|
||||
try {
|
||||
return JSON.parse(readFileSync(join(root, 'tools', 'observer-classification-map.json'), 'utf-8')).map || {};
|
||||
} catch { return {}; }
|
||||
}
|
||||
|
||||
function loadDormancy(root) {
|
||||
try {
|
||||
return JSON.parse(readFileSync(join(root, 'tools', '.node-dormancy.json'), 'utf-8'));
|
||||
} catch { return {}; }
|
||||
}
|
||||
|
||||
function readSettings(root) {
|
||||
try {
|
||||
return JSON.parse(readFileSync(join(root, '.claude', 'settings.json'), 'utf-8'));
|
||||
@@ -81,14 +108,23 @@ export function runCoverageChecker(root = process.cwd()) {
|
||||
const hookRegistered = isObserverStopRegistered(settings);
|
||||
const coverage = checkCoverage(countEpisodes(root), hookRegistered);
|
||||
const registration = checkRegistration(settings, existsSync(join(root, '.git', 'hooks', 'post-commit')));
|
||||
return { coverage, registration };
|
||||
const episodes = loadEpisodes(root).filter((e) => e && e.schema_version === 2 && !e.observer_error);
|
||||
const missed = detectMissedActivations(
|
||||
dedupeEpisodes(episodes),
|
||||
loadClassificationMap(root),
|
||||
loadDormancy(root)
|
||||
);
|
||||
return { coverage, registration, missed };
|
||||
}
|
||||
|
||||
if (process.argv[1] && process.argv[1].replace(/\\/g, '/').endsWith('/observer-coverage-checker.mjs')) {
|
||||
const { coverage, registration } = runCoverageChecker();
|
||||
const { coverage, registration, missed } = runCoverageChecker();
|
||||
if (!coverage.ok) console.warn(`[observer-coverage-checker] WARN — coverage: ${coverage.detail}`);
|
||||
if (!registration.ok) console.warn(`[observer-coverage-checker] WARN — registration: ${registration.detail}`);
|
||||
if (coverage.ok && registration.ok) {
|
||||
if (missed.totalMissed > 0) {
|
||||
console.warn(`[observer-coverage-checker] WARN — missed activations: ${missed.totalMissed} (see /brain-retro)`);
|
||||
}
|
||||
if (coverage.ok && registration.ok && missed.totalMissed === 0) {
|
||||
console.log(`[observer-coverage-checker] OK — ${coverage.detail}; ${registration.detail}`);
|
||||
}
|
||||
process.exit(0); // warn-only — never blocks a commit
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { checkCoverage, checkRegistration } from './observer-coverage-checker.mjs';
|
||||
import { checkCoverage, checkRegistration, runCoverageChecker } from './observer-coverage-checker.mjs';
|
||||
|
||||
describe('checkCoverage', () => {
|
||||
// COV-1 fix: the metric is driven by Stop-hook registration, NOT by recent
|
||||
@@ -59,3 +59,13 @@ describe('checkRegistration', () => {
|
||||
expect(checkRegistration({}, false).ok).toBe(false);
|
||||
});
|
||||
});
|
||||
|
||||
describe('runCoverageChecker — missed surfacing', () => {
|
||||
it('returns a missed field with totalMissed', () => {
|
||||
const { missed } = runCoverageChecker();
|
||||
expect(missed).toBeDefined();
|
||||
expect(typeof missed.totalMissed).toBe('number');
|
||||
expect(missed.byNode).toBeDefined();
|
||||
expect(missed.byClassification).toBeDefined();
|
||||
});
|
||||
});
|
||||
|
||||
@@ -0,0 +1,58 @@
|
||||
#!/usr/bin/env node
|
||||
/**
|
||||
* One-shot retrofill: add primary_rationale.chain_ref to existing v2 episodes
|
||||
* in docs/observer/episodes-*.jsonl. Idempotent (skips lines that already have
|
||||
* chain_ref), atomic per file (tmp + rename). Pure fs, no LLM.
|
||||
*
|
||||
* Usage: node tools/observer-retrofill-chain-ref.mjs [--dry-run]
|
||||
*/
|
||||
import { readFileSync, writeFileSync, renameSync, readdirSync } from 'node:fs';
|
||||
import { fileURLToPath } from 'node:url';
|
||||
import { dirname, join } from 'node:path';
|
||||
import { loadChainMap, chainsFor } from './observer-chain-detector.mjs';
|
||||
|
||||
const __dirname = dirname(fileURLToPath(import.meta.url));
|
||||
const OBS_DIR = join(__dirname, '..', 'docs', 'observer');
|
||||
|
||||
/** Add chain_ref to a single parsed episode object (pure). Idempotent. */
|
||||
export function retrofillLine(ep, map) {
|
||||
if (!ep || ep.schema_version !== 2 || !ep.primary_rationale) return ep;
|
||||
if ('chain_ref' in ep.primary_rationale) return ep; // idempotent
|
||||
ep.primary_rationale.chain_ref = chainsFor(ep.primary_rationale.node_chosen, map);
|
||||
return ep;
|
||||
}
|
||||
|
||||
/** Process one JSONL file atomically (tmp + rename). Returns {changed, total}. */
|
||||
export function retrofillFile(path, map, { dryRun = false } = {}) {
|
||||
const lines = readFileSync(path, 'utf8').split(/\r?\n/);
|
||||
let changed = 0;
|
||||
let total = 0;
|
||||
const out = lines.map((line) => {
|
||||
if (!line.trim()) return line;
|
||||
total++;
|
||||
const ep = JSON.parse(line);
|
||||
const before = ep.primary_rationale && 'chain_ref' in ep.primary_rationale;
|
||||
const next = retrofillLine(ep, map);
|
||||
const after = next.primary_rationale && 'chain_ref' in next.primary_rationale;
|
||||
if (!before && after) changed++;
|
||||
return JSON.stringify(next);
|
||||
});
|
||||
if (!dryRun && changed > 0) {
|
||||
const tmp = `${path}.tmp`;
|
||||
writeFileSync(tmp, out.join('\n'), 'utf8');
|
||||
renameSync(tmp, path);
|
||||
}
|
||||
return { changed, total };
|
||||
}
|
||||
|
||||
function main() {
|
||||
const dryRun = process.argv.includes('--dry-run');
|
||||
const map = loadChainMap();
|
||||
const files = readdirSync(OBS_DIR).filter((f) => /^episodes-\d{4}-\d{2}\.jsonl$/.test(f));
|
||||
for (const f of files) {
|
||||
const { changed, total } = retrofillFile(join(OBS_DIR, f), map, { dryRun });
|
||||
console.log(`${dryRun ? '[dry-run] ' : ''}${f}: ${changed}/${total} lines get chain_ref`);
|
||||
}
|
||||
}
|
||||
|
||||
if (process.argv[1]?.endsWith('observer-retrofill-chain-ref.mjs')) main();
|
||||
@@ -0,0 +1,28 @@
|
||||
import { describe, it, expect } from 'vitest';
|
||||
import { retrofillLine } from './observer-retrofill-chain-ref.mjs';
|
||||
import { loadChainMap } from './observer-chain-detector.mjs';
|
||||
|
||||
const map = loadChainMap();
|
||||
|
||||
describe('retrofillLine', () => {
|
||||
it('adds chain_ref to a v2 episode with a known node', () => {
|
||||
const ep = { schema_version: 2, primary_rationale: { node_chosen: 'billing-audit' } };
|
||||
const out = retrofillLine(ep, map);
|
||||
expect(out.primary_rationale.chain_ref).toEqual(['L13']);
|
||||
});
|
||||
|
||||
it('sets chain_ref null for a direct v2 episode', () => {
|
||||
const ep = { schema_version: 2, primary_rationale: { node_chosen: 'direct' } };
|
||||
expect(retrofillLine(ep, map).primary_rationale.chain_ref).toBeNull();
|
||||
});
|
||||
|
||||
it('is idempotent — does not overwrite existing chain_ref', () => {
|
||||
const ep = { schema_version: 2, primary_rationale: { node_chosen: 'direct', chain_ref: ['L1'] } };
|
||||
expect(retrofillLine(ep, map).primary_rationale.chain_ref).toEqual(['L1']);
|
||||
});
|
||||
|
||||
it('skips v1 episodes (no schema_version 2)', () => {
|
||||
const ep = { foo: 'bar' };
|
||||
expect(retrofillLine(ep, map)).toEqual({ foo: 'bar' });
|
||||
});
|
||||
});
|
||||
@@ -16,6 +16,14 @@
|
||||
*/
|
||||
|
||||
import { detectChoiceProvenance, detectAskUserQuestionChoice } from './observer-choice-detector.mjs';
|
||||
import { loadChainMap, chainsFor } from './observer-chain-detector.mjs';
|
||||
|
||||
let CHAIN_MAP = null;
|
||||
try {
|
||||
CHAIN_MAP = loadChainMap();
|
||||
} catch {
|
||||
CHAIN_MAP = new Map(); // битый/отсутствующий JSON -> chainsFor вернёт null, observer не падает
|
||||
}
|
||||
|
||||
const SUPERPOWERS_PREFIX = 'superpowers:';
|
||||
|
||||
@@ -694,6 +702,7 @@ export function parseTranscript(transcriptText, fallbackSessionId = null) {
|
||||
return {
|
||||
step: 1,
|
||||
node_chosen: skills.length > 0 ? skills[0] : 'direct',
|
||||
chain_ref: chainsFor(skills.length > 0 ? skills[0] : 'direct', CHAIN_MAP),
|
||||
triggers_matched: merge(extractTriggers(turn), tag ? tag.triggers : []),
|
||||
candidates_considered: merge(extractCandidates(turn), tag ? tag.candidates : []),
|
||||
boundaries_applied: merge(extractBoundaries(turn), tag ? tag.boundaries : []),
|
||||
|
||||
@@ -106,6 +106,25 @@ describe('parseTranscript', () => {
|
||||
expect(parseTranscript(t).primary_rationale.node_chosen).toBe('direct');
|
||||
});
|
||||
|
||||
it('attaches chain_ref for a node that belongs to a chain', () => {
|
||||
const t = jsonl([
|
||||
userPrompt('go', '2026-05-19T10:00:00Z'),
|
||||
assistantTurn(
|
||||
[{ type: 'tool_use', id: 't1', name: 'Skill', input: { skill: 'billing-audit' } }],
|
||||
'2026-05-19T10:01:00Z'
|
||||
),
|
||||
]);
|
||||
expect(parseTranscript(t).primary_rationale.chain_ref).toEqual(['L13']);
|
||||
});
|
||||
|
||||
it('sets chain_ref null for a direct episode', () => {
|
||||
const t = jsonl([
|
||||
userPrompt('go', '2026-05-19T10:00:00Z'),
|
||||
assistantTurn([{ type: 'tool_use', id: 't1', name: 'Read', input: {} }], '2026-05-19T10:01:00Z'),
|
||||
]);
|
||||
expect(parseTranscript(t).primary_rationale.chain_ref).toBeNull();
|
||||
});
|
||||
|
||||
it('hard_floor invoked when a superpowers skill is used', () => {
|
||||
const t = jsonl([
|
||||
userPrompt('go', '2026-05-19T10:00:00Z'),
|
||||
|
||||
@@ -10,6 +10,8 @@ function iconFor(status) {
|
||||
|
||||
export function renderStatus(inputs) {
|
||||
const { now, c1, c2, c3, c5, observer, lastRetroDaysAgo } = inputs;
|
||||
const c6 = inputs.c6 || { status: 'ok', detail: '—' };
|
||||
const missed = inputs.missed || { totalMissed: 0, byNode: {}, byClassification: {} };
|
||||
const retroLine = (lastRetroDaysAgo === null || lastRetroDaysAgo === undefined)
|
||||
? 'never'
|
||||
: `${lastRetroDaysAgo} day(s) ago`;
|
||||
@@ -24,13 +26,14 @@ Last updated: ${now}
|
||||
| C3 Observer-of-observer | ${iconFor(c3.status)} | ${c3.detail || '—'} |
|
||||
| C4 Сигнальный статус | ✅ | This file (self-reference) |
|
||||
| C5 Observer-coverage | ${iconFor(c5.status)} | ${c5.detail || '—'} |
|
||||
| C6 Chain map sync | ${iconFor(c6.status)} | ${c6.detail || '—'} |
|
||||
|
||||
## Метрики (информационные, не алерты)
|
||||
|
||||
- Observer evidence: ${observer.episodeCount} episodes this month, ${observer.observerErrors} observer_error markers, ${observer.piiMatches} PII matches before filter
|
||||
- Legacy v1 episodes (not in factor analysis): ${observer.v1Episodes || 0}
|
||||
- Last /brain-retro: ${retroLine}
|
||||
- Использование узлов: см. \`/brain-retro\` (раз в спринт). **Неиспользованные узлы — не проблема** (capability-readiness; см. memory \`feedback_brain_unused_tools_not_problem\` — outside-repo memory store).
|
||||
- Использование узлов: см. \`/brain-retro\` (раз в спринт). missed_activations: ${missed.totalMissed}. **Неиспользованные узлы — не алерт, если профильной задачи не было** (Pravila §16.4 v1.36; capability-readiness; см. memory \`feedback_brain_unused_tools_not_problem\` — outside-repo memory store).
|
||||
|
||||
## Алерт-индикаторы
|
||||
|
||||
@@ -104,22 +107,26 @@ function countV1Episodes() {
|
||||
|
||||
if (process.argv[1] && process.argv[1].replace(/\\/g, '/').endsWith('/status-md-generator.mjs')) {
|
||||
const cov = runCoverageChecker();
|
||||
const c5ok = cov.coverage.ok && cov.registration.ok;
|
||||
const c5ok = cov.coverage.ok && cov.registration.ok && cov.missed.totalMissed === 0;
|
||||
const c5detail = [
|
||||
cov.coverage.detail,
|
||||
cov.registration.detail,
|
||||
cov.missed.totalMissed > 0 ? `${cov.missed.totalMissed} missed activation(s) — see /brain-retro` : null,
|
||||
].filter(Boolean).join(' · ');
|
||||
const inputs = {
|
||||
now: new Date().toISOString(),
|
||||
c1: runControllerNode(['tools/l1-watcher.mjs']),
|
||||
c2: runControllerNode(['tools/cross-ref-checker.mjs']),
|
||||
c3: runControllerNode(['tools/observer-of-observer.mjs', 'check']),
|
||||
c5: {
|
||||
status: c5ok ? 'ok' : 'warn',
|
||||
detail: [cov.coverage.detail, cov.registration.detail].join(' · '),
|
||||
},
|
||||
c5: { status: c5ok ? 'ok' : 'warn', detail: c5detail },
|
||||
c6: runControllerNode(['tools/observer-chain-map-checker.mjs']),
|
||||
observer: {
|
||||
episodeCount: countEpisodes(),
|
||||
observerErrors: countObserverErrors(),
|
||||
piiMatches: countPiiMatches(),
|
||||
v1Episodes: countV1Episodes(),
|
||||
},
|
||||
missed: cov.missed,
|
||||
lastRetroDaysAgo: lastRetroDaysAgo(),
|
||||
};
|
||||
const md = renderStatus(inputs);
|
||||
|
||||
@@ -7,7 +7,9 @@ const baseInputs = (overrides = {}) => ({
|
||||
c2: { status: 'ok', detail: '0 version drift' },
|
||||
c3: { status: 'ok', detail: 'last read today' },
|
||||
c5: { status: 'ok', detail: 'coverage OK · registration OK' },
|
||||
c6: { status: 'ok', detail: '14 chains in sync' },
|
||||
observer: { episodeCount: 12, observerErrors: 0, piiMatches: 0 },
|
||||
missed: { totalMissed: 0, byNode: {}, byClassification: {} },
|
||||
...overrides,
|
||||
});
|
||||
|
||||
@@ -23,6 +25,11 @@ describe('renderStatus', () => {
|
||||
expect(md).toContain('12 episodes');
|
||||
});
|
||||
|
||||
it('includes a C6 chain-map row', () => {
|
||||
const md = renderStatus(baseInputs());
|
||||
expect(md).toContain('| C6 Chain map sync | ✅');
|
||||
});
|
||||
|
||||
it('shows a warn status for the coverage controller', () => {
|
||||
const md = renderStatus(baseInputs({ c5: { status: 'warn', detail: '3 commits, 0 episodes' } }));
|
||||
expect(md).toContain('| C5 Observer-coverage | ⚠️');
|
||||
@@ -38,9 +45,10 @@ describe('renderStatus', () => {
|
||||
expect(md).toContain('| C1 L1-watcher | 🔴');
|
||||
});
|
||||
|
||||
it('mentions the capability-readiness behavioral rule', () => {
|
||||
it('mentions the conditional capability-readiness behavioral rule (§16.4 v1.36)', () => {
|
||||
const md = renderStatus(baseInputs());
|
||||
expect(md).toContain('capability-readiness');
|
||||
expect(md).toContain('Неиспользованные узлы — не алерт');
|
||||
expect(md).toContain('если профильной задачи не было');
|
||||
expect(md).toContain('feedback_brain_unused_tools_not_problem');
|
||||
});
|
||||
|
||||
@@ -75,3 +83,29 @@ describe('renderStatus — v1 episodes count surface (Task 18)', () => {
|
||||
expect(md).toMatch(/Legacy v1 episodes \(not in factor analysis\):\s*0/);
|
||||
});
|
||||
});
|
||||
|
||||
describe('renderStatus — missed activations (Task 7, Pravila §16.4 v1.36)', () => {
|
||||
it('renders missed_activations: 0 when there are no misses', () => {
|
||||
const md = renderStatus(baseInputs());
|
||||
expect(md).toContain('missed_activations: 0');
|
||||
});
|
||||
|
||||
it('renders missed_activations: N when misses occur', () => {
|
||||
const md = renderStatus(baseInputs({
|
||||
missed: { totalMissed: 3, byNode: { '#11': 2, '#12': 1 }, byClassification: { refactor: 3 } },
|
||||
}));
|
||||
expect(md).toContain('missed_activations: 3');
|
||||
});
|
||||
|
||||
it('keeps C5 ✅ when controller is ok and no misses', () => {
|
||||
const md = renderStatus(baseInputs());
|
||||
expect(md).toContain('| C5 Observer-coverage | ✅');
|
||||
});
|
||||
|
||||
it('honors the c5 status override (warn) regardless of missed count', () => {
|
||||
const md = renderStatus(baseInputs({
|
||||
c5: { status: 'warn', detail: '16 missed activation(s)' },
|
||||
}));
|
||||
expect(md).toContain('| C5 Observer-coverage | ⚠️');
|
||||
});
|
||||
});
|
||||
|
||||
@@ -9,7 +9,7 @@
|
||||
**перепроверять реальной командой**, не доверять снимку вслепую.
|
||||
- Обновляется по команде заказчика **«обнови эталон»**.
|
||||
|
||||
**Снимок снят:** 21.05.2026 (ночь, после сквозного чек-листа всего портала + 6 фиксов: 3 stale эпик-теста под схему v8.26 + 3 UI-бага; запушено в main; volatile §1–§4 пересверены).
|
||||
**Снимок снят:** 21.05.2026 (день, после фичи «удаление проектов вместо архива + дедуп источника + человеческие ошибки» — 10 коммитов FF в main; volatile §1–§4 пересверены).
|
||||
|
||||
---
|
||||
|
||||
@@ -18,9 +18,9 @@
|
||||
- Git-корень репозитория — папка `Документация/` (**не** `app/`).
|
||||
- Remote: `CoralMinister/lidpotok` (приватный).
|
||||
- Текущая локальная ветка: **`feat/project-migration-redesign`**.
|
||||
- Локальный HEAD = origin/main HEAD = **`31b5355`** (style(backend): pint concat_space — tip параллельного эпика A1 backend-tooling; мои 6 фиксов чек-листа ниже на `a0e18a1..b7466eb`; сверять `git log -1 origin/main`).
|
||||
- Push паттерн: `git push origin <ветка>:main`. Мой push 21.05: `a0e18a1..b7466eb` (4 коммита FF: `95ee664` sync stale эпик-тестов + schema header v8.26, `ba49805` dashboard greeting, `17e3c04` topbar title, `b7466eb` admin mock-счётчики; pre-push gitleaks-full 1119/0, lychee 64/0). **Поверх** параллельная сессия влила эпик A1 backend-tooling `b7466eb..31b5355` (8 коммитов: Rector #64 / PHP Insights #65 / laravel-backend-patterns skill #66 / #67 + ADR-013 + нормативка Tooling v2.19 / PSR v3.19 / Pravila v1.35 / CLAUDE v2.22 + карта 137→141 узлов) — детали в её памяти/нормативке.
|
||||
- Pre-push lefthook прошёл чисто (gitleaks-full 1115/0, lychee 64/0) — `--no-verify` не понадобился.
|
||||
- Локальный HEAD = origin/main HEAD = **`22e81cc`** (chore(gitleaks): allowlist Nuclei docs false-positive — tail коммит моего эпика «удаление проектов»; сверять `git log -1 origin/main`).
|
||||
- Push паттерн: `git push origin <ветка>:main`. Мой push 21.05 (день): `3b6992d..22e81cc` (10 коммитов FF — спека+план+7 имплементаций+gitleaks-allowlist); pre-push gitleaks-full 1155/0, lychee 64/0.
|
||||
- Pre-push lefthook прошёл чисто после добавления `.gitleaksignore` (Nuclei docs `-u http://...` ловился rule `curl-auth-user` — false-positive из параллельной ветки `worktree-a8-infosec-tooling`, добавлен fingerprint).
|
||||
- **Незакоммиченного нет** (фикс + тест запушены).
|
||||
- Прочее незакоммиченное: `docs/observer/STATUS.md` + `episodes-2026-05.jsonl` (hook-артефакты brain governance, не мои); untracked artifacts (см. §4).
|
||||
- Остатки от rebase (безопасно): `stash@{0}` с не-моими hook/parallel-артефактами (+5 других parallel-стэшей); `/tmp/plan4-rebase-bak/` — устаревшие untracked-копии 2 observer-файлов (committed-версии origin/main авторитетнее).
|
||||
@@ -74,10 +74,11 @@
|
||||
## 5. Ключевые факты (стабильно)
|
||||
|
||||
- Структура: git-корень `Документация/` · Laravel-приложение `app/` · фронтенд `app/resources/js/`
|
||||
· сборка фронтенда → `app/public/build/` · схема БД `db/schema.sql` (**фактическая v8.26** —
|
||||
Plans 1+3 эпика project-migration-redesign добавили `supplier_projects.subject_code` +
|
||||
· сборка фронтенда → `app/public/build/` · схема БД `db/schema.sql` (**фактическая v8.27** —
|
||||
21.05.2026 дропнута колонка `projects.archived_at` (фича архива заменена настоящим удалением);
|
||||
v8.26 наследие — Plans 1+3 эпика project-migration-redesign: `supplier_projects.subject_code` +
|
||||
`project_supplier_links` pivot + `deals.subject_code` + seed `supplier_export_mode` + CHECK chk_deals_subject_code.
|
||||
Schema header синхронизирован на v8.26 (65 таблиц / 123 индекса, commit `95ee664`); CHANGELOG_schema.md содержит v8.26 entries).
|
||||
Header v8.27 в commit `07d7387`; CHANGELOG_schema.md содержит v8.27 + v8.26 entries).
|
||||
- Стек: Laravel 13 + PHP 8.3 · Vue 3 + Vuetify 3 (не Tailwind/Inertia) · PostgreSQL 16 · Redis.
|
||||
- **Демо-доступ к порталу:** 5 изолированных компаний — `admin@demo.local` (Demo Tenant, 4 проекта) + `manager1@demo.local` (Компания Ивана) + `manager2@demo.local` (Компания Анны) + `manager3@demo.local` (Компания Петра) + `manager4@demo.local` (Компания Марии). Пароль у всех **`password`**. Каждый логин видит только своё. Админка `/admin/*` в local открыта любому залогиненному (`EnsureSaasAdmin` — стаб local/testing).
|
||||
- Поставщик лидов: `crm.bp-gr.ru` (учётка в `.env` `SUPPLIER_*`); портал — Vue 2 + Element UI;
|
||||
@@ -87,6 +88,19 @@
|
||||
|
||||
## 6. Текущие рабочие нити (детали — в памяти Claude)
|
||||
|
||||
- **Удаление проектов вместо архива + дедуп источника + человеческие ошибки — DONE+ЗАПУШЕНО** (21.05.2026 день,
|
||||
`3b6992d..22e81cc`, 10 коммитов FF в main). По заказу: (1) кнопка «архивировать» заменена на настоящее удаление
|
||||
(если по проекту есть сделки — блок с понятным сообщением, иначе hard delete + удаление донора у поставщика
|
||||
с учётом шеринга — пере-синк остатка, если другие клиенты ещё пользуются источником); (2) подтверждена возможность
|
||||
шеринга одинаковых источников между разными клиентами (модель не меняется); (3) добавлен запрет дубля источника
|
||||
внутри одного клиента (раньше не было); (4) сырой `SQLSTATE` больше не уходит в UI — глобальный handler
|
||||
`QueryException` + app-level pre-checks (имя/источник) с понятными сообщениями. Колонка `projects.archived_at`
|
||||
и все `scopeActive/scopeArchived/->archived()` снесены, миграция `2026_05_21_000000`, schema bump v8.26→**v8.27**.
|
||||
Новый job `App\Jobs\Supplier\DeleteSupplierProjectJob` — удаление/пере-синк донора при шеринге.
|
||||
Регрессия: Pest filter=Project **229/228+1sk/0**, Vitest **896/3sk/0**, vue-tsc 0, Vite build OK, Pint/Larastan 0.
|
||||
Живая проверка 5 сценариев (имя/источник дубль / sharing / удаление пустого / блок при сделках) — все ✓ с чисткой.
|
||||
Спека: `docs/superpowers/specs/2026-05-21-project-delete-dedup-errors-design.md`, план:
|
||||
`docs/superpowers/plans/2026-05-21-project-delete-dedup-errors.md`. Память: `project_delete_dedup_errors.md`.
|
||||
- **Сквозной чек-лист всего портала + 6 фиксов — ЗАПУШЕНО** (21.05.2026, `a0e18a1..b7466eb`). Прогон всех ~30 экранов
|
||||
(Playwright) + полная регрессия (Pest 1010/0, Vitest 895/0, pint/larastan/vue-tsc/build чисто). Портал здоров: все
|
||||
экраны рендерятся, консоль чистая. Найдено и исправлено 3 UI-бага: (1) дашборд здоровался захардкоженным «Доброе утро,
|
||||
|
||||
Reference in New Issue
Block a user