Audit log is permanently dead — setLogBasePath() never called

[igouss]

Summary

setLogBasePath() in workflow-logger.ts is never called from any production code path. The entire persistent audit log feature (.gsd/audit-log.jsonl) silently does nothing.

Impact

Critical — The S2-T3 objective from the Single-Writer Engine v3 plan ("audit entries survive context resets") is undelivered despite the code being written.

All logWarning()/logError() calls go to stderr and the in-memory _buffer only. After context reset, every warning and error is gone. The forensics story promised by the engine is broken.

Root Cause

1. setLogBasePath(basePath) at workflow-logger.ts:56 exists and works correctly, but is called from zero production files. _auditBasePath stays null for the entire process lifetime.
2. Every _push() call at workflow-logger.ts:233 skips the if (_auditBasePath) branch — no entries are ever flushed to disk.
3. _resetLogs() at workflow-logger.ts:200-203 nulls out _auditBasePath even if it were somehow set, so the next unit start wipes the path. This field should be stable for the process lifetime.

Fix

1. Call setLogBasePath(basePath) in the engine init path — wherever openDatabase() is called or a basePath is first established.
2. Change _resetLogs() to only clear _buffer, not _auditBasePath.

Files

• src/resources/extensions/gsd/workflow-logger.ts — lines 56, 200-203, 233

Confidence

97% — verified via codebase-wide search for setLogBasePath references.

[igouss]

Investigation Results — CONFIRMED

Evidence

setLogBasePath is defined but never called:

• Defined at workflow-logger.ts:56-58 — sets _auditBasePath
• _auditBasePath initialized as null at line 49
• Zero calls to setLogBasePath anywhere in src/ — only the definition exists

_push() persistence is gated on _auditBasePath:

• workflow-logger.ts:232-242 — if (_auditBasePath) guards the appendFileSync to .gsd/audit-log.jsonl
• Since _auditBasePath is always null, this branch never executes

_resetLogs() kills the audit path:

• workflow-logger.ts:200-203 — _resetLogs() sets _auditBasePath = null (line 202)
• This is called at the start of each unit per the module header comment (line 13)
• Even if setLogBasePath were called once at init, the first _resetLogs() would wipe it

Verified Fix Requirements

1. Wire setLogBasePath(basePath) into the engine init path (wherever openDatabase() is called)
2. Change _resetLogs() to only clear _buffer — _auditBasePath should be stable for the process lifetime
3. These are two independent changes that both need to happen

Severity

Critical — the entire S2-T3 deliverable ("audit entries survive context resets") is non-functional.