@oss-scout/core 0.2.0 → 0.3.0

package/dist/core/http-cache.js

@@ -9,13 +9,13 @@
  * for the same endpoint (e.g., star counts for two PRs in the same repo)
  * share a single HTTP round-trip.
  */
-import * as fs from 'fs';
-import * as path from 'path';
-import * as crypto from 'crypto';
-import { getCacheDir } from './utils.js';
-import { debug, warn } from './logger.js';
-import { errorMessage, getHttpStatusCode } from './errors.js';
-const MODULE = 'http-cache';
+import * as fs from "fs";
+import * as path from "path";
+import * as crypto from "crypto";
+import { getCacheDir } from "./utils.js";
+import { debug, warn } from "./logger.js";
+import { errorMessage, getHttpStatusCode } from "./errors.js";
+const MODULE = "http-cache";
 /**
  * Maximum age (in ms) before a cache entry is considered stale and eligible for
  * eviction during cleanup. Defaults to 24 hours. Entries older than this are
@@ -39,7 +39,7 @@ export class HttpCache {
     }
     /** Derive a filesystem-safe cache key from a URL. */
     keyFor(url) {
-        return crypto.createHash('sha256').update(url).digest('hex');
+        return crypto.createHash("sha256").update(url).digest("hex");
     }
     /** Full path to the cache file for a given URL. */
     pathFor(url) {
@@ -65,7 +65,7 @@ export class HttpCache {
     get(url) {
         const filePath = this.pathFor(url);
         try {
-            const raw = fs.readFileSync(filePath, 'utf-8');
+            const raw = fs.readFileSync(filePath, "utf-8");
             const entry = JSON.parse(raw);
             // Sanity-check: the file should contain the URL we asked for
             if (entry.url !== url) {
@@ -76,7 +76,7 @@ export class HttpCache {
         }
         catch (err) {
             const code = err?.code;
-            if (code === 'ENOENT')
+            if (code === "ENOENT")
                 return null;
             if (err instanceof SyntaxError) {
                 debug(MODULE, `Corrupt cache entry, deleting: ${url}`);
@@ -103,7 +103,10 @@ export class HttpCache {
             cachedAt: new Date().toISOString(),
         };
         try {
-            fs.writeFileSync(this.pathFor(url), JSON.stringify(entry), { encoding: 'utf-8', mode: 0o600 });
+            fs.writeFileSync(this.pathFor(url), JSON.stringify(entry), {
+                encoding: "utf-8",
+                mode: 0o600,
+            });
             debug(MODULE, `Cached response for ${url}`);
         }
         catch (err) {
@@ -137,11 +140,11 @@ export class HttpCache {
             const files = fs.readdirSync(this.cacheDir);
             const now = Date.now();
             for (const file of files) {
-                if (!file.endsWith('.json'))
+                if (!file.endsWith(".json"))
                     continue;
                 const filePath = path.join(this.cacheDir, file);
                 try {
-                    const raw = fs.readFileSync(filePath, 'utf-8');
+                    const raw = fs.readFileSync(filePath, "utf-8");
                     const entry = JSON.parse(raw);
                     const age = now - new Date(entry.cachedAt).getTime();
                     if (age > maxAgeMs) {
@@ -163,7 +166,7 @@ export class HttpCache {
         }
         catch (err) {
             const code = err?.code;
-            if (code !== 'ENOENT') {
+            if (code !== "ENOENT") {
                 warn(MODULE, `Failed to evict stale cache entries: ${errorMessage(err)}`);
             }
         }
@@ -179,15 +182,15 @@ export class HttpCache {
         try {
             const files = fs.readdirSync(this.cacheDir);
             for (const file of files) {
-                if (!file.endsWith('.json'))
+                if (!file.endsWith(".json"))
                     continue;
                 fs.unlinkSync(path.join(this.cacheDir, file));
             }
-            debug(MODULE, 'Cache cleared');
+            debug(MODULE, "Cache cleared");
         }
         catch (err) {
             const code = err?.code;
-            if (code !== 'ENOENT') {
+            if (code !== "ENOENT") {
                 warn(MODULE, `Failed to clear cache: ${errorMessage(err)}`);
             }
         }
@@ -197,11 +200,12 @@ export class HttpCache {
      */
     size() {
         try {
-            return fs.readdirSync(this.cacheDir).filter((f) => f.endsWith('.json')).length;
+            return fs.readdirSync(this.cacheDir).filter((f) => f.endsWith(".json"))
+                .length;
         }
         catch (err) {
             const code = err?.code;
-            if (code !== 'ENOENT') {
+            if (code !== "ENOENT") {
                 debug(MODULE, `Failed to read cache size: ${errorMessage(err)}`);
             }
             return 0;
@@ -253,12 +257,12 @@ export async function cachedRequest(cache, url, fetcher) {
         const extraHeaders = {};
         const cached = cache.get(url);
         if (cached) {
-            extraHeaders['if-none-match'] = cached.etag;
+            extraHeaders["if-none-match"] = cached.etag;
         }
         try {
             const response = await fetcher(extraHeaders);
             // Store ETag if present (headers may be absent in test mocks)
-            const etag = response.headers?.['etag'];
+            const etag = response.headers?.["etag"];
             if (etag) {
                 cache.set(url, etag, response.data);
             }
@@ -302,7 +306,7 @@ export async function cachedTimeBased(cache, key, maxAgeMs, fetcher) {
         return cached;
     }
     const result = await fetcher();
-    cache.set(key, '', result);
+    cache.set(key, "", result);
     return result;
 }
 /**

package/dist/core/issue-discovery.d.ts

@@ -11,15 +11,14 @@
  *
  * All state is injected via constructor parameters (ScoutStateReader + ScoutPreferences).
  */
-import { type IssueCandidate } from './types.js';
-import type { ScoutPreferences, SearchStrategy } from './schemas.js';
-import { type ScoutStateReader } from './issue-vetting.js';
+import { type IssueCandidate } from "./types.js";
+import type { ScoutPreferences, SearchStrategy } from "./schemas.js";
+import { type ScoutStateReader } from "./issue-vetting.js";
 /**
  * Multi-phase issue discovery engine that searches GitHub for contributable issues.
  *
  * Search phases (in priority order):
  * 0. Repos where user has merged PRs (highest merge probability)
- * 0.5. Preferred organizations
  * 1. Starred repos
  * 2. General label-filtered search
  * 3. Actively maintained repos
@@ -47,8 +46,8 @@ export declare class IssueDiscovery {
     getStarredRepos(): string[];
     /**
      * Search for issues matching our criteria.
-     * Searches in priority order: merged-PR repos first (no label filter), then preferred
-     * organizations, then starred repos, then general search, then actively maintained repos.
+     * Searches in priority order: merged-PR repos first (no label filter), then starred
+     * repos, then general search, then actively maintained repos.
      * Filters out issues from low-scoring and excluded repos.
      *
      * @param options - Search configuration
@@ -74,6 +73,7 @@ export declare class IssueDiscovery {
         labels?: string[];
         maxResults?: number;
         strategies?: SearchStrategy[];
+        skippedUrls?: Set<string>;
     }): Promise<{
         candidates: IssueCandidate[];
         strategiesUsed: SearchStrategy[];