circuschief 0.6.0 → 0.8.0

package/packages/server/src/api/sessions.js

@@ -8,7 +8,6 @@ import { requireSession, requireSessionAndProject } from '../middleware/sessionL
 import { commandRunner } from '../services/commandRunner.js';
 
 // Import sub-routers
-import notesRouter from './sessions-notes.js';
 import conversationsRouter from './sessions-conversations.js';
 import commandsRouter from './sessions-commands.js';
 import patchRouter from './sessions-patch.js';
@@ -21,7 +20,6 @@ import draftRouter from './sessions-draft.js';
 const router = Router();
 
 // Mount sub-routers
-router.use('/', notesRouter);
 router.use('/', conversationsRouter);
 router.use('/', commandsRouter);
 router.use('/', patchRouter);
@@ -108,11 +106,14 @@ router.get('/:id', requireSession, (req, res) => {
   const allMessages = messages.getBySessionId(req.params.id);
   const hasResponses = allMessages.some(msg => msg.role === 'assistant');
 
-  // Get command run statuses (latest run per button for this session)
+  // Session details are for the requested session; command runs are workflow-root scoped.
+  const commandRunSessionId = sessions.getRootSessionId(req.params.id) || req.params.id;
+
+  // Get command run statuses (latest run per button for this workflow)
   // Completed runs from DB
-  const dbRuns = commandRuns.getLatestRunsForSession(req.params.id);
+  const dbRuns = commandRuns.getLatestRunsForSession(commandRunSessionId);
   // Currently running commands from memory - filter all runs for this session
-  const allRunning = commandRunner.getRunsBySession(req.params.id);
+  const allRunning = commandRunner.getRunsBySession(commandRunSessionId);
   const runningRuns = allRunning.filter(run => run.status === 'running');
 
   // Build map of buttonId -> run data

package/packages/server/src/api/settings.js

@@ -1,8 +1,9 @@
 import { Router } from 'express';
-import { settings } from '../db/index.js';
+import { modelProviders, settings } from '../db/index.js';
 import { DEFAULT_SESSION_TITLE_PROMPT } from '../services/summaryService.js';
 
 const router = Router();
+const SUPPORTED_SUMMARY_PROVIDER_KINDS = new Set(['anthropic', 'openai']);
 
 /**
  * GET /api/settings/token-weights
@@ -93,19 +94,42 @@ router.get('/summary', (req, res) => {
  */
 router.put('/summary', (req, res) => {
   try {
-    const { disableSessionSummaries, sessionTitlePrompt } = req.body;
+    const body = req.body || {};
+    const {
+      disableSessionSummaries,
+      sessionTitlePrompt,
+      summaryModel,
+      summaryProviderId,
+    } = body;
 
     // Validate that all required fields are present
     if (typeof disableSessionSummaries !== 'boolean' ||
-        typeof sessionTitlePrompt !== 'string') {
+        typeof sessionTitlePrompt !== 'string' ||
+        !Object.prototype.hasOwnProperty.call(body, 'summaryModel') ||
+        !Object.prototype.hasOwnProperty.call(body, 'summaryProviderId')) {
       return res.status(400).json({
-        error: 'Invalid summary settings. disableSessionSummaries must be a boolean, sessionTitlePrompt must be a string'
+        error: 'Invalid summary settings. disableSessionSummaries must be a boolean, sessionTitlePrompt must be a string, summaryModel must be present, and summaryProviderId must be present'
       });
     }
 
+    if (typeof summaryModel !== 'string') {
+      return res.status(400).json({ error: 'summaryModel must be a string' });
+    }
+
+    if (summaryProviderId !== null && typeof summaryProviderId !== 'string') {
+      return res.status(400).json({ error: 'summaryProviderId must be a string or null' });
+    }
+
+    const validationError = validateSummaryModelSelection(summaryModel, summaryProviderId);
+    if (validationError) {
+      return res.status(400).json({ error: validationError });
+    }
+
     const updatedSettings = settings.setSummarySettings({
       disableSessionSummaries,
       sessionTitlePrompt,
+      summaryModel,
+      summaryProviderId,
     });
 
     // Include the default prompt for UI display/editing
@@ -119,6 +143,30 @@ router.put('/summary', (req, res) => {
   }
 });
 
+function validateSummaryModelSelection(summaryModel, summaryProviderId) {
+  if (!summaryModel) {
+    if (summaryProviderId !== null) {
+      return 'summaryProviderId must be null when summaryModel is empty';
+    }
+    return null;
+  }
+
+  if (!summaryProviderId) {
+    return 'summaryProviderId is required when summaryModel is set';
+  }
+
+  const provider = modelProviders.getById(summaryProviderId);
+  if (!provider) return `Unknown summary provider: ${summaryProviderId}`;
+  if (!SUPPORTED_SUMMARY_PROVIDER_KINDS.has(provider.kind || 'anthropic')) {
+    return `Unsupported summary provider kind: ${provider.kind}`;
+  }
+  const ownsModel = provider.models?.some((model) => model.modelId === summaryModel);
+  if (!ownsModel) {
+    return `Provider ${summaryProviderId} does not own summary model ${summaryModel}`;
+  }
+  return null;
+}
+
 /**
  * DELETE /api/settings/summary
  * Reset summary settings to defaults

package/packages/server/src/database.js

@@ -10,7 +10,6 @@ export {
   MessageRepository,
   ConversationRepository,
   CanvasItemRepository,
-  SessionNoteRepository,
   TodoRepository,
   WorkLogRepository,
   SessionSummaryRepository,
@@ -35,7 +34,6 @@ export {
   messages,
   conversations,
   canvasItems,
-  sessionNotes,
   todos,
   workLogs,
   sessionSummaries,

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

@@ -237,6 +237,7 @@ export class ConversationRepository extends BaseRepository {
    * @param {Object} usage - Usage data
    * @param {number} usage.inputTokens
    * @param {number} usage.outputTokens
+   * @param {number} usage.thinkingTokens
    * @param {number} usage.cacheReadInputTokens
    * @param {number} usage.cacheCreationInputTokens
    * @param {number} usage.webSearchRequests
@@ -255,6 +256,7 @@ export class ConversationRepository extends BaseRepository {
         `UPDATE conversations SET
           input_tokens = ?,
           output_tokens = ?,
+          thinking_tokens = ?,
           cache_read_input_tokens = ?,
           cache_creation_input_tokens = ?,
           web_search_requests = ?,
@@ -265,6 +267,7 @@ export class ConversationRepository extends BaseRepository {
       .run(
         usage.inputTokens,
         usage.outputTokens,
+        usage.thinkingTokens || 0,
         usage.cacheReadInputTokens,
         usage.cacheCreationInputTokens,
         usage.webSearchRequests,
@@ -290,9 +293,12 @@ export class ConversationRepository extends BaseRepository {
       const now = Date.now();
 
       this.db
-        .prepare(
-          `INSERT INTO conversations (id, session_id, name, summary, is_active, created_at, updated_at)
-           VALUES (?, ?, ?, ?, ?, ?, ?)`
+      .prepare(
+          `INSERT INTO conversations (id, session_id, name, summary, is_active,
+                                      input_tokens, output_tokens, thinking_tokens,
+                                      cache_read_input_tokens, cache_creation_input_tokens,
+                                      web_search_requests, context_window, created_at, updated_at)
+           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
         )
         .run(
           id,
@@ -300,6 +306,13 @@ export class ConversationRepository extends BaseRepository {
           conv.name,
           conv.summary,
           conv.isActive ? 1 : 0,
+          conv.inputTokens,
+          conv.outputTokens,
+          conv.thinkingTokens,
+          conv.cacheReadInputTokens,
+          conv.cacheCreationInputTokens,
+          conv.webSearchRequests,
+          conv.contextWindow,
           now,
           now
         );

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

@@ -4,6 +4,7 @@ import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
 import crypto from 'crypto';
 import { allMigrations } from './migrations/index.js';
+import { seedBaselineData } from './seedBaselineData.js';
 
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = dirname(__filename);
@@ -32,7 +33,10 @@ export class DatabaseManager {
     const schema = readFileSync(join(__dirname, '..', 'schema.sql'), 'utf-8');
     this.#db.exec(schema);
 
-    // Run migrations for existing databases
+    // Seed required baseline data before future migrations run.
+    seedBaselineData(this.#db);
+
+    // Run post-baseline migrations for existing databases.
     this.#runMigrations();
 
     return this.#db;

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

@@ -12,6 +12,7 @@ const UPSERT_FIELD_MAP = [
   { key: 'gitMode', column: 'git_mode' },
   { key: 'gitBranch', column: 'git_branch' },
   { key: 'model', column: 'model' },
+  { key: 'providerId', column: 'provider_id' },
   { key: 'effortLevel', column: 'effort_level' },
 ];
 
@@ -47,6 +48,7 @@ export class ProjectDefaultsRepository extends BaseRepository {
       gitMode: row.git_mode || null,
       gitBranch: row.git_branch || null,
       model: row.model || null,
+      providerId: row.provider_id || null,
       effortLevel: row.effort_level || null,
       createdAt: row.created_at,
       updatedAt: row.updated_at,
@@ -72,47 +74,53 @@ export class ProjectDefaultsRepository extends BaseRepository {
    * @param {Object} data - Defaults data (all fields optional)
    * @returns {Object} Updated defaults object
    */
+  #insertDefaults(projectId, data) {
+    const id = databaseManager.generateId();
+    const now = Date.now();
+
+    this.db
+      .prepare(
+        `INSERT INTO project_session_defaults
+         (id, project_id, mode, thinking_enabled, start_immediately, git_mode, git_branch, model, provider_id, effort_level, created_at, updated_at)
+         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
+      )
+      .run(
+        id,
+        projectId,
+        data.mode || null,
+        data.thinkingEnabled !== undefined ? (data.thinkingEnabled ? 1 : 0) : null,
+        data.startImmediately !== undefined ? (data.startImmediately ? 1 : 0) : null,
+        data.gitMode || null,
+        data.gitBranch || null,
+        data.model || null,
+        data.providerId || null,
+        data.effortLevel || null,
+        now,
+        now
+      );
+  }
+
+  #updateDefaults(projectId, data) {
+    const { updates, values } = buildUpdateFields(data);
+
+    if (updates.length > 0) {
+      updates.push('updated_at = ?');
+      values.push(Date.now());
+      values.push(projectId);
+
+      this.db
+        .prepare(`UPDATE project_session_defaults SET ${updates.join(', ')} WHERE project_id = ?`)
+        .run(...values);
+    }
+  }
+
   upsert(projectId, data) {
-    // Get existing defaults if any
     const existing = this.getByProjectId(projectId);
 
     if (!existing) {
-      // Create new defaults record
-      const id = databaseManager.generateId();
-      const now = Date.now();
-
-      this.db
-        .prepare(
-          `INSERT INTO project_session_defaults
-           (id, project_id, mode, thinking_enabled, start_immediately, git_mode, git_branch, model, effort_level, created_at, updated_at)
-           VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
-        )
-        .run(
-          id,
-          projectId,
-          data.mode || null,
-          data.thinkingEnabled !== undefined ? (data.thinkingEnabled ? 1 : 0) : null,
-          data.startImmediately !== undefined ? (data.startImmediately ? 1 : 0) : null,
-          data.gitMode || null,
-          data.gitBranch || null,
-          data.model || null,
-          data.effortLevel || null,
-          now,
-          now
-        );
+      this.#insertDefaults(projectId, data);
     } else {
-      // Update existing defaults
-      const { updates, values } = buildUpdateFields(data);
-
-      if (updates.length > 0) {
-        updates.push('updated_at = ?');
-        values.push(Date.now());
-        values.push(projectId);
-
-        this.db
-          .prepare(`UPDATE project_session_defaults SET ${updates.join(', ')} WHERE project_id = ?`)
-          .run(...values);
-      }
+      this.#updateDefaults(projectId, data);
     }
 
     return this.getByProjectId(projectId);
@@ -137,7 +145,8 @@ export class ProjectDefaultsRepository extends BaseRepository {
       .prepare(
         `UPDATE project_session_defaults
          SET mode = NULL, thinking_enabled = NULL, start_immediately = NULL,
-             git_mode = NULL, git_branch = NULL, model = NULL, effort_level = NULL, updated_at = ?
+             git_mode = NULL, git_branch = NULL, model = NULL, provider_id = NULL,
+             effort_level = NULL, updated_at = ?
          WHERE project_id = ?`
       )
       .run(Date.now(), projectId);
@@ -161,12 +170,13 @@ export class ProjectDefaultsRepository extends BaseRepository {
    */
   static getSystemDefaults() {
     return {
-      mode: 'standard',
-      thinkingEnabled: false,
+      mode: 'yolo',
+      thinkingEnabled: true,
       startImmediately: true,
-      gitMode: null,
+      gitMode: 'worktree',
       gitBranch: null,
       model: null,
+      providerId: null,
       effortLevel: null,
     };
   }

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

@@ -1,6 +1,7 @@
 import { BaseRepository } from './BaseRepository.js';
 import { databaseManager } from './DatabaseManager.js';
 import { encrypt, decrypt } from '../services/encryption.js';
+import { normalizeCommitAttributionOverride } from '../../../shared/src/contracts/providers.js';
 
 /**
  * Valid values for `providers.kind`. Maps 1:1 to an agent adapter:
@@ -18,6 +19,55 @@ export const AGENT_TYPE_BY_KIND = Object.freeze({
   openai: 'codex',
 });
 
+const BUILT_IN_MUTABLE_FIELDS = Object.freeze(['commitAttributionOverride']);
+
+const UPDATE_COLUMN_BUILDERS = Object.freeze({
+  name: (value) => ['name = ?', value],
+  baseUrl: (value) => ['base_url = ?', value],
+  authToken: (value) => ['auth_token = ?', encrypt(value)],
+  apiTimeoutMs: (value) => ['api_timeout_ms = ?', value],
+  additionalEnvVars: (value) => [
+    'additional_env_vars = ?',
+    value ? JSON.stringify(value) : null,
+  ],
+  commitAttributionOverride: (value) => [
+    'commit_attribution_override = ?',
+    normalizeCommitAttributionOverride(value),
+  ],
+});
+
+function validateBuiltInUpdate(provider, data) {
+  if (!provider.isBuiltIn) return;
+
+  const unsupportedFields = Object.keys(data || {}).filter(
+    (key) => !BUILT_IN_MUTABLE_FIELDS.includes(key)
+  );
+  if (unsupportedFields.length > 0) {
+    throw new Error(
+      `Built-in providers can only update commitAttributionOverride. Rejected fields: ${unsupportedFields.join(', ')}.`
+    );
+  }
+}
+
+function validateKindImmutable(data) {
+  if (!data || !Object.prototype.hasOwnProperty.call(data, 'kind')) return;
+
+  throw new Error(
+    "Provider kind is immutable after create. Delete and recreate the provider to change kind."
+  );
+}
+
+function buildUpdateColumns(data = {}) {
+  return Object.entries(UPDATE_COLUMN_BUILDERS).reduce((result, [field, buildColumn]) => {
+    if (data[field] === undefined) return result;
+
+    const [update, value] = buildColumn(data[field]);
+    result.updates.push(update);
+    result.values.push(value);
+    return result;
+  }, { updates: [], values: [] });
+}
+
 /**
  * Provider repository class (replaces ModelProviderRepository).
  *
@@ -44,6 +94,7 @@ export class ProviderRepository extends BaseRepository {
       authToken: decrypt(row.auth_token),
       apiTimeoutMs: row.api_timeout_ms,
       additionalEnvVars: row.additional_env_vars ? JSON.parse(row.additional_env_vars) : null,
+      commitAttributionOverride: row.commit_attribution_override ?? null,
       isBuiltIn: row.is_built_in === 1,
       kind: row.kind || 'anthropic',
       createdAt: row.created_at,
@@ -83,6 +134,7 @@ export class ProviderRepository extends BaseRepository {
       authToken = null,
       apiTimeoutMs = null,
       additionalEnvVars = null,
+      commitAttributionOverride = null,
       kind = 'anthropic',
     } = data;
 
@@ -95,8 +147,8 @@ export class ProviderRepository extends BaseRepository {
 
     this.db
       .prepare(
-        `INSERT INTO providers (id, name, base_url, auth_token, api_timeout_ms, additional_env_vars, kind, created_at, updated_at)
-         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)`
+        `INSERT INTO providers (id, name, base_url, auth_token, api_timeout_ms, additional_env_vars, commit_attribution_override, kind, created_at, updated_at)
+         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
       )
       .run(
         id,
@@ -105,6 +157,7 @@ export class ProviderRepository extends BaseRepository {
         encrypt(authToken),
         apiTimeoutMs,
         additionalEnvVars ? JSON.stringify(additionalEnvVars) : null,
+        normalizeCommitAttributionOverride(commitAttributionOverride),
         kind,
         now,
         now
@@ -143,38 +196,13 @@ export class ProviderRepository extends BaseRepository {
    * @returns {Object} Updated provider (with models array)
    */
   update(id, data) {
-    // `kind` is immutable after create. Existing models + env wiring depend on it,
-    // so changing it in place would silently corrupt sessions already attached to
-    // this provider.
-    if (data && Object.prototype.hasOwnProperty.call(data, 'kind')) {
-      throw new Error(
-        "Provider kind is immutable after create. Delete and recreate the provider to change kind."
-      );
-    }
+    const provider = this.getById(id);
+    if (!provider) return null;
 
-    const updates = [];
-    const values = [];
+    validateBuiltInUpdate(provider, data);
+    validateKindImmutable(data);
 
-    if (data.name !== undefined) {
-      updates.push('name = ?');
-      values.push(data.name);
-    }
-    if (data.baseUrl !== undefined) {
-      updates.push('base_url = ?');
-      values.push(data.baseUrl);
-    }
-    if (data.authToken !== undefined) {
-      updates.push('auth_token = ?');
-      values.push(encrypt(data.authToken));
-    }
-    if (data.apiTimeoutMs !== undefined) {
-      updates.push('api_timeout_ms = ?');
-      values.push(data.apiTimeoutMs);
-    }
-    if (data.additionalEnvVars !== undefined) {
-      updates.push('additional_env_vars = ?');
-      values.push(data.additionalEnvVars ? JSON.stringify(data.additionalEnvVars) : null);
-    }
+    const { updates, values } = buildUpdateColumns(data);
 
     if (updates.length > 0) {
       updates.push('updated_at = ?');
@@ -348,6 +376,33 @@ export class ProviderRepository extends BaseRepository {
     return provider;
   }
 
+  /**
+   * Look up provider metadata for a model without applying runtime env fallback
+   * rules. Unlike getProviderByModelId, this returns built-in Anthropic too.
+   *
+   * @param {string|null|undefined} modelId
+   * @returns {Object|null}
+   */
+  getProviderMetadataByModelId(modelId) {
+    if (!modelId) return null;
+
+    const tierNames = ['sonnet', 'opus', 'haiku'];
+    if (tierNames.includes(modelId.toLowerCase())) {
+      return this.getById('anthropic-default');
+    }
+
+    const row = this.db
+      .prepare(
+        `SELECT p.id FROM providers p
+         JOIN provider_models pm ON p.id = pm.provider_id
+         WHERE pm.model_id = ?
+         ORDER BY p.is_built_in ASC, p.name ASC`
+      )
+      .get(modelId);
+
+    return row ? this.getById(row.id) : null;
+  }
+
   /**
    * Resolve a provider's agent type from its id.
    * @param {string|null|undefined} providerId

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

@@ -45,6 +45,7 @@ export class SessionRepository extends BaseRepository {
       status: row.status,
       mode: row.mode,
       model: row.model,
+      providerId: row.provider_id || null,
       thinkingEnabled: Boolean(row.thinking_enabled),
       archived: Boolean(row.archived),
       starred: Boolean(row.starred),
@@ -52,6 +53,7 @@ export class SessionRepository extends BaseRepository {
       gitBranch: row.git_branch,
       gitWorktree: row.git_worktree,
       prUrl: row.pr_url,
+      prUrlAutoLinkDisabled: Boolean(row.pr_url_auto_link_disabled),
       error: row.error,
       costUsd: row.cost_usd,
       claudeSessionId: row.claude_session_id,
@@ -84,7 +86,7 @@ export class SessionRepository extends BaseRepository {
     return this.map(row);
   }
 
-  /** Create a new session with optional config (mode, thinkingEnabled, gitBranch, parentSessionId, status, model, effortLevel, agentType) */
+  /** Create a new session with optional config (mode, thinkingEnabled, gitBranch, parentSessionId, status, model, providerId, effortLevel, agentType) */
   create(projectId, name, prompt, options = {}) {
     const config = parseCreateConfig(options, Array.prototype.slice.call(arguments, 4));
 
@@ -98,8 +100,8 @@ export class SessionRepository extends BaseRepository {
     const now = Date.now();
     this.db
       .prepare(
-        `INSERT INTO sessions (id, project_id, name, status, mode, thinking_enabled, git_branch, parent_session_id, model, effort_level, agent_type, created_at, updated_at)
-         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
+        `INSERT INTO sessions (id, project_id, name, status, mode, thinking_enabled, git_branch, parent_session_id, model, provider_id, effort_level, agent_type, created_at, updated_at)
+         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
       )
       .run(
         id,
@@ -111,6 +113,7 @@ export class SessionRepository extends BaseRepository {
         config.gitBranch,
         config.parentSessionId,
         config.model,
+        config.providerId,
         config.effortLevel,
         agentType,
         now,
@@ -282,10 +285,10 @@ export class SessionRepository extends BaseRepository {
     // Insert new session with same settings but new ID and status
     this.db
       .prepare(
-        `INSERT INTO sessions (id, project_id, name, status, mode, thinking_enabled, git_branch, model, effort_level, agent_type, context_window,
-                               input_tokens, output_tokens, cache_read_input_tokens, cache_creation_input_tokens,
+        `INSERT INTO sessions (id, project_id, name, status, mode, thinking_enabled, git_branch, model, provider_id, effort_level, agent_type, context_window,
+                               input_tokens, output_tokens, thinking_tokens, cache_read_input_tokens, cache_creation_input_tokens,
                                web_search_requests, cost_usd, created_at, updated_at)
-         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
+         VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`
       )
       .run(
         id,
@@ -296,11 +299,13 @@ export class SessionRepository extends BaseRepository {
         source.thinkingEnabled ? 1 : 0,
         source.gitBranch,  // Copy branch name (NOT worktree path)
         source.model,
+        source.providerId,
         source.effortLevel,
         source.agentType || DEFAULT_AGENT_TYPE,
         source.contextWindow,
         source.inputTokens,
         source.outputTokens,
+        source.thinkingTokens,
         source.cacheReadInputTokens,
         source.cacheCreationInputTokens,
         source.webSearchRequests,
@@ -326,11 +331,11 @@ export class SessionRepository extends BaseRepository {
   updateUsage(id, usage) {
     this.db
       .prepare(
-        `UPDATE sessions SET input_tokens = ?, output_tokens = ?, cache_read_input_tokens = ?,
+        `UPDATE sessions SET input_tokens = ?, output_tokens = ?, thinking_tokens = ?, cache_read_input_tokens = ?,
           cache_creation_input_tokens = ?, web_search_requests = ?, context_window = ?, updated_at = ?
         WHERE id = ?`
       )
-      .run(usage.inputTokens, usage.outputTokens, usage.cacheReadInputTokens,
+      .run(usage.inputTokens, usage.outputTokens, usage.thinkingTokens || 0, usage.cacheReadInputTokens,
         usage.cacheCreationInputTokens, usage.webSearchRequests, usage.contextWindow, Date.now(), id);
     return this.getById(id);
   }