Compare commits
2 Commits
008435f1c2
...
feat/sqlit
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
6fdbc302d2 | ||
| 2a6413821a |
92
packages/core/src/cache/cache-v0.test.ts
vendored
Normal file
92
packages/core/src/cache/cache-v0.test.ts
vendored
Normal file
@@ -0,0 +1,92 @@
|
||||
import { DatabaseSync } from 'node:sqlite'
|
||||
|
||||
import { describe, expect, it } from 'vitest'
|
||||
|
||||
import type { GiteaIssue } from '../gitea/types.js'
|
||||
import { extractLabelFacts } from '../labels/label-schema.js'
|
||||
import { type CacheDriver, initCache, readIssue, upsertIssue } from './cache-v0.js'
|
||||
|
||||
/** Adapt node:sqlite's DatabaseSync to the CacheDriver seam (main uses better-sqlite3). */
|
||||
function memoryDriver(): CacheDriver {
|
||||
const db = new DatabaseSync(':memory:')
|
||||
return {
|
||||
exec: (sql) => db.exec(sql),
|
||||
run: (sql, params = []) => {
|
||||
db.prepare(sql).run(...(params as never[]))
|
||||
},
|
||||
get: (sql, params = []) => db.prepare(sql).get(...(params as never[])) as Record<string, unknown> | undefined,
|
||||
all: (sql, params = []) => db.prepare(sql).all(...(params as never[])) as Record<string, unknown>[],
|
||||
}
|
||||
}
|
||||
|
||||
function issue(over: Partial<GiteaIssue> = {}): GiteaIssue {
|
||||
const labels = over.labels ?? ['est/5d', 'p/1', 'deadline/hard']
|
||||
return {
|
||||
number: 42,
|
||||
title: 'Monte Carlo engine',
|
||||
body: 'percentile bands',
|
||||
state: 'open',
|
||||
labels,
|
||||
facts: extractLabelFacts(labels),
|
||||
milestone: { id: 7, title: 'P2 — Scheduler', dueOn: '2026-09-01T00:00:00Z' },
|
||||
assignee: 'christian',
|
||||
assignees: ['christian'],
|
||||
createdAt: '2026-07-08T00:00:00Z',
|
||||
updatedAt: '2026-07-08T01:00:00Z',
|
||||
closedAt: null,
|
||||
url: 'https://gitea/christian/commitea/issues/42',
|
||||
...over,
|
||||
}
|
||||
}
|
||||
|
||||
describe('cache-v0', () => {
|
||||
it('mirrors one issue and reads its facts back through extractLabelFacts (acceptance)', () => {
|
||||
const d = memoryDriver()
|
||||
initCache(d)
|
||||
upsertIssue(d, issue({ labels: ['est/5d', 'p/1', 'deadline/hard'] }))
|
||||
|
||||
const back = readIssue(d, 42)!
|
||||
expect(back.labels).toEqual(['est/5d', 'p/1', 'deadline/hard'])
|
||||
// facts are re-derived on read, not stored
|
||||
expect(back.facts.estimateDays).toBe(5)
|
||||
expect(back.facts.priority).toBe(1)
|
||||
expect(back.facts.hardDeadline).toBe(true)
|
||||
// the rest of the domain shape round-trips
|
||||
expect(back.milestone).toEqual({ id: 7, title: 'P2 — Scheduler', dueOn: '2026-09-01T00:00:00Z' })
|
||||
expect(back.assignee).toBe('christian')
|
||||
expect(back.state).toBe('open')
|
||||
})
|
||||
|
||||
it('re-derives facts from the current labels after a re-reconcile (upsert in place, no dup)', () => {
|
||||
const d = memoryDriver()
|
||||
initCache(d)
|
||||
upsertIssue(d, issue({ labels: ['est/2d', 'p/3'] }))
|
||||
// reconcile again with changed labels + closed
|
||||
upsertIssue(d, issue({ labels: ['est/8d', 'p/1'], state: 'closed', closedAt: '2026-07-09T00:00:00Z' }))
|
||||
|
||||
expect(d.all('SELECT number FROM issues')).toHaveLength(1) // upsert by number, not a second row
|
||||
const back = readIssue(d, 42)!
|
||||
expect(back.facts.estimateDays).toBe(8)
|
||||
expect(back.facts.priority).toBe(1)
|
||||
expect(back.facts.hardDeadline).toBe(false) // deadline/hard dropped
|
||||
expect(back.state).toBe('closed')
|
||||
expect(back.closedAt).toBe('2026-07-09T00:00:00Z')
|
||||
})
|
||||
|
||||
it('reads an issue with no milestone / empty labels', () => {
|
||||
const d = memoryDriver()
|
||||
initCache(d)
|
||||
upsertIssue(d, issue({ number: 9, labels: [], milestone: null, assignee: null, assignees: [] }))
|
||||
const back = readIssue(d, 9)!
|
||||
expect(back.milestone).toBeNull()
|
||||
expect(back.labels).toEqual([])
|
||||
expect(back.facts.estimateDays).toBeNull()
|
||||
expect(back.assignee).toBeNull()
|
||||
})
|
||||
|
||||
it('returns null for an uncached issue', () => {
|
||||
const d = memoryDriver()
|
||||
initCache(d)
|
||||
expect(readIssue(d, 999)).toBeNull()
|
||||
})
|
||||
})
|
||||
159
packages/core/src/cache/cache-v0.ts
vendored
Normal file
159
packages/core/src/cache/cache-v0.ts
vendored
Normal file
@@ -0,0 +1,159 @@
|
||||
/**
|
||||
* SQLite cache, v0 (#3) — a rebuildable local mirror of the reconciled backlog.
|
||||
* It is an index over the durable truth in gitea, never the source of truth (D4):
|
||||
* delete it, resync, lose nothing. This module owns the schema + the pure
|
||||
* row<->domain mappers; the actual SQLite handle is injected as a `CacheDriver`,
|
||||
* so core stays free of any native driver (better-sqlite3 lives in main; tests
|
||||
* use node:sqlite). Facts are never stored — they are re-derived from the label
|
||||
* set on read via `extractLabelFacts`, so the mirror can't drift from the label
|
||||
* semantics.
|
||||
*/
|
||||
|
||||
import type { GiteaIssue, GiteaMilestoneRef } from '../gitea/types.js'
|
||||
import { extractLabelFacts } from '../labels/label-schema.js'
|
||||
|
||||
/**
|
||||
* The injected IO boundary: a thin synchronous SQL executor. Core writes the SQL;
|
||||
* the host binds a real driver (better-sqlite3 in the desktop main process,
|
||||
* node:sqlite in tests). Kept minimal on purpose — no ORM, no query builder.
|
||||
*/
|
||||
export interface CacheDriver {
|
||||
/** Run one or more DDL/utility statements (no params, no result). */
|
||||
exec(sql: string): void
|
||||
/** Execute a single parameterized write. */
|
||||
run(sql: string, params?: readonly unknown[]): void
|
||||
/** First row of a parameterized query, or undefined. */
|
||||
get(sql: string, params?: readonly unknown[]): Record<string, unknown> | undefined
|
||||
/** All rows of a parameterized query. */
|
||||
all(sql: string, params?: readonly unknown[]): Record<string, unknown>[]
|
||||
}
|
||||
|
||||
/** The cache schema — five tables mirroring gitea's shape. Regenerable; drop and rebuild freely. */
|
||||
export const CACHE_SCHEMA = `
|
||||
CREATE TABLE IF NOT EXISTS milestones (
|
||||
id INTEGER PRIMARY KEY,
|
||||
title TEXT NOT NULL,
|
||||
state TEXT,
|
||||
due_on TEXT
|
||||
);
|
||||
CREATE TABLE IF NOT EXISTS issues (
|
||||
number INTEGER PRIMARY KEY,
|
||||
title TEXT NOT NULL,
|
||||
body TEXT NOT NULL DEFAULT '',
|
||||
state TEXT NOT NULL,
|
||||
labels TEXT NOT NULL DEFAULT '[]', -- JSON array of label names; facts re-derived on read
|
||||
milestone_id INTEGER,
|
||||
assignee TEXT,
|
||||
assignees TEXT NOT NULL DEFAULT '[]', -- JSON array of logins
|
||||
created_at TEXT,
|
||||
updated_at TEXT,
|
||||
closed_at TEXT,
|
||||
url TEXT,
|
||||
FOREIGN KEY (milestone_id) REFERENCES milestones(id)
|
||||
);
|
||||
CREATE TABLE IF NOT EXISTS labels (
|
||||
id INTEGER PRIMARY KEY,
|
||||
name TEXT NOT NULL
|
||||
);
|
||||
CREATE TABLE IF NOT EXISTS comments (
|
||||
id INTEGER PRIMARY KEY,
|
||||
issue_number INTEGER NOT NULL,
|
||||
author TEXT,
|
||||
body TEXT NOT NULL DEFAULT '',
|
||||
created_at TEXT
|
||||
);
|
||||
CREATE TABLE IF NOT EXISTS issue_events (
|
||||
id INTEGER PRIMARY KEY AUTOINCREMENT,
|
||||
issue_number INTEGER NOT NULL,
|
||||
type TEXT NOT NULL,
|
||||
at TEXT NOT NULL
|
||||
);
|
||||
CREATE INDEX IF NOT EXISTS idx_issue_events_number ON issue_events(issue_number);
|
||||
CREATE INDEX IF NOT EXISTS idx_comments_number ON comments(issue_number);
|
||||
`
|
||||
|
||||
/** Create the schema if absent. Idempotent. */
|
||||
export function initCache(driver: CacheDriver): void {
|
||||
driver.exec(CACHE_SCHEMA)
|
||||
}
|
||||
|
||||
const UPSERT_ISSUE = `
|
||||
INSERT INTO issues (number, title, body, state, labels, milestone_id, assignee, assignees, created_at, updated_at, closed_at, url)
|
||||
VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
|
||||
ON CONFLICT(number) DO UPDATE SET
|
||||
title = excluded.title, body = excluded.body, state = excluded.state, labels = excluded.labels,
|
||||
milestone_id = excluded.milestone_id, assignee = excluded.assignee, assignees = excluded.assignees,
|
||||
created_at = excluded.created_at, updated_at = excluded.updated_at, closed_at = excluded.closed_at, url = excluded.url
|
||||
`
|
||||
|
||||
const UPSERT_MILESTONE = `
|
||||
INSERT INTO milestones (id, title, state, due_on) VALUES (?, ?, ?, ?)
|
||||
ON CONFLICT(id) DO UPDATE SET title = excluded.title, state = excluded.state, due_on = excluded.due_on
|
||||
`
|
||||
|
||||
/**
|
||||
* Mirror one reconciled issue into the cache (and its milestone, if any). Upsert
|
||||
* by `number`, so re-reconciling the same issue updates in place — never duplicates.
|
||||
*/
|
||||
export function upsertIssue(driver: CacheDriver, issue: GiteaIssue): void {
|
||||
if (issue.milestone) {
|
||||
driver.run(UPSERT_MILESTONE, [issue.milestone.id, issue.milestone.title, null, issue.milestone.dueOn])
|
||||
}
|
||||
driver.run(UPSERT_ISSUE, [
|
||||
issue.number,
|
||||
issue.title,
|
||||
issue.body,
|
||||
issue.state,
|
||||
JSON.stringify(issue.labels),
|
||||
issue.milestone?.id ?? null,
|
||||
issue.assignee,
|
||||
JSON.stringify(issue.assignees),
|
||||
issue.createdAt,
|
||||
issue.updatedAt,
|
||||
issue.closedAt,
|
||||
issue.url,
|
||||
])
|
||||
}
|
||||
|
||||
const READ_ISSUE = `
|
||||
SELECT i.*, m.title AS m_title, m.due_on AS m_due
|
||||
FROM issues i LEFT JOIN milestones m ON m.id = i.milestone_id
|
||||
WHERE i.number = ?
|
||||
`
|
||||
|
||||
function str(v: unknown): string {
|
||||
return typeof v === 'string' ? v : ''
|
||||
}
|
||||
|
||||
/**
|
||||
* Read one mirrored issue back as a domain object, re-deriving `facts` from the
|
||||
* stored label set (so the mirror can't disagree with the label semantics).
|
||||
* Returns null when the issue isn't cached.
|
||||
*/
|
||||
export function readIssue(driver: CacheDriver, number: number): GiteaIssue | null {
|
||||
const row = driver.get(READ_ISSUE, [number])
|
||||
if (!row) return null
|
||||
|
||||
const labels = (JSON.parse(str(row.labels) || '[]') as string[]) ?? []
|
||||
const assignees = (JSON.parse(str(row.assignees) || '[]') as string[]) ?? []
|
||||
const milestone: GiteaMilestoneRef | null =
|
||||
row.milestone_id != null
|
||||
? { id: Number(row.milestone_id), title: str(row.m_title), dueOn: (row.m_due as string | null) ?? null }
|
||||
: null
|
||||
|
||||
return {
|
||||
number: Number(row.number),
|
||||
title: str(row.title),
|
||||
body: str(row.body),
|
||||
state: row.state === 'closed' ? 'closed' : 'open',
|
||||
labels,
|
||||
facts: extractLabelFacts(labels),
|
||||
milestone,
|
||||
assignee: (row.assignee as string | null) ?? null,
|
||||
assignees,
|
||||
createdAt: str(row.created_at),
|
||||
updatedAt: str(row.updated_at),
|
||||
closedAt: (row.closed_at as string | null) ?? null,
|
||||
url: str(row.url),
|
||||
}
|
||||
}
|
||||
@@ -23,6 +23,8 @@ export type {
|
||||
GiteaRequestInit,
|
||||
} from './gitea/types.js'
|
||||
|
||||
export { CACHE_SCHEMA, initCache, readIssue, upsertIssue } from './cache/cache-v0.js'
|
||||
export type { CacheDriver } from './cache/cache-v0.js'
|
||||
export { describeChange, isLabelChange, planIssueChange, proposalsFor, summarizeChange } from './changes/apply-changes-v0.js'
|
||||
export type {
|
||||
ChangeProposal,
|
||||
|
||||
Reference in New Issue
Block a user