@getfamiliar/shared 0.1.11 → 0.1.12

package/build/HandlerCatalog.d.ts

@@ -18,16 +18,6 @@ export interface HandlerPath {
     /** User-facing slash form, e.g. `/chat/telegram/index`. */
     readonly slashPath: string;
 }
-/**
- * Whether a workspace-relative path lies inside the top-level `skills/`
- * folder. Shared by host (catalog, cron scheduler, `cron list`) and
- * container (`HandlerFile.load`) so every side agrees that skills are
- * never handlers.
- *
- * @param relativePath Workspace-relative posix path, e.g. `skills/jira-issue/SKILL.md`.
- * @returns `true` when the first path segment is `skills`.
- */
-export declare function isUnderSkillsFolder(relativePath: string): boolean;
 /**
  * Read-only view over the `workspace/` directory's handler files.
  *

package/build/HandlerCatalog.js

@@ -2,24 +2,14 @@ import { promises as fs } from "node:fs";
 import path from "node:path";
 import { matchesAnyGlob } from "./PathGlob.js";
 /**
- * Top-level workspace folder holding shared recipes
- * (`skills/<name>/SKILL.md`). Skills are pure context loaded via
- * `fs_read`, never agent entry points — so nothing under this folder
- * is ever listed, resolved, loaded or cron-scheduled as a handler.
+ * Workspace subtrees excluded from {@link HandlerCatalog.list} (cli-chat
+ * tab-completion), so `SKILL.md` recipes don't clutter the proposals.
+ * This is a listing concern only: handlers that live next to a skill
+ * (e.g. `skills/memory/save.md`, or a `skills/memory/dream.md` with a
+ * `cron:` field) are deliberately loadable, resolvable and
+ * cron-scheduled like any other handler.
  */
-const SKILLS_FOLDER = "skills";
-/**
- * Whether a workspace-relative path lies inside the top-level `skills/`
- * folder. Shared by host (catalog, cron scheduler, `cron list`) and
- * container (`HandlerFile.load`) so every side agrees that skills are
- * never handlers.
- *
- * @param relativePath Workspace-relative posix path, e.g. `skills/jira-issue/SKILL.md`.
- * @returns `true` when the first path segment is `skills`.
- */
-export function isUnderSkillsFolder(relativePath) {
-    return relativePath.split("/")[0] === SKILLS_FOLDER;
-}
+const RESERVED_TOP_DIRS = new Set(["skills"]);
 /**
  * Read-only view over the `workspace/` directory's handler files.
  *
@@ -92,11 +82,6 @@ export class HandlerCatalog {
             const relPosix = rel.split(path.sep).join("/");
             const absolute = path.join(this.workspaceDir, rel);
             if (await fileExists(absolute)) {
-                // Skills are shared recipes, never handlers — mirror the
-                // container's `HandlerFile.load` guard.
-                if (isUnderSkillsFolder(relPosix)) {
-                    return null;
-                }
                 // A file under `core.writablePaths` is never a handler, even
                 // when it exists — mirror the container's `HandlerFile.load`
                 // guard so host-side callers agree it does not resolve.
@@ -125,7 +110,7 @@ async function walkMarkdown(dir, segments, out, workspaceDir, writablePathGlobs)
     }
     for (const entry of entries) {
         if (entry.isDirectory()) {
-            if (segments.length === 0 && entry.name === SKILLS_FOLDER) {
+            if (segments.length === 0 && RESERVED_TOP_DIRS.has(entry.name)) {
                 continue;
             }
             await walkMarkdown(path.join(dir, entry.name), [...segments, entry.name], out, workspaceDir, writablePathGlobs);

package/build/index.d.ts

@@ -11,7 +11,7 @@ export type { EventFile, EventFilter, EventPatch, EventRow, EventState, NewEvent
 export { EVENT_PRIORITY } from "./Event.js";
 export { DuplicateIdempotencyKeyError, EventBus } from "./EventBus.js";
 export { estimateTokens } from "./estimateTokens.js";
-export { HandlerCatalog, type HandlerPath, isUnderSkillsFolder } from "./HandlerCatalog.js";
+export { HandlerCatalog, type HandlerPath } from "./HandlerCatalog.js";
 export type { InferenceEventRow, InferenceOutcome, NewInferenceEvent, } from "./InferenceEvent.js";
 export { InferenceEventBus } from "./InferenceEventBus.js";
 export { type CreateLoggerOptions, createLogger, jsonStdoutStream, type Logger, type LogLevel, type LogStream, prettyStdoutStream, } from "./logging/Logger.js";

package/build/index.js

@@ -6,7 +6,7 @@ export { parseCron } from "./CronExpression.js";
 export { EVENT_PRIORITY } from "./Event.js";
 export { DuplicateIdempotencyKeyError, EventBus } from "./EventBus.js";
 export { estimateTokens } from "./estimateTokens.js";
-export { HandlerCatalog, isUnderSkillsFolder } from "./HandlerCatalog.js";
+export { HandlerCatalog } from "./HandlerCatalog.js";
 export { InferenceEventBus } from "./InferenceEventBus.js";
 export { createLogger, jsonStdoutStream, prettyStdoutStream, } from "./logging/Logger.js";
 export { buildMailId, parseMailId, } from "./Mail.js";

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@getfamiliar/shared",
   "type": "module",
-  "version": "0.1.11",
+  "version": "0.1.12",
   "description": "Familiar — shared types and bus-state client between host and container.",
   "publishConfig": {
     "access": "public"