singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

fix: lazy-open GSD database on first tool call in manual sessions (#1606)

Browse source 
In manual sessions (no auto-mode), bootstrapAutoSession never runs, so the
GSD database is never opened. This causes gsd_save_decision,
gsd_update_requirement, and gsd_save_summary tools to always fail with
'GSD database is not available'.

Add ensureDbOpen() helper that checks isDbAvailable() first, then tries to
open the DB from the expected .gsd/gsd.db path if it exists. All three tool
handlers now use this helper instead of the check-only pattern.

The fix is backward-compatible: in auto-mode the DB is already open, so
ensureDbOpen() returns true immediately on the isDbAvailable() check.
This commit is contained in:
Marek Hovadík 2026-03-20 17:27:04 +01:00 • committed by GitHub
parent 7c25036ed9
commit f2bcd049ae
1 changed files with 21 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

37
src/resources/extensions/gsd/index.ts
View file

@ -92,6 +92,23 @@ function warnDeprecatedAgentInstructions(): void {
// ── Depth verification state ──────────────────────────────────────────────
let depthVerificationDone = false;
// ── DB lazy-open helper ───────────────────────────────────────────────────
// In manual sessions (no auto-mode), the DB is never opened by bootstrapAutoSession.
// This helper ensures the DB is lazily opened on first tool call that needs it.
async function ensureDbOpen(): Promise<boolean> {
try {
const db = await import("./gsd-db.js");
if (db.isDbAvailable()) return true;
const dbPath = join(process.cwd(), ".gsd", "gsd.db");
if (existsSync(dbPath)) {
return db.openDatabase(dbPath);
}
return false;
} catch {
return false;
}
}
// ── Queue phase tracking ──────────────────────────────────────────────────
// When true, the LLM is in a queue flow writing CONTEXT.md files.
// The write-gate applies during queue flows just like discussion flows.
@ -300,12 +317,8 @@ export default function (pi: ExtensionAPI) {
when_context: Type.Optional(Type.String({ description: "When/context for the decision (e.g. milestone ID)" })),
}),
async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
// Check DB availability
let dbAvailable = false;
try {
const db = await import("./gsd-db.js");
dbAvailable = db.isDbAvailable();
} catch { /* dynamic import failed */ }
// Ensure DB is open (lazy-open on first tool call in manual sessions)
const dbAvailable = await ensureDbOpen();
if (!dbAvailable) {
return {
@ -367,11 +380,7 @@ export default function (pi: ExtensionAPI) {
supporting_slices: Type.Optional(Type.String({ description: "Supporting slices" })),
}),
async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
let dbAvailable = false;
try {
const db = await import("./gsd-db.js");
dbAvailable = db.isDbAvailable();
} catch { /* dynamic import failed */ }
const dbAvailable = await ensureDbOpen();
if (!dbAvailable) {
return {
@ -441,11 +450,7 @@ export default function (pi: ExtensionAPI) {
content: Type.String({ description: "The full markdown content of the artifact" }),
}),
async execute(_toolCallId, params, _signal, _onUpdate, _ctx) {
let dbAvailable = false;
try {
const db = await import("./gsd-db.js");
dbAvailable = db.isDbAvailable();
} catch { /* dynamic import failed */ }
const dbAvailable = await ensureDbOpen();
if (!dbAvailable) {
return {