@oss-scout/core 0.2.0 → 0.2.1

package/dist/cli.js

@@ -2,41 +2,41 @@
 /**
  * oss-scout CLI — Find open source issues personalized to your contribution history.
  */
-import { Command } from 'commander';
-import { enableDebug } from './core/logger.js';
-import { getCLIVersion } from './core/utils.js';
-import { formatJsonSuccess, formatJsonError } from './formatters/json.js';
-import { errorMessage, resolveErrorCode } from './core/errors.js';
-import { hasLocalState, loadLocalState, saveLocalState } from './core/local-state.js';
-import { CONCRETE_STRATEGIES, SearchStrategySchema } from './core/schemas.js';
+import { Command } from "commander";
+import { enableDebug } from "./core/logger.js";
+import { getCLIVersion } from "./core/utils.js";
+import { formatJsonSuccess, formatJsonError } from "./formatters/json.js";
+import { errorMessage, resolveErrorCode } from "./core/errors.js";
+import { hasLocalState, loadLocalState, saveLocalState, } from "./core/local-state.js";
+import { CONCRETE_STRATEGIES, SearchStrategySchema } from "./core/schemas.js";
 function handleCommandError(err, options) {
     if (options.json) {
         console.log(formatJsonError(errorMessage(err), resolveErrorCode(err)));
     }
     else {
-        console.error('Error:', errorMessage(err));
+        console.error("Error:", errorMessage(err));
     }
     process.exit(1);
 }
 const program = new Command();
 program
-    .name('oss-scout')
-    .description('Find open source issues personalized to your contribution history')
+    .name("oss-scout")
+    .description("Find open source issues personalized to your contribution history")
     .version(getCLIVersion())
-    .option('--debug', 'Enable debug output');
+    .option("--debug", "Enable debug output");
 // Parse --debug early so it's available in preAction hooks
-program.hook('preAction', (_thisCommand, _actionCommand) => {
+program.hook("preAction", (_thisCommand, _actionCommand) => {
     const opts = program.opts();
     if (opts.debug)
         enableDebug();
 });
 program
-    .command('setup')
-    .description('Interactive first-run configuration')
-    .option('--json', 'Output as JSON')
+    .command("setup")
+    .description("Interactive first-run configuration")
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        const { runSetup } = await import('./commands/setup.js');
+        const { runSetup } = await import("./commands/setup.js");
         const prefs = await runSetup();
         const state = loadLocalState();
         state.preferences = prefs;
@@ -50,17 +50,21 @@ program
     }
 });
 program
-    .command('bootstrap')
-    .description('Import starred repos and PR history from GitHub')
-    .option('--json', 'Output as JSON')
+    .command("bootstrap")
+    .description("Import starred repos and PR history from GitHub")
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        const { bootstrapScout } = await import('./core/bootstrap.js');
-        const { createScout } = await import('./scout.js');
-        const { requireGitHubToken } = await import('./core/utils.js');
+        const { bootstrapScout } = await import("./core/bootstrap.js");
+        const { createScout } = await import("./scout.js");
+        const { requireGitHubToken } = await import("./core/utils.js");
         const token = requireGitHubToken();
         const state = loadLocalState();
-        const scout = await createScout({ githubToken: token, persistence: 'provided', initialState: state });
+        const scout = await createScout({
+            githubToken: token,
+            persistence: "provided",
+            initialState: state,
+        });
         const result = await bootstrapScout(scout, token);
         saveLocalState(scout.getState());
         if (options.json) {
@@ -68,7 +72,7 @@ program
         }
         else {
             if (result.skippedDueToRateLimit) {
-                console.log('Skipped: GitHub API rate limit too low. Try again later.');
+                console.log("Skipped: GitHub API rate limit too low. Try again later.");
             }
             else {
                 console.log(`Imported ${result.mergedPRCount} merged PRs, ${result.closedPRCount} closed PRs, ${result.starredRepoCount} starred repos`);
@@ -81,35 +85,41 @@ program
     }
 });
 program
-    .command('search [count]')
-    .description('Search for contributable issues using multi-strategy discovery')
-    .option('--json', 'Output as JSON')
-    .option('--strategy <strategies>', `Search strategies (${CONCRETE_STRATEGIES.join(',')},all)`, 'all')
+    .command("search [count]")
+    .description("Search for contributable issues using multi-strategy discovery")
+    .option("--json", "Output as JSON")
+    .option("--strategy <strategies>", `Search strategies (${CONCRETE_STRATEGIES.join(",")},all)`, "all")
     .action(async (count, options) => {
     try {
         if (!hasLocalState()) {
-            console.log('💡 Run `oss-scout setup` to configure your preferences for personalized search results.\n');
+            console.log("💡 Run `oss-scout setup` to configure your preferences for personalized search results.\n");
         }
-        const { runSearch } = await import('./commands/search.js');
+        const { runSearch } = await import("./commands/search.js");
         const maxResults = count ? parseInt(count, 10) : 10;
         if (isNaN(maxResults) || maxResults < 1) {
-            console.error('Error: count must be a positive integer');
+            console.error("Error: count must be a positive integer");
             process.exit(1);
         }
         const state = loadLocalState();
         if (state.mergedPRs.length === 0 &&
             state.starredRepos.length === 0 &&
             state.preferences.githubUsername) {
-            console.log('Run `oss-scout bootstrap` to import your starred repos and PR history for better results.\n');
+            console.log("Run `oss-scout bootstrap` to import your starred repos and PR history for better results.\n");
         }
         // Parse --strategy option
-        const strategyTokens = (options.strategy ?? 'all').split(',').map(s => s.trim()).filter(Boolean);
+        const strategyTokens = (options.strategy ?? "all")
+            .split(",")
+            .map((s) => s.trim())
+            .filter(Boolean);
         const strategies = [];
         for (const token of strategyTokens) {
             const parsed = SearchStrategySchema.safeParse(token);
             if (!parsed.success) {
-                const valid = [...CONCRETE_STRATEGIES, 'all'].join(', ');
-                console.error('Error: unknown strategy "' + token + '". Valid strategies: ' + valid);
+                const valid = [...CONCRETE_STRATEGIES, "all"].join(", ");
+                console.error('Error: unknown strategy "' +
+                    token +
+                    '". Valid strategies: ' +
+                    valid);
                 process.exit(1);
             }
             strategies.push(parsed.data);
@@ -122,7 +132,11 @@ program
             // Human-readable output
             console.log(`\nFound ${results.candidates.length} issue candidates:\n`);
             for (const c of results.candidates) {
-                const icon = c.recommendation === 'approve' ? '✅' : c.recommendation === 'skip' ? '❌' : '⚠️';
+                const icon = c.recommendation === "approve"
+                    ? "✅"
+                    : c.recommendation === "skip"
+                        ? "❌"
+                        : "⚠️";
                 console.log(`  ${icon} ${c.issue.repo}#${c.issue.number} [${c.viabilityScore}/100]`);
                 console.log(`     ${c.issue.title}`);
                 console.log(`     ${c.issue.url}`);
@@ -142,33 +156,33 @@ program
 });
 // ── results command ────────────────────────────────────────────────
 const resultsCmd = program
-    .command('results')
-    .description('Show saved search results');
+    .command("results")
+    .description("Show saved search results");
 resultsCmd
-    .command('show', { isDefault: true })
-    .description('Display saved search results')
-    .option('--json', 'Output as JSON')
+    .command("show", { isDefault: true })
+    .description("Display saved search results")
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        const { runResults } = await import('./commands/results.js');
+        const { runResults } = await import("./commands/results.js");
         const results = await runResults(options);
         if (options.json) {
             console.log(formatJsonSuccess(results));
         }
         else {
             if (results.length === 0) {
-                console.log('\nNo saved results. Run `oss-scout search` to find issues.\n');
+                console.log("\nNo saved results. Run `oss-scout search` to find issues.\n");
                 return;
             }
             console.log(`\nSaved results (${results.length}):\n`);
-            console.log('  Score  Repo                              Issue   Recommendation  Title');
-            console.log('  ─────  ────────────────────────────────  ──────  ──────────────  ─────');
+            console.log("  Score  Repo                              Issue   Recommendation  Title");
+            console.log("  ─────  ────────────────────────────────  ──────  ──────────────  ─────");
             for (const r of results) {
                 const score = String(r.viabilityScore).padStart(3);
                 const repo = r.repo.padEnd(32).slice(0, 32);
                 const issue = `#${r.number}`.padEnd(6);
                 const rec = r.recommendation.padEnd(14);
-                const title = r.title.length > 50 ? r.title.slice(0, 47) + '...' : r.title;
+                const title = r.title.length > 50 ? r.title.slice(0, 47) + "..." : r.title;
                 console.log(`  ${score}    ${repo}  ${issue}  ${rec}  ${title}`);
             }
             console.log();
@@ -179,18 +193,18 @@ resultsCmd
     }
 });
 resultsCmd
-    .command('clear')
-    .description('Clear all saved results')
-    .option('--json', 'Output as JSON')
+    .command("clear")
+    .description("Clear all saved results")
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        const { runResultsClear } = await import('./commands/results.js');
+        const { runResultsClear } = await import("./commands/results.js");
         await runResultsClear();
         if (options.json) {
             console.log(formatJsonSuccess({ cleared: true }));
         }
         else {
-            console.log('Saved results cleared.');
+            console.log("Saved results cleared.");
         }
     }
     catch (err) {
@@ -199,12 +213,12 @@ resultsCmd
 });
 // ── config command ──────────────────────────────────────────────────
 const configCmd = program
-    .command('config')
-    .description('View and update preferences')
-    .option('--json', 'Output as JSON')
+    .command("config")
+    .description("View and update preferences")
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        const { runConfigShow, getConfigData } = await import('./commands/config.js');
+        const { runConfigShow, getConfigData } = await import("./commands/config.js");
         if (options.json) {
             console.log(formatJsonSuccess(getConfigData()));
         }
@@ -217,12 +231,12 @@ const configCmd = program
     }
 });
 configCmd
-    .command('set <key> <value>')
-    .description('Update a single preference (e.g. config set minStars 100)')
-    .option('--json', 'Output as JSON')
+    .command("set <key> <value>")
+    .description("Update a single preference (e.g. config set minStars 100)")
+    .option("--json", "Output as JSON")
     .action(async (key, value, options) => {
     try {
-        const { runConfigSet } = await import('./commands/config.js');
+        const { runConfigSet } = await import("./commands/config.js");
         const updated = runConfigSet(key, value);
         if (options.json) {
             console.log(formatJsonSuccess(updated));
@@ -236,18 +250,18 @@ configCmd
     }
 });
 configCmd
-    .command('reset')
-    .description('Reset all preferences to defaults')
-    .option('--json', 'Output as JSON')
+    .command("reset")
+    .description("Reset all preferences to defaults")
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        const { runConfigReset } = await import('./commands/config.js');
+        const { runConfigReset } = await import("./commands/config.js");
         const defaults = runConfigReset();
         if (options.json) {
             console.log(formatJsonSuccess(defaults));
         }
         else {
-            console.log('✅ Preferences reset to defaults.');
+            console.log("✅ Preferences reset to defaults.");
         }
     }
     catch (err) {
@@ -255,18 +269,19 @@ configCmd
     }
 });
 program
-    .command('vet-list')
-    .description('Re-vet all saved search results and classify their current status')
-    .option('--prune', 'Remove unavailable issues from saved results')
-    .option('--concurrency <n>', 'Max concurrent API requests (default: 5)', parseInt)
-    .option('--json', 'Output as JSON')
+    .command("vet-list")
+    .description("Re-vet all saved search results and classify their current status")
+    .option("--prune", "Remove unavailable issues from saved results")
+    .option("--concurrency <n>", "Max concurrent API requests (default: 5)", parseInt)
+    .option("--json", "Output as JSON")
     .action(async (options) => {
     try {
-        if (options.concurrency !== undefined && (isNaN(options.concurrency) || options.concurrency < 1)) {
-            console.error('Error: --concurrency must be a positive integer');
+        if (options.concurrency !== undefined &&
+            (isNaN(options.concurrency) || options.concurrency < 1)) {
+            console.error("Error: --concurrency must be a positive integer");
             process.exit(1);
         }
-        const { runVetList } = await import('./commands/vet-list.js');
+        const { runVetList } = await import("./commands/vet-list.js");
         const state = loadLocalState();
         const result = await runVetList({
             state,
@@ -278,16 +293,21 @@ program
         }
         else {
             if (result.results.length === 0) {
-                console.log('\nNo saved results to vet. Run `oss-scout search` first.\n');
+                console.log("\nNo saved results to vet. Run `oss-scout search` first.\n");
                 return;
             }
             console.log(`\nVet-list results (${result.summary.total}):\n`);
             for (const r of result.results) {
-                const icon = r.status === 'still_available' ? '✅' :
-                    r.status === 'claimed' ? '🔒' :
-                        r.status === 'has_pr' ? '🔀' :
-                            r.status === 'closed' ? '🚫' : '❌';
-                const score = r.viabilityScore != null ? ` [${r.viabilityScore}/100]` : '';
+                const icon = r.status === "still_available"
+                    ? "✅"
+                    : r.status === "claimed"
+                        ? "🔒"
+                        : r.status === "has_pr"
+                            ? "🔀"
+                            : r.status === "closed"
+                                ? "🚫"
+                                : "❌";
+                const score = r.viabilityScore != null ? ` [${r.viabilityScore}/100]` : "";
                 console.log(`  ${icon} ${r.repo}#${r.number} — ${r.status}${score}`);
                 console.log(`     ${r.title}`);
             }
@@ -303,33 +323,37 @@ program
     }
 });
 program
-    .command('vet <issue-url>')
-    .description('Vet a specific GitHub issue for claimability and project health')
-    .option('--json', 'Output as JSON')
+    .command("vet <issue-url>")
+    .description("Vet a specific GitHub issue for claimability and project health")
+    .option("--json", "Output as JSON")
     .action(async (issueUrl, options) => {
     try {
-        const { runVet } = await import('./commands/vet.js');
+        const { runVet } = await import("./commands/vet.js");
         const state = loadLocalState();
         const result = await runVet({ issueUrl, state });
         if (options.json) {
             console.log(formatJsonSuccess(result));
         }
         else {
-            const icon = result.recommendation === 'approve' ? '✅' : result.recommendation === 'skip' ? '❌' : '⚠️';
+            const icon = result.recommendation === "approve"
+                ? "✅"
+                : result.recommendation === "skip"
+                    ? "❌"
+                    : "⚠️";
             console.log(`\n${icon} ${result.issue.repo}#${result.issue.number}: ${result.recommendation.toUpperCase()}`);
             console.log(`   ${result.issue.title}`);
             console.log(`   ${result.issue.url}\n`);
             if (result.reasonsToApprove.length > 0) {
-                console.log('Reasons to approve:');
+                console.log("Reasons to approve:");
                 for (const r of result.reasonsToApprove)
                     console.log(`  + ${r}`);
             }
             if (result.reasonsToSkip.length > 0) {
-                console.log('Reasons to skip:');
+                console.log("Reasons to skip:");
                 for (const r of result.reasonsToSkip)
                     console.log(`  - ${r}`);
             }
-            console.log(`\nProject health: ${result.projectHealth.isActive ? 'Active' : 'Inactive'}`);
+            console.log(`\nProject health: ${result.projectHealth.isActive ? "Active" : "Inactive"}`);
             console.log(`  Last commit: ${result.projectHealth.daysSinceLastCommit} days ago`);
             console.log(`  CI status: ${result.projectHealth.ciStatus}`);
         }

package/dist/commands/config.d.ts

@@ -1,7 +1,7 @@
 /**
  * Config command — view and update oss-scout preferences.
  */
-import type { ScoutPreferences } from '../core/schemas.js';
+import type { ScoutPreferences } from "../core/schemas.js";
 /**
  * Display current preferences in human-readable format.
  */

package/dist/commands/config.js

@@ -1,41 +1,37 @@
 /**
  * Config command — view and update oss-scout preferences.
  */
-import { loadLocalState, saveLocalState } from '../core/local-state.js';
-import { ScoutPreferencesSchema, IssueScopeSchema, ProjectCategorySchema, PersistenceModeSchema } from '../core/schemas.js';
-import { ValidationError } from '../core/errors.js';
-/** All known preference keys and their types. */
-const ARRAY_FIELDS = new Set([
-    'languages',
-    'labels',
-    'preferredOrgs',
-    'projectCategories',
-    'excludeRepos',
-    'excludeOrgs',
-    'aiPolicyBlocklist',
-]);
-const NUMBER_FIELDS = new Set(['minStars', 'maxIssueAgeDays', 'minRepoScoreThreshold']);
-const BOOLEAN_FIELDS = new Set(['includeDocIssues']);
-const STRING_FIELDS = new Set(['githubUsername']);
-const SCOPE_FIELD = 'scope';
-const ENUM_FIELDS = {
-    persistence: PersistenceModeSchema.options,
+import { loadLocalState, saveLocalState } from "../core/local-state.js";
+import { ScoutPreferencesSchema, IssueScopeSchema, ProjectCategorySchema, PersistenceModeSchema, SearchStrategySchema, } from "../core/schemas.js";
+import { ValidationError } from "../core/errors.js";
+const FIELD_CONFIGS = {
+    languages: { type: "array" },
+    labels: { type: "array" },
+    excludeRepos: { type: "array" },
+    excludeOrgs: { type: "array" },
+    aiPolicyBlocklist: { type: "array" },
+    preferredOrgs: { type: "array" },
+    minStars: { type: "number" },
+    maxIssueAgeDays: { type: "number" },
+    minRepoScoreThreshold: { type: "number" },
+    includeDocIssues: { type: "boolean" },
+    scope: { type: "enum-array", validValues: IssueScopeSchema.options },
+    projectCategories: {
+        type: "enum-array",
+        validValues: ProjectCategorySchema.options,
+    },
+    persistence: { type: "enum", validValues: PersistenceModeSchema.options },
+    defaultStrategy: {
+        type: "enum-array",
+        validValues: SearchStrategySchema.options,
+    },
+    githubUsername: { type: "string" },
 };
-const ALL_FIELDS = new Set([
-    ...ARRAY_FIELDS,
-    ...NUMBER_FIELDS,
-    ...BOOLEAN_FIELDS,
-    ...STRING_FIELDS,
-    ...Object.keys(ENUM_FIELDS),
-    SCOPE_FIELD,
-]);
-const VALID_SCOPES = IssueScopeSchema.options;
-const VALID_CATEGORIES = ProjectCategorySchema.options;
 function parseBoolean(value) {
     const lower = value.toLowerCase();
-    if (lower === 'true' || lower === 'yes')
+    if (lower === "true" || lower === "yes")
         return true;
-    if (lower === 'false' || lower === 'no')
+    if (lower === "false" || lower === "no")
         return false;
     throw new ValidationError(`Invalid boolean value: "${value}". Use true/false or yes/no.`);
 }
@@ -48,7 +44,7 @@ function parseNumber(value, key) {
 }
 function parseArrayValue(value) {
     return value
-        .split(',')
+        .split(",")
         .map((s) => s.trim())
         .filter((s) => s.length > 0);
 }
@@ -56,7 +52,7 @@ function parseArrayValue(value) {
  * Apply an array update: plain set, +append, or -remove.
  */
 function updateArray(current, value) {
-    if (value.startsWith('+')) {
+    if (value.startsWith("+")) {
         const toAdd = parseArrayValue(value.slice(1));
         const merged = [...current];
         for (const item of toAdd) {
@@ -65,14 +61,14 @@ function updateArray(current, value) {
         }
         return merged;
     }
-    if (value.startsWith('-')) {
+    if (value.startsWith("-")) {
         const toRemove = new Set(parseArrayValue(value.slice(1)));
         return current.filter((item) => !toRemove.has(item));
     }
     return parseArrayValue(value);
 }
 function formatArray(arr) {
-    return arr.length > 0 ? arr.join(', ') : '(none)';
+    return arr.length > 0 ? arr.join(", ") : "(none)";
 }
 /**
  * Display current preferences in human-readable format.
@@ -84,11 +80,11 @@ export function runConfigShow(options) {
         // JSON output handled by caller
         return;
     }
-    console.log('\n⚙️  oss-scout preferences\n');
-    console.log(`  githubUsername:        ${prefs.githubUsername || '(not set)'}`);
+    console.log("\n⚙️  oss-scout preferences\n");
+    console.log(`  githubUsername:        ${prefs.githubUsername || "(not set)"}`);
     console.log(`  languages:            ${formatArray(prefs.languages)}`);
     console.log(`  labels:               ${formatArray(prefs.labels)}`);
-    console.log(`  scope:                ${prefs.scope ? formatArray(prefs.scope) : '(all)'}`);
+    console.log(`  scope:                ${prefs.scope ? formatArray(prefs.scope) : "(all)"}`);
     console.log(`  minStars:             ${prefs.minStars}`);
     console.log(`  maxIssueAgeDays:      ${prefs.maxIssueAgeDays}`);
     console.log(`  minRepoScoreThreshold: ${prefs.minRepoScoreThreshold}`);
@@ -98,6 +94,7 @@ export function runConfigShow(options) {
     console.log(`  excludeRepos:         ${formatArray(prefs.excludeRepos)}`);
     console.log(`  excludeOrgs:          ${formatArray(prefs.excludeOrgs)}`);
     console.log(`  aiPolicyBlocklist:    ${formatArray(prefs.aiPolicyBlocklist)}`);
+    console.log(`  defaultStrategy:      ${prefs.defaultStrategy ? formatArray(prefs.defaultStrategy) : "(all)"}`);
     console.log(`  persistence:          ${prefs.persistence}`);
     console.log();
 }
@@ -112,46 +109,53 @@ export function getConfigData() {
  * Update a single preference by key.
  */
 export function runConfigSet(key, value) {
-    if (!ALL_FIELDS.has(key)) {
-        throw new ValidationError(`Unknown config key: "${key}". Valid keys: ${[...ALL_FIELDS].sort().join(', ')}`);
+    const field = FIELD_CONFIGS[key];
+    if (!field) {
+        throw new ValidationError(`Unknown config key: "${key}". Valid keys: ${Object.keys(FIELD_CONFIGS).sort().join(", ")}`);
     }
     const state = loadLocalState();
     const prefs = { ...state.preferences };
-    if (STRING_FIELDS.has(key)) {
-        prefs[key] = value;
-    }
-    else if (BOOLEAN_FIELDS.has(key)) {
-        prefs[key] = parseBoolean(value);
-    }
-    else if (NUMBER_FIELDS.has(key)) {
-        prefs[key] = parseNumber(value, key);
-    }
-    else if (key === SCOPE_FIELD) {
-        const updated = updateArray(prefs.scope ?? [], value);
-        const invalid = updated.filter((s) => !VALID_SCOPES.includes(s));
-        if (invalid.length > 0) {
-            throw new ValidationError(`Invalid scope value(s): ${invalid.join(', ')}. Valid: ${VALID_SCOPES.join(', ')}`);
+    switch (field.type) {
+        case "string":
+            prefs[key] = value;
+            break;
+        case "boolean":
+            prefs[key] = parseBoolean(value);
+            break;
+        case "number":
+            prefs[key] = parseNumber(value, key);
+            break;
+        case "array": {
+            const current = prefs[key] ?? [];
+            prefs[key] = updateArray(current, value);
+            break;
         }
-        prefs.scope = updated.length > 0 ? updated : undefined;
-    }
-    else if (key === 'projectCategories') {
-        const updated = updateArray(prefs.projectCategories, value);
-        const invalid = updated.filter((s) => !VALID_CATEGORIES.includes(s));
-        if (invalid.length > 0) {
-            throw new ValidationError(`Invalid category value(s): ${invalid.join(', ')}. Valid: ${VALID_CATEGORIES.join(', ')}`);
+        case "enum": {
+            const validValues = field.validValues;
+            if (!validValues.includes(value)) {
+                throw new ValidationError(`Invalid value for "${key}": "${value}". Valid: ${validValues.join(", ")}`);
+            }
+            prefs[key] = value;
+            break;
         }
-        prefs.projectCategories = updated;
-    }
-    else if (key in ENUM_FIELDS) {
-        const validValues = ENUM_FIELDS[key];
-        if (!validValues.includes(value)) {
-            throw new ValidationError(`Invalid value for "${key}": "${value}". Valid: ${validValues.join(', ')}`);
+        case "enum-array": {
+            const current = prefs[key] ?? [];
+            const updated = updateArray(current, value);
+            const validValues = field.validValues;
+            const invalid = updated.filter((s) => !validValues.includes(s));
+            if (invalid.length > 0) {
+                throw new ValidationError(`Invalid value(s) for "${key}": ${invalid.join(", ")}. Valid: ${validValues.join(", ")}`);
+            }
+            // For 'scope', empty array means undefined (all scopes)
+            if (key === "scope") {
+                prefs[key] =
+                    updated.length > 0 ? updated : undefined;
+            }
+            else {
+                prefs[key] = updated;
+            }
+            break;
         }
-        prefs[key] = value;
-    }
-    else if (ARRAY_FIELDS.has(key)) {
-        const current = prefs[key] ?? [];
-        prefs[key] = updateArray(current, value);
     }
     // Validate the full preferences object
     const validated = ScoutPreferencesSchema.parse(prefs);

package/dist/commands/results.d.ts

@@ -1,7 +1,7 @@
 /**
  * Results command — display and manage saved search results.
  */
-import type { SavedCandidate } from '../core/schemas.js';
+import type { SavedCandidate } from "../core/schemas.js";
 export declare function runResults(_options: {
     json?: boolean;
 }): Promise<SavedCandidate[]>;

package/dist/commands/results.js

@@ -1,7 +1,7 @@
 /**
  * Results command — display and manage saved search results.
  */
-import { loadLocalState, saveLocalState } from '../core/local-state.js';
+import { loadLocalState, saveLocalState } from "../core/local-state.js";
 export async function runResults(_options) {
     const state = loadLocalState();
     return state.savedResults ?? [];