hungry-ghost-hive 0.48.0 → 0.49.0

package/src/db/queries/teams.ts

@@ -1,8 +1,8 @@
 // Licensed under the Hungry Ghost Hive License. See LICENSE.
 
 import { nanoid } from 'nanoid';
-import type { Database } from 'sql.js';
-import { queryAll, queryOne, run, type TeamRow } from '../client.js';
+import { type TeamRow } from '../client.js';
+import type { DatabaseProvider } from '../provider.js';
 
 export type { TeamRow };
 
@@ -12,12 +12,14 @@ export interface CreateTeamInput {
   name: string;
 }
 
-export function createTeam(db: Database, input: CreateTeamInput): TeamRow {
+export async function createTeam(
+  provider: DatabaseProvider,
+  input: CreateTeamInput
+): Promise<TeamRow> {
   const id = `team-${nanoid(10)}`;
   const now = new Date().toISOString();
 
-  run(
-    db,
+  await provider.run(
     `
     INSERT INTO teams (id, repo_url, repo_path, name, created_at)
     VALUES (?, ?, ?, ?, ?)
@@ -25,21 +27,27 @@ export function createTeam(db: Database, input: CreateTeamInput): TeamRow {
     [id, input.repoUrl, input.repoPath, input.name, now]
   );
 
-  return getTeamById(db, id)!;
+  return (await getTeamById(provider, id))!;
 }
 
-export function getTeamById(db: Database, id: string): TeamRow | undefined {
-  return queryOne<TeamRow>(db, 'SELECT * FROM teams WHERE id = ?', [id]);
+export async function getTeamById(
+  provider: DatabaseProvider,
+  id: string
+): Promise<TeamRow | undefined> {
+  return await provider.queryOne<TeamRow>('SELECT * FROM teams WHERE id = ?', [id]);
 }
 
-export function getTeamByName(db: Database, name: string): TeamRow | undefined {
-  return queryOne<TeamRow>(db, 'SELECT * FROM teams WHERE name = ?', [name]);
+export async function getTeamByName(
+  provider: DatabaseProvider,
+  name: string
+): Promise<TeamRow | undefined> {
+  return await provider.queryOne<TeamRow>('SELECT * FROM teams WHERE name = ?', [name]);
 }
 
-export function getAllTeams(db: Database): TeamRow[] {
-  return queryAll<TeamRow>(db, 'SELECT * FROM teams ORDER BY created_at');
+export async function getAllTeams(provider: DatabaseProvider): Promise<TeamRow[]> {
+  return await provider.queryAll<TeamRow>('SELECT * FROM teams ORDER BY created_at');
 }
 
-export function deleteTeam(db: Database, id: string): void {
-  run(db, 'DELETE FROM teams WHERE id = ?', [id]);
+export async function deleteTeam(provider: DatabaseProvider, id: string): Promise<void> {
+  await provider.run('DELETE FROM teams WHERE id = ?', [id]);
 }

package/src/integrations/jira/repair.test.ts

@@ -3,11 +3,11 @@
 import { mkdtempSync, writeFileSync } from 'fs';
 import { tmpdir } from 'os';
 import { join } from 'path';
-import type { Database } from 'sql.js';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { TokenStore } from '../../auth/token-store.js';
 import type { JiraConfig } from '../../config/schema.js';
 import { queryAll, run } from '../../db/client.js';
+import { SqliteProvider } from '../../db/provider.js';
 import { createStory, getStoryById } from '../../db/queries/stories.js';
 import { createTestDatabase } from '../../db/queries/test-helpers.js';
 import { repairMissedAssignmentHooks } from './sync.js';
@@ -20,7 +20,7 @@ vi.mock('./stories.js');
 vi.mock('./transitions.js');
 
 describe('repairMissedAssignmentHooks', () => {
-  let db: Database;
+  let db: SqliteProvider;
   let envDir: string;
 
   const baseConfig: JiraConfig = {
@@ -53,34 +53,35 @@ describe('repairMissedAssignmentHooks', () => {
 
   function addJiraColumnsToStories(): void {
     // The test helper schema only has jira_issue_key. Add the other Jira columns.
-    const columnInfo = queryAll<{ name: string }>(db, 'PRAGMA table_info(stories)');
+    const columnInfo = queryAll<{ name: string }>(db.db, 'PRAGMA table_info(stories)');
     const columnNames = columnInfo.map(c => c.name);
 
     if (!columnNames.includes('jira_issue_id')) {
-      run(db, 'ALTER TABLE stories ADD COLUMN jira_issue_id TEXT');
+      run(db.db, 'ALTER TABLE stories ADD COLUMN jira_issue_id TEXT');
     }
     if (!columnNames.includes('jira_project_key')) {
-      run(db, 'ALTER TABLE stories ADD COLUMN jira_project_key TEXT');
+      run(db.db, 'ALTER TABLE stories ADD COLUMN jira_project_key TEXT');
     }
     if (!columnNames.includes('jira_subtask_key')) {
-      run(db, 'ALTER TABLE stories ADD COLUMN jira_subtask_key TEXT');
+      run(db.db, 'ALTER TABLE stories ADD COLUMN jira_subtask_key TEXT');
     }
     if (!columnNames.includes('jira_subtask_id')) {
-      run(db, 'ALTER TABLE stories ADD COLUMN jira_subtask_id TEXT');
+      run(db.db, 'ALTER TABLE stories ADD COLUMN jira_subtask_id TEXT');
     }
   }
 
   beforeEach(async () => {
-    db = await createTestDatabase();
+    const rawDb = await createTestDatabase();
+    db = new SqliteProvider(rawDb);
     envDir = mkdtempSync(join(tmpdir(), 'hive-repair-test-'));
     // Create a manager agent for logging purposes
-    db.run(`INSERT INTO agents (id, type, status) VALUES ('manager', 'tech_lead', 'idle')`);
+    db.db.run(`INSERT INTO agents (id, type, status) VALUES ('manager', 'tech_lead', 'idle')`);
     // Create a team
-    db.run(
+    db.db.run(
       `INSERT INTO teams (id, repo_url, repo_path, name) VALUES ('team-test', 'https://github.com/test/test.git', 'repos/test', 'test-team')`
     );
     // Create a working agent
-    db.run(
+    db.db.run(
       `INSERT INTO agents (id, type, status, team_id, tmux_session) VALUES ('agent-senior-1', 'senior', 'working', 'team-test', 'hive-senior-test-team')`
     );
     // Add missing Jira columns to test schema
@@ -94,12 +95,12 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('skips stories that already have a subtask key', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'Already has subtask',
       description: 'Test',
     });
     run(
-      db,
+      db.db,
       `UPDATE stories SET jira_issue_key = ?, external_issue_key = ?, assigned_agent_id = ?, jira_subtask_key = ?, external_subtask_key = ?, status = ? WHERE id = ?`,
       ['TEST-1', 'TEST-1', 'agent-senior-1', 'TEST-2', 'TEST-2', 'in_progress', story.id]
     );
@@ -110,11 +111,11 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('skips stories without jira_issue_key', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'No Jira key',
       description: 'Test',
     });
-    run(db, `UPDATE stories SET assigned_agent_id = ?, status = ? WHERE id = ?`, [
+    run(db.db, `UPDATE stories SET assigned_agent_id = ?, status = ? WHERE id = ?`, [
       'agent-senior-1',
       'in_progress',
       story.id,
@@ -126,12 +127,12 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('skips stories without assigned_agent_id', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'Not assigned',
       description: 'Test',
     });
     run(
-      db,
+      db.db,
       `UPDATE stories SET jira_issue_key = ?, external_issue_key = ?, status = ? WHERE id = ?`,
       ['TEST-1', 'TEST-1', 'planned', story.id]
     );
@@ -142,12 +143,12 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('repairs assigned story missing subtask by creating subtask and posting comment', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'Needs repair',
       description: 'Test',
     });
     run(
-      db,
+      db.db,
       `UPDATE stories SET jira_issue_key = ?, external_issue_key = ?, jira_project_key = ?, external_project_key = ?, assigned_agent_id = ?, status = ? WHERE id = ?`,
       ['TEST-10', 'TEST-10', 'TEST', 'TEST', 'agent-senior-1', 'in_progress', story.id]
     );
@@ -199,7 +200,7 @@ describe('repairMissedAssignmentHooks', () => {
     );
 
     // Verify subtask key was persisted to DB (both legacy and external columns)
-    const updatedStory = getStoryById(db, story.id);
+    const updatedStory = await getStoryById(db, story.id);
     expect(updatedStory?.jira_subtask_key).toBe('TEST-11');
     expect(updatedStory?.jira_subtask_id).toBe('20001');
     expect(updatedStory?.external_subtask_key).toBe('TEST-11');
@@ -216,12 +217,12 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('does not create duplicate subtasks when repair runs twice', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'Repair idempotency',
       description: 'Test',
     });
     run(
-      db,
+      db.db,
       `UPDATE stories SET jira_issue_key = ?, external_issue_key = ?, assigned_agent_id = ?, status = ? WHERE id = ?`,
       ['TEST-20', 'TEST-20', 'agent-senior-1', 'in_progress', story.id]
     );
@@ -258,12 +259,12 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('skips merged stories', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'Merged story',
       description: 'Test',
     });
     run(
-      db,
+      db.db,
       `UPDATE stories SET jira_issue_key = ?, external_issue_key = ?, assigned_agent_id = ?, status = ? WHERE id = ?`,
       ['TEST-30', 'TEST-30', 'agent-senior-1', 'merged', story.id]
     );
@@ -274,12 +275,12 @@ describe('repairMissedAssignmentHooks', () => {
   });
 
   it('handles API errors gracefully without crashing', async () => {
-    const story = createStory(db, {
+    const story = await createStory(db, {
       title: 'Error story',
       description: 'Test',
     });
     run(
-      db,
+      db.db,
       `UPDATE stories SET jira_issue_key = ?, external_issue_key = ?, assigned_agent_id = ?, status = ? WHERE id = ?`,
       ['TEST-40', 'TEST-40', 'agent-senior-1', 'in_progress', story.id]
     );

package/src/integrations/jira/stories.test.ts

@@ -3,10 +3,10 @@
 import { mkdtempSync, rmSync } from 'fs';
 import { tmpdir } from 'os';
 import { join } from 'path';
-import type { Database } from 'sql.js';
 import { afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 import { TokenStore } from '../../auth/token-store.js';
 import type { JiraConfig } from '../../config/schema.js';
+import { SqliteProvider } from '../../db/provider.js';
 import { createStory } from '../../db/queries/stories.js';
 import { createTestDatabase } from '../../db/queries/test-helpers.js';
 import { AnthropicProvider } from '../../llm/anthropic.js';
@@ -260,7 +260,7 @@ describe('Jira Story Creation', () => {
   });
 
   describe('Story Points Fallback', () => {
-    let db: Database;
+    let db: SqliteProvider;
     let envDir: string;
     let tokenStore: TokenStore;
 
@@ -274,7 +274,8 @@ describe('Jira Story Creation', () => {
     };
 
     beforeEach(async () => {
-      db = await createTestDatabase();
+      const rawDb = await createTestDatabase();
+      db = new SqliteProvider(rawDb);
       envDir = mkdtempSync(join(tmpdir(), 'hive-test-'));
       tokenStore = new TokenStore(envDir);
 
@@ -296,7 +297,7 @@ describe('Jira Story Creation', () => {
 
     it('should use complexity_score when story_points is null', async () => {
       const { updateStory } = await import('../../db/queries/stories.js');
-      const story = createStory(db, {
+      const story = await createStory(db, {
         title: 'Test Story',
         description: 'Test description',
         requirementId: null,
@@ -305,15 +306,14 @@ describe('Jira Story Creation', () => {
       });
 
       // Update with complexity_score but leave story_points null
-      updateStory(db, story.id, {
+      await updateStory(db, story.id, {
         complexityScore: 5,
         storyPoints: null,
       });
 
-      const updatedStory = (await import('../../db/queries/stories.js')).getStoryById(
-        db,
-        story.id
-      )!;
+      const updatedStory = (await (
+        await import('../../db/queries/stories.js')
+      ).getStoryById(db, story.id))!;
 
       await syncStoryToJira(db, tokenStore, mockConfig, updatedStory);
 
@@ -329,7 +329,7 @@ describe('Jira Story Creation', () => {
 
     it('should use story_points when both story_points and complexity_score are set', async () => {
       const { updateStory } = await import('../../db/queries/stories.js');
-      const story = createStory(db, {
+      const story = await createStory(db, {
         title: 'Test Story',
         description: 'Test description',
         requirementId: null,
@@ -338,15 +338,14 @@ describe('Jira Story Creation', () => {
       });
 
       // Update with both complexity_score and story_points
-      updateStory(db, story.id, {
+      await updateStory(db, story.id, {
         complexityScore: 5,
         storyPoints: 8,
       });
 
-      const updatedStory = (await import('../../db/queries/stories.js')).getStoryById(
-        db,
-        story.id
-      )!;
+      const updatedStory = (await (
+        await import('../../db/queries/stories.js')
+      ).getStoryById(db, story.id))!;
 
       await syncStoryToJira(db, tokenStore, mockConfig, updatedStory);
 
@@ -361,7 +360,7 @@ describe('Jira Story Creation', () => {
     });
 
     it('should not include story points field when both are null', async () => {
-      const story = createStory(db, {
+      const story = await createStory(db, {
         title: 'Test Story',
         description: 'Test description',
         requirementId: null,

package/src/integrations/jira/stories.ts

@@ -1,10 +1,10 @@
 // Licensed under the Hungry Ghost Hive License. See LICENSE.
 
-import type { Database } from 'sql.js';
 import { loadEnvIntoProcess } from '../../auth/env-store.js';
 import type { TokenStore } from '../../auth/token-store.js';
 import type { JiraConfig } from '../../config/schema.js';
 import type { StoryRow } from '../../db/client.js';
+import type { DatabaseProvider } from '../../db/provider.js';
 import { createSyncRecord, getSyncRecordByEntity } from '../../db/queries/integration-sync.js';
 import { updateRequirement, type RequirementRow } from '../../db/queries/requirements.js';
 import { getStoryById, getStoryDependencies, updateStory } from '../../db/queries/stories.js';
@@ -204,7 +204,7 @@ export async function tryMoveToActiveSprint(
  * and moves stories to the active sprint.
  */
 export async function syncRequirementToJira(
-  db: Database,
+  db: DatabaseProvider,
   tokenStore: TokenStore,
   config: JiraConfig,
   requirement: RequirementRow,
@@ -260,14 +260,14 @@ export async function syncRequirementToJira(
       result.epicId = epic.id;
 
       // Update requirement with epic info (provider-agnostic)
-      updateRequirement(db, requirement.id, {
+      await updateRequirement(db, requirement.id, {
         externalEpicKey: epic.key,
         externalEpicId: epic.id,
         externalProvider: 'jira',
       });
 
       // Record sync state
-      createSyncRecord(db, {
+      await createSyncRecord(db, {
         entityType: 'requirement',
         entityId: requirement.id,
         provider: 'jira',
@@ -284,7 +284,7 @@ export async function syncRequirementToJira(
   const createdStoryKeys: string[] = [];
 
   for (const storyId of storyIds) {
-    const story = getStoryById(db, storyId);
+    const story = await getStoryById(db, storyId);
     if (!story) {
       result.errors.push(`Story ${storyId} not found in local DB`);
       continue;
@@ -296,7 +296,7 @@ export async function syncRequirementToJira(
       storyKeyMap[storyId] = story.jira_issue_key;
       continue;
     }
-    const existingSync = getSyncRecordByEntity(db, 'story', storyId, 'jira');
+    const existingSync = await getSyncRecordByEntity(db, 'story', storyId, 'jira');
     if (existingSync && existingSync.sync_status === 'synced') {
       logger.debug(`Story ${storyId} already has sync record, skipping Jira creation`);
       continue;
@@ -330,7 +330,7 @@ export async function syncRequirementToJira(
       const jiraStory = await createIssue(client, { fields } as any);
 
       // Update local story with external integration info
-      updateStory(db, storyId, {
+      await updateStory(db, storyId, {
         externalIssueKey: jiraStory.key,
         externalIssueId: jiraStory.id,
         externalProjectKey: config.project_key,
@@ -338,7 +338,7 @@ export async function syncRequirementToJira(
       });
 
       // Record sync state
-      createSyncRecord(db, {
+      await createSyncRecord(db, {
         entityType: 'story',
         entityId: storyId,
         provider: 'jira',
@@ -363,7 +363,7 @@ export async function syncRequirementToJira(
     const jiraKey = storyKeyMap[storyId];
     if (!jiraKey) continue;
 
-    const dependencies = getStoryDependencies(db, storyId);
+    const dependencies = await getStoryDependencies(db, storyId);
     for (const dep of dependencies) {
       const depJiraKey = storyKeyMap[dep.id];
       if (!depJiraKey) continue;
@@ -385,7 +385,7 @@ export async function syncRequirementToJira(
   const movedToSprint = await tryMoveToActiveSprint(client, config, createdStoryKeys);
   if (movedToSprint) {
     for (const { storyId } of result.stories) {
-      updateStory(db, storyId, { inSprint: true });
+      await updateStory(db, storyId, { inSprint: true });
     }
   }
 
@@ -398,7 +398,7 @@ export async function syncRequirementToJira(
  * Moves the story to the active sprint after creation.
  */
 export async function syncStoryToJira(
-  db: Database,
+  db: DatabaseProvider,
   tokenStore: TokenStore,
   config: JiraConfig,
   story: StoryRow,
@@ -416,7 +416,7 @@ export async function syncStoryToJira(
   let epicKey: string | undefined;
   if (story.requirement_id) {
     const { getRequirementById } = await import('../../db/queries/requirements.js');
-    const req = getRequirementById(db, story.requirement_id);
+    const req = await getRequirementById(db, story.requirement_id);
     if (req?.external_epic_key) {
       epicKey = req.external_epic_key;
     }
@@ -449,7 +449,7 @@ export async function syncStoryToJira(
   const jiraStory = await createIssue(client, { fields } as any);
 
   // Update local story with external integration info
-  updateStory(db, story.id, {
+  await updateStory(db, story.id, {
     externalIssueKey: jiraStory.key,
     externalIssueId: jiraStory.id,
     externalProjectKey: config.project_key,
@@ -457,7 +457,7 @@ export async function syncStoryToJira(
   });
 
   // Record sync
-  createSyncRecord(db, {
+  await createSyncRecord(db, {
     entityType: 'story',
     entityId: story.id,
     provider: 'jira',
@@ -467,7 +467,7 @@ export async function syncStoryToJira(
   // Move to active sprint
   const movedToSprint = await tryMoveToActiveSprint(client, config, [jiraStory.key]);
   if (movedToSprint) {
-    updateStory(db, story.id, { inSprint: true });
+    await updateStory(db, story.id, { inSprint: true });
   }
 
   return { jiraKey: jiraStory.key, jiraId: jiraStory.id };