circuschief 0.7.0 → 1.0.0

package/packages/server/src/db/migrations/sessionsMigrations.js

@@ -3,6 +3,7 @@
  * Each export is an array of { name, up(db) } migration objects.
  */
 import { addColumnIfMissing, getColumns, getTableSql } from './migrationUtils.js';
+import { migrateSessionsDefaultModeAndThinking } from './sessionTableRecreate.js';
 
 // Table name constants for migrations
 const TABLE_SESSIONS = 'sessions';
@@ -18,11 +19,12 @@ const SESSIONS_BASE_COLUMNS = `
     project_id TEXT NOT NULL REFERENCES projects(id) ON DELETE CASCADE,
     name TEXT NOT NULL,
     status TEXT NOT NULL DEFAULT 'starting' CHECK (status IN ('starting', 'running', 'waiting', 'stopped', 'completed', 'error', 'scheduled')),
-    mode TEXT NOT NULL DEFAULT 'standard' CHECK (mode IN ('plan', 'standard', 'yolo')),
-    thinking_enabled INTEGER NOT NULL DEFAULT 0,
+    mode TEXT NOT NULL DEFAULT 'yolo' CHECK (mode IN ('plan', 'standard', 'yolo')),
+    thinking_enabled INTEGER NOT NULL DEFAULT 1,
     git_branch TEXT,
     git_worktree TEXT,
     pr_url TEXT,
+    pr_url_auto_link_disabled INTEGER NOT NULL DEFAULT 0,
     error TEXT,
     effort_level TEXT CHECK(effort_level IN ('low', 'medium', 'high', 'max', 'auto')),
     cost_usd REAL DEFAULT 0,
@@ -41,7 +43,7 @@ const SESSIONS_BASE_COLUMNS = `
     starred INTEGER NOT NULL DEFAULT 0,
     manually_named INTEGER NOT NULL DEFAULT 0,
     scheduled_at INTEGER DEFAULT NULL,
-    reschedule_delay_minutes INTEGER DEFAULT 15,
+    reschedule_delay_minutes INTEGER DEFAULT 60, -- keep in sync with DEFAULT_RESCHEDULE_DELAY_MINUTES in shared/constants.js
     auto_reschedule_enabled INTEGER DEFAULT 0,
     reschedule_on_token_limit INTEGER DEFAULT 1,
     reschedule_on_service_error INTEGER DEFAULT 1,
@@ -58,7 +60,7 @@ const SESSIONS_BASE_COLUMNS = `
  */
 const SESSIONS_ALL_COLUMNS = [
   'id', 'project_id', 'name', 'status', 'mode', 'thinking_enabled',
-  'git_branch', 'git_worktree', 'pr_url', 'error', 'effort_level',
+  'git_branch', 'git_worktree', 'pr_url', 'pr_url_auto_link_disabled', 'error', 'effort_level',
   'cost_usd', 'claude_session_id', 'model', 'next_template_id',
   'parent_session_id', 'input_tokens', 'output_tokens', 'thinking_tokens',
   'cache_read_input_tokens', 'cache_creation_input_tokens',
@@ -139,7 +141,7 @@ export const sessionsMigrations = [
   },
   {
     name: 'sessions-add-reschedule_delay_minutes',
-    up(db) { addColumnIfMissing(db, TABLE_SESSIONS, 'reschedule_delay_minutes', 'INTEGER DEFAULT 15'); },
+    up(db) { addColumnIfMissing(db, TABLE_SESSIONS, 'reschedule_delay_minutes', 'INTEGER DEFAULT 60'); /* keep in sync with DEFAULT_RESCHEDULE_DELAY_MINUTES */ },
   },
   {
     name: 'sessions-add-auto_reschedule_enabled',
@@ -252,6 +254,10 @@ export const sessionsMigrations = [
     name: 'sessions-add-manually_named',
     up(db) { addColumnIfMissing(db, TABLE_SESSIONS, 'manually_named', 'INTEGER NOT NULL DEFAULT 0'); },
   },
+  {
+    name: 'sessions-add-pr_url_auto_link_disabled',
+    up(db) { addColumnIfMissing(db, TABLE_SESSIONS, 'pr_url_auto_link_disabled', 'INTEGER NOT NULL DEFAULT 0'); },
+  },
 
   // --- Pending prompt / slash commands / pending model / auto send ---
   {
@@ -284,4 +290,11 @@ export const sessionsMigrations = [
     name: 'sessions-add-agent_type',
     up(db) { addColumnIfMissing(db, TABLE_SESSIONS, 'agent_type', "TEXT DEFAULT 'claude-code'"); },
   },
+
+  // --- Default mode / thinking changes ---
+  {
+    name: 'sessions-migrate-default-mode-thinking',
+    up(db) { migrateSessionsDefaultModeAndThinking(db); },
+  },
+
 ];

package/packages/server/src/db/seedBaselineData.js

@@ -0,0 +1,137 @@
+import { randomUUID } from 'node:crypto';
+import { OPENAI_MODELS } from '../../../shared/src/index.js';
+
+export const BUILT_IN_ANTHROPIC_PROVIDER = {
+  id: 'anthropic-default',
+  name: 'Anthropic (Official)',
+  kind: 'anthropic',
+};
+
+export const BUILT_IN_OPENAI_PROVIDER = {
+  id: 'openai-default',
+  name: 'OpenAI (Official)',
+  kind: 'openai',
+};
+
+export const BUILT_IN_ANTHROPIC_MODELS = [
+  { id: 'anthropic-haiku', providerId: BUILT_IN_ANTHROPIC_PROVIDER.id, modelId: 'claude-haiku-4-5-20251001', displayName: 'Haiku 4.5', description: 'Fast & lightweight', tier: 'haiku' },
+  { id: 'anthropic-sonnet', providerId: BUILT_IN_ANTHROPIC_PROVIDER.id, modelId: 'claude-sonnet-4-6', displayName: 'Sonnet 4.6', description: 'Balanced', tier: 'sonnet' },
+  { id: 'anthropic-opus', providerId: BUILT_IN_ANTHROPIC_PROVIDER.id, modelId: 'claude-opus-4-6', displayName: 'Opus 4.6', description: 'Previous generation', tier: 'opus' },
+  { id: 'anthropic-opus-4-7', providerId: BUILT_IN_ANTHROPIC_PROVIDER.id, modelId: 'claude-opus-4-7', displayName: 'Opus 4.7', description: 'Most capable (default)', tier: 'opus' },
+];
+
+export const BUILT_IN_OPENAI_MODELS = OPENAI_MODELS.map((model) => ({
+  id: model.seedId,
+  providerId: BUILT_IN_OPENAI_PROVIDER.id,
+  modelId: model.id,
+  displayName: model.name,
+  description: model.description,
+  tier: 'custom',
+}));
+
+export const DEFAULT_QUICK_RESPONSES = [
+  { label: 'Put a plan on the canvas', content: 'Put a plan on the canvas to get this done', autoSubmit: false, sortOrder: 0 },
+  { label: 'Yes', content: 'Yes', autoSubmit: true, sortOrder: 1 },
+  { label: 'Review the plan', content: `Review the plan on the canvas. Are there any issues that you can find? Is test coverage specified explicitly enough? Does the current code match the assumptions in the plan?\n\nList the issues that you find and then update the plan on the canvas to address any issues that you find. Don't talk about issues with the original plan in the plan itself. Just tell me what the issues are, and update the plan so that the plan doesn't have the issues.`, autoSubmit: true, sortOrder: 2 },
+  { label: 'Implement the plan on the canvas', content: 'Implement the plan on the canvas', autoSubmit: true, sortOrder: 3 },
+  { label: 'Create / Update PR', content: `Ensure all relevant changes are committed and pushed. Then look at the session's summary and create a draft PR if no PR already exists.`, autoSubmit: true, sortOrder: 4 },
+  { label: 'Review PR', content: 'Look at the PR related to the root session. Review the PR. Are there any issues? Are best practices adhered to? Does the PR accomplish the goal? Are all changes covered by tests?', autoSubmit: true, sortOrder: 5 },
+  { label: 'Add tests', content: 'Inspect the changes on our branch. For each change, ensure we have tests that assert the change is working in the expected way. Implement the tests.', autoSubmit: true, sortOrder: 6 },
+  { label: 'Merge in main', content: 'Merge in the latest main branch', autoSubmit: true, sortOrder: 7 },
+  { label: 'Add tests to the plan', content: 'Call out specific test cases in the plan, we should have an assertion for each change called for in the plan', autoSubmit: true, sortOrder: 8 },
+  { label: 'Tests are failing', content: 'Tests are failing. Look at the canvas for details', autoSubmit: true, sortOrder: 9 },
+  { label: 'Continue', content: 'Continue', autoSubmit: true, sortOrder: 10 },
+];
+
+export const DEFAULT_SESSION_TEMPLATE_PROMPTS = {
+  REVIEW: `Review the plan on the canvas. If there's more than one plan then review the most recently updated plan. if there are no plans on the canvas then look for the most recently updated plan on the root session canvas.
+
+Make sure there are tests explicitly called out for all changes. Make sure that all context necessary to hand off the task is included in the plan.
+
+Are there any gaps in the plan? Is test coverage spelled out explicitly? Does the code match the assumptions in the plan?
+
+Update the plan according to your review recommendations.`,
+  IMPLEMENT: `Implement the plan on the canvas. If there's more than one plan on the canvas then use the most recently updated plan. If you don't see a plan on the canvas then look at the parent session's canvas.`,
+  PR: `Ensure all relevant changes are committed and pushed. Then determine the session's goals. You can typically find details about the goals of the session by looking at the most recently modified markdown documents on the root session's canvas - these are typically plans that were implemented during the session. You can also look at the root session's summary, but don't trigger a new summary to be created if the summary is missing.
+
+Create a draft pr and ensure all changes are committed and pushed.`,
+};
+
+export const DEFAULT_SESSION_TEMPLATES = [
+  { name: 'Review the plan', prompt: DEFAULT_SESSION_TEMPLATE_PROMPTS.REVIEW },
+  { name: 'Implement the plan on the canvas', prompt: DEFAULT_SESSION_TEMPLATE_PROMPTS.IMPLEMENT },
+  { name: 'Create/update PR', prompt: DEFAULT_SESSION_TEMPLATE_PROMPTS.PR },
+];
+
+function seedBuiltInProviders(db) {
+  const now = Date.now();
+
+  db.prepare(
+    `INSERT OR IGNORE INTO providers (
+       id, name, base_url, auth_token, kind, is_built_in, created_at, updated_at
+     )
+     VALUES (?, ?, NULL, NULL, ?, 1, ?, ?)`
+  ).run(BUILT_IN_ANTHROPIC_PROVIDER.id, BUILT_IN_ANTHROPIC_PROVIDER.name, BUILT_IN_ANTHROPIC_PROVIDER.kind, now, now);
+
+  db.prepare(
+    `INSERT OR IGNORE INTO providers (
+       id, name, base_url, auth_token, kind, is_built_in, created_at, updated_at
+     )
+     VALUES (?, ?, NULL, NULL, ?, 1, ?, ?)`
+  ).run(BUILT_IN_OPENAI_PROVIDER.id, BUILT_IN_OPENAI_PROVIDER.name, BUILT_IN_OPENAI_PROVIDER.kind, now, now);
+
+  const insertModel = db.prepare(
+    `INSERT OR IGNORE INTO provider_models (
+       id, provider_id, model_id, display_name, description, tier, created_at
+     )
+     VALUES (?, ?, ?, ?, ?, ?, ?)`
+  );
+
+  for (const model of [...BUILT_IN_ANTHROPIC_MODELS, ...BUILT_IN_OPENAI_MODELS]) {
+    insertModel.run(model.id, model.providerId, model.modelId, model.displayName, model.description, model.tier, now);
+  }
+}
+
+function seedDefaultQuickResponses(db) {
+  const count = db.prepare('SELECT COUNT(*) AS cnt FROM quick_responses').get().cnt;
+  if (count > 0) return;
+
+  const now = Date.now();
+  const stmt = db.prepare(
+    `INSERT INTO quick_responses (
+       id, project_id, label, content, auto_submit, category, sort_order, created_at, updated_at
+     )
+     VALUES (?, NULL, ?, ?, ?, NULL, ?, ?, ?)`
+  );
+
+  for (const item of DEFAULT_QUICK_RESPONSES) {
+    stmt.run(randomUUID(), item.label, item.content, item.autoSubmit ? 1 : 0, item.sortOrder, now, now);
+  }
+}
+
+function seedDefaultSessionTemplates(db) {
+  const count = db.prepare(
+    'SELECT COUNT(*) AS cnt FROM session_templates WHERE project_id IS NULL'
+  ).get().cnt;
+  if (count > 0) return;
+
+  const now = Date.now();
+  const stmt = db.prepare(`
+    INSERT INTO session_templates (
+      id, project_id, name, prompt,
+      next_template_id, thinking_enabled,
+      git_branch, git_mode, model, mode, effort_level, target_lane_id,
+      created_at, updated_at
+    ) VALUES (?, NULL, ?, ?, NULL, 1, NULL, NULL, NULL, 'yolo', NULL, NULL, ?, ?)
+  `);
+
+  for (const item of DEFAULT_SESSION_TEMPLATES) {
+    stmt.run(randomUUID(), item.name, item.prompt, now, now);
+  }
+}
+
+export function seedBaselineData(db) {
+  seedBuiltInProviders(db);
+  seedDefaultQuickResponses(db);
+  seedDefaultSessionTemplates(db);
+}

package/packages/server/src/db/session-helpers.js

@@ -3,9 +3,36 @@
  * config parsing, and update-clause builders.
  */
 
+import { DEFAULT_RESCHEDULE_DELAY_MINUTES } from '../../../shared/src/index.js';
+
 /** Reusable SQL fragment for computed activity fields on sessions */
 export const ACTIVITY_FIELDS_SQL = `
-  (SELECT MAX(cm.timestamp) FROM conversation_messages cm WHERE cm.session_id = s.id) AS last_activity_at,
+  (
+    SELECT MAX(activity_at)
+    FROM (
+      SELECT cm.timestamp AS activity_at
+      FROM conversation_messages cm
+      WHERE cm.session_id = s.id
+      UNION ALL
+      SELECT ss.generated_at AS activity_at
+      FROM session_summaries ss
+      WHERE ss.session_id = s.id
+      UNION ALL
+      SELECT ss.updated_at AS activity_at
+      FROM session_summaries ss
+      WHERE ss.session_id = s.id
+      UNION ALL
+      SELECT COALESCE(cr.completed_at, cr.started_at) AS activity_at
+      FROM command_runs cr
+      WHERE cr.session_id = s.id
+    )
+    WHERE activity_at IS NOT NULL
+  ) AS last_activity_at,
+  (
+    SELECT MAX(cm.timestamp)
+    FROM conversation_messages cm
+    WHERE cm.session_id = s.id
+  ) AS last_message_at,
   (CAST(
     CASE
       WHEN (SELECT MAX(cm2.timestamp) FROM conversation_messages cm2 WHERE cm2.session_id = s.id) IS NOT NULL
@@ -40,7 +67,7 @@ export function mapTokenUsage(row) {
 export function mapScheduling(row) {
   return {
     scheduledAt: row.scheduled_at || null,
-    rescheduleDelayMinutes: row.reschedule_delay_minutes || 15,
+    rescheduleDelayMinutes: row.reschedule_delay_minutes || DEFAULT_RESCHEDULE_DELAY_MINUTES,
     autoRescheduleEnabled: Boolean(row.auto_reschedule_enabled),
     rescheduleOnTokenLimit: Boolean(row.reschedule_on_token_limit),
     rescheduleOnServiceError: Boolean(row.reschedule_on_service_error),
@@ -53,8 +80,8 @@ export function mapScheduling(row) {
 
 /** Default values for session-create config fields */
 const CONFIG_DEFAULTS = {
-  mode: 'standard',
-  thinkingEnabled: false,
+  mode: 'yolo',
+  thinkingEnabled: true,
   gitBranch: null,
   parentSessionId: null,
   status: 'starting',
@@ -125,6 +152,7 @@ export const BOOLEAN_FIELD_MAP = {
   archived: 'archived',
   starred: 'starred',
   manuallyNamed: 'manually_named',
+  prUrlAutoLinkDisabled: 'pr_url_auto_link_disabled',
   autoRescheduleEnabled: 'auto_reschedule_enabled',
   rescheduleOnTokenLimit: 'reschedule_on_token_limit',
   rescheduleOnServiceError: 'reschedule_on_service_error',

package/packages/server/src/middleware/sessionLookup.js

@@ -1,5 +1,9 @@
 import { sessions, projects } from '../database.js';
 
+const SESSION_NOT_FOUND = 'Session not found';
+const PROJECT_NOT_FOUND = 'Project not found';
+const SESSION_NOT_IN_PROJECT = 'Session does not belong to this project';
+
 /**
  * Middleware: Look up a session by req.params.id and attach to req.session_.
  * Returns 404 if session not found.
@@ -8,17 +12,12 @@ import { sessions, projects } from '../database.js';
 export function requireSession(req, res, next) {
   const session = sessions.getById(req.params.id);
   if (!session) {
-    return res.status(404).json({ error: 'Session not found' });
+    return res.status(404).json({ error: SESSION_NOT_FOUND });
   }
   req.session_ = session;
   next();
 }
 
-/**
- * Middleware: Look up session AND its project. Attaches req.session_ and req.project.
- * Returns 404 if either is not found.
- * Also resolves req.workingDirectory (gitWorktree || project.workingDirectory).
- */
 /**
  * Middleware factory: Validates that req.session_.status is one of the allowed statuses.
  * Must be used AFTER requireSession or requireSessionAndProject middleware.
@@ -39,15 +38,20 @@ export function requireSessionStatus(allowedStatuses, errorMessage) {
   };
 }
 
+/**
+ * Middleware: Look up session AND its project. Attaches req.session_ and req.project.
+ * Returns 404 if either is not found.
+ * Also resolves req.workingDirectory (gitWorktree || project.workingDirectory).
+ */
 export function requireSessionAndProject(req, res, next) {
   const session = sessions.getById(req.params.id);
   if (!session) {
-    return res.status(404).json({ error: 'Session not found' });
+    return res.status(404).json({ error: SESSION_NOT_FOUND });
   }
 
   const project = projects.getById(session.projectId);
   if (!project) {
-    return res.status(404).json({ error: 'Project not found' });
+    return res.status(404).json({ error: PROJECT_NOT_FOUND });
   }
 
   req.session_ = session;
@@ -55,3 +59,72 @@ export function requireSessionAndProject(req, res, next) {
   req.workingDirectory = session.gitWorktree || project.workingDirectory;
   next();
 }
+
+/**
+ * Middleware: Look up the provided session ID, then attach the workflow root
+ * session/project for resources shared across a session tree.
+ */
+export function requireRootSessionAndProject(req, res, next) {
+  const providedSession = sessions.getById(req.params.id);
+  if (!providedSession) {
+    return res.status(404).json({ error: SESSION_NOT_FOUND });
+  }
+
+  const rootSessionId = sessions.getRootSessionId(providedSession.id) || providedSession.id;
+  const rootSession = sessions.getById(rootSessionId);
+  if (!rootSession) {
+    return res.status(404).json({ error: SESSION_NOT_FOUND });
+  }
+
+  if (providedSession.projectId !== rootSession.projectId) {
+    return res.status(400).json({ error: 'Session parent chain crosses projects' });
+  }
+
+  const rootProject = projects.getById(rootSession.projectId);
+  if (!rootProject) {
+    return res.status(404).json({ error: PROJECT_NOT_FOUND });
+  }
+
+  req.providedSession_ = providedSession;
+  req.rootSession_ = rootSession;
+  req.rootSessionId = rootSessionId;
+  req.rootProject = rootProject;
+  req.rootWorkingDirectory = rootSession.gitWorktree || rootProject.workingDirectory;
+  next();
+}
+
+/**
+ * Middleware factory for project-scoped endpoints that receive a sessionId in
+ * the JSON body and should operate on that session tree's root.
+ */
+export function resolveBodyRootSessionForProject(projectParam = 'projectId') {
+  return (req, res, next) => {
+    const providedSessionId = req.body?.sessionId;
+    if (!providedSessionId) {
+      return res.status(400).json({ error: 'sessionId is required' });
+    }
+
+    const providedSession = sessions.getById(providedSessionId);
+    if (!providedSession) {
+      return res.status(404).json({ error: SESSION_NOT_FOUND });
+    }
+
+    if (providedSession.projectId !== req.params[projectParam]) {
+      return res.status(400).json({ error: SESSION_NOT_IN_PROJECT });
+    }
+
+    const rootSessionId = sessions.getRootSessionId(providedSession.id) || providedSession.id;
+    const rootSession = sessions.getById(rootSessionId);
+    if (!rootSession) {
+      return res.status(404).json({ error: SESSION_NOT_FOUND });
+    }
+
+    if (rootSession.projectId !== req.params[projectParam]) {
+      return res.status(400).json({ error: SESSION_NOT_IN_PROJECT });
+    }
+
+    req.bodyRootSession_ = rootSession;
+    req.bodyRootSessionId = rootSessionId;
+    next();
+  };
+}