dohertj2/ScadaBridge
1
0
Fork 0
Code Issues 1 Pull Requests Actions Packages Projects Releases Wiki Activity

docs(archreview): sync PLAN-05 tasks.json — Wave 6 complete (T6, T8, T14 → 20/25)

Browse Source 
Claude-Session: https://claude.ai/code/session_01MtdgwpEeCUn6cUA5f1LMPj
This commit is contained in:
Joseph Doherty
2026-07-09 17:44:58 -04:00
parent 7c74bbe4b5
commit 8e92198078
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
archreview/plans/PLAN-05-templates-deployment-transport.md.tasks.json
+4 -4
View File
@@ -6,15 +6,15 @@
{ "id": 3, "subject": "Task 3: Carry MinTimeBetweenRuns/ExecutionTimeoutSeconds through import", "status": "completed", "blockedBy": [2] },
{ "id": 4, "subject": "Task 4: Add LockedInDerived to template child DTOs and apply paths", "status": "completed", "blockedBy": [3] },
{ "id": 5, "subject": "Task 5: Transport template NativeAlarmSources end-to-end", "status": "completed", "blockedBy": [4] },
{ "id": 6, "subject": "Task 6: Single-source per-entity field comparers for diff + sync", "status": "pending", "blockedBy": [5] },
{ "id": 6, "subject": "Task 6: Single-source per-entity field comparers for diff + sync", "status": "completed", "blockedBy": [5] },
{ "id": 7, "subject": "Task 7: Export real AreaName (finish half-shipped Area transport)", "status": "completed", "blockedBy": [] },
{ "id": 8, "subject": "Task 8: Round-trip export→import equivalence test suite", "status": "pending", "blockedBy": [5, 6, 7] },
{ "id": 8, "subject": "Task 8: Round-trip export→import equivalence test suite", "status": "completed", "blockedBy": [5, 6, 7] },
{ "id": 9, "subject": "Task 9: Fix repeated-composition member loss in FlatteningService", "status": "completed", "blockedBy": [] },
{ "id": 10, "subject": "Task 10: Fix CollisionDetector blindness under repeated composition", "status": "completed", "blockedBy": [9] },
{ "id": 11, "subject": "Task 11: Add NativeAlarmSources to RevisionHashService (planned hash migration)", "status": "completed", "blockedBy": [13] },
{ "id": 12, "subject": "Task 12: Add NativeAlarmSources to DiffService", "status": "completed", "blockedBy": [13] },
{ "id": 13, "subject": "Task 13: Enforce locks on native alarm source overrides", "status": "completed", "blockedBy": [] },
{ "id": 14, "subject": "Task 14: Script-artifact invalidation contract — Commons seam + Transport publisher (plan-06 handoff)", "status": "pending", "blockedBy": [] },
{ "id": 14, "subject": "Task 14: Script-artifact invalidation contract — Commons seam + Transport publisher (plan-06 handoff)", "status": "completed", "blockedBy": [] },
{ "id": 15, "subject": "Task 15: Cache script compile verdicts by code hash", "status": "completed", "blockedBy": [] },
{ "id": 16, "subject": "Task 16: Skip script compilation on read-only staleness/comparison paths", "status": "completed", "blockedBy": [] },
{ "id": 17, "subject": "Task 17: Isolate the post-success deployment audit write", "status": "completed", "blockedBy": [16] },
@@ -27,5 +27,5 @@
{ "id": 24, "subject": "Task 24: Low-severity cleanup batch (session cap, full error lists, semantic-validator warning)", "status": "pending", "blockedBy": [15, 19] },
{ "id": 25, "subject": "Task 25: Design-doc + CLAUDE.md sync sweep and deferred-decision ledger", "status": "pending", "blockedBy": [1, 2, 3, 4, 5, 6, 7, 8, 9, 10, 11, 12, 13, 14, 15, 16, 17, 18, 19, 20, 21, 22, 23, 24] }
],
"lastUpdated": "2026-07-09 (Wave 5 COMPLETE: T5 done — 17/25. T5 (transport template NativeAlarmSources end-to-end) in-plan-worktree, high-risk: new TemplateNativeAlarmSourceDto + init-only TemplateDto.NativeAlarmSources (empty default, additive) + export/FromBundleContent/BuildTemplate wiring + new SyncTemplateNativeAlarmSourcesAsync Overwrite child-sync + ArtifactDiff.CompareTemplate DiffChildren. Proven by end-to-end flatten test (instance native-alarm-source override that dangled pre-fix now resolves). Code-reviewed (code-reviewer subagent: no substantive defects; folded 2 test-coverage gaps — no-change zero-audit + IsInherited/LockedInDerived fidelity — into a follow-up test commit). Commits: e4136cb9 T5 + 7b6d6bb8 test-hardening. Combined build 0/0; Transport 131 unit / 74 integ. HEAD 7b6d6bb8 (pre-tasks-sync). Ready set now: {6 (single-source TemplateChildEquality — BundleImporter+ArtifactDiff, in-worktree serial after T5✓), 14 (Commons+Host+BundleImporter publish hook)}. Chain remaining: T6→T19→T20→T23 all in-worktree serial via BundleImporter/ArtifactDiff; T8 (round-trip reflection suite, test-only) unblocks after T6 (5,7 already done). T24 needs T19. T25 docs last.) PREV: (Wave 4 COMPLETE: T4, T11, T12 done — 16/25. T4 (LockedInDerived on template child DTOs: EntityDtos+EntitySerializer+BundleImporter, additive trailing bool) done in-plan-worktree (opens the serial BundleImporter chain, high-risk); T11 (RevisionHashService NativeAlarmSources, null-when-empty surgical hash, pinned baseline) + T12 (DiffService+ConfigurationDiff NativeAlarmSources, generic DiffEntry<T>) via off-main isolated agents — BOTH had to cherry-pick T13 (57f7f657) into their off-main base first since ResolvedNativeAlarmSource.IsLocked is absent on main; grafted their fix commits (0cd01772→0302694f, cbc70c4d→d8332199) onto the plan branch which already has T13. Combined build 0/0 + suites green: Transport 130 unit/69 integ, TemplateEngine 463, DeploymentManager 112. HEAD d8332199. Ready set now: {5 (BundleImporter chain, in-worktree serial after T4✓), 14 (Commons+Host+BundleImporter publish hook — serialize w/ T5 chain), 24 (blockedBy 15✓,19-pending → still blocked)}. Chain remaining: T5→T6→T19→T20→T23 all in-plan-worktree serial through BundleImporter.cs; T8 (round-trip test suite) after 5,6,7✓. T14 also edits BundleImporter → serialize with the chain, NOT parallel.)"
"lastUpdated": "2026-07-09 (Wave 6 COMPLETE: T6, T8, T14 done — 20/25. T6 (single-source TemplateChildEquality: new file + ArtifactDiff + 4 BundleImporter sync predicates all route through it, complete field set incl. alarm on-trigger by name) in-worktree. T14 (ScriptArtifactsChanged Commons seam + Host InProcessScriptArtifactChangeBus singleton + BundleImporter optional dep + post-commit publish; consumer=plan06) in-worktree. T8 (round-trip reflection equivalence suite, 10 facts/23 entity types) via off-main isolated agent (reset to plan HEAD 38edfefc, committed e498c6e1) — it SURFACED 5 genuine silent-data-loss holes (3 red tests); per T8's 'fix in importer/exporter not the test' discipline I FIXED all 5 (commit 7c74bbe4): ES MaxRetries/RetryDelay on Add+Overwrite, ES methods on Add (new PersistAddedExternalSystemMethodsAsync post-flush pass — no parent nav), NotificationRecipient export (Include), TemplateFolder.ParentFolderId on Add (new ResolveFolderParentEdgesAsync rewire). Commits: 38edfefc T6, 2c839def T14, e498c6e1 T8-suite, 7c74bbe4 T8-fixes. Combined build 0/0; Transport 135 unit/86 integ, ConfigurationDatabase 337, NotificationService 55, Host bus 5 + CompositionRoot 96, ArtifactDiff 25. HEAD 7c74bbe4 (pre-tasks-sync). Ready set now: {19 (import blocker heuristic — BundleImporter, in-worktree serial after T6✓), 24 (blockedBy 15✓+19 → ready AFTER 19)}. Chain remaining: T19→T20→T23 all in-worktree serial via BundleImporter; T23 fixes ArtifactDiff placeholder-identity; T24 low-sev cleanup batch; T25 docs+ledger sweep LAST (blockedBy all). Only 5 tasks left: 19,20,23,24,25.) PREV: (Wave 5 COMPLETE: T5 done — 17/25. T5 (transport template NativeAlarmSources end-to-end) in-plan-worktree, high-risk: new TemplateNativeAlarmSourceDto + init-only TemplateDto.NativeAlarmSources (empty default, additive) + export/FromBundleContent/BuildTemplate wiring + new SyncTemplateNativeAlarmSourcesAsync Overwrite child-sync + ArtifactDiff.CompareTemplate DiffChildren. Proven by end-to-end flatten test (instance native-alarm-source override that dangled pre-fix now resolves). Code-reviewed (code-reviewer subagent: no substantive defects; folded 2 test-coverage gaps — no-change zero-audit + IsInherited/LockedInDerived fidelity — into a follow-up test commit). Commits: e4136cb9 T5 + 7b6d6bb8 test-hardening. Combined build 0/0; Transport 131 unit / 74 integ. HEAD 7b6d6bb8 (pre-tasks-sync). Ready set now: {6 (single-source TemplateChildEquality — BundleImporter+ArtifactDiff, in-worktree serial after T5✓), 14 (Commons+Host+BundleImporter publish hook)}. Chain remaining: T6→T19→T20→T23 all in-worktree serial via BundleImporter/ArtifactDiff; T8 (round-trip reflection suite, test-only) unblocks after T6 (5,7 already done). T24 needs T19. T25 docs last.) PREV: (Wave 4 COMPLETE: T4, T11, T12 done — 16/25. T4 (LockedInDerived on template child DTOs: EntityDtos+EntitySerializer+BundleImporter, additive trailing bool) done in-plan-worktree (opens the serial BundleImporter chain, high-risk); T11 (RevisionHashService NativeAlarmSources, null-when-empty surgical hash, pinned baseline) + T12 (DiffService+ConfigurationDiff NativeAlarmSources, generic DiffEntry<T>) via off-main isolated agents — BOTH had to cherry-pick T13 (57f7f657) into their off-main base first since ResolvedNativeAlarmSource.IsLocked is absent on main; grafted their fix commits (0cd01772→0302694f, cbc70c4d→d8332199) onto the plan branch which already has T13. Combined build 0/0 + suites green: Transport 130 unit/69 integ, TemplateEngine 463, DeploymentManager 112. HEAD d8332199. Ready set now: {5 (BundleImporter chain, in-worktree serial after T4✓), 14 (Commons+Host+BundleImporter publish hook — serialize w/ T5 chain), 24 (blockedBy 15✓,19-pending → still blocked)}. Chain remaining: T5→T6→T19→T20→T23 all in-plan-worktree serial through BundleImporter.cs; T8 (round-trip test suite) after 5,6,7✓. T14 also edits BundleImporter → serialize with the chain, NOT parallel.)"
}