@jini-ai/ui 0.1.2 → 0.3.0

package/dist/features/skills/rules.js

@@ -0,0 +1,272 @@
+/**
+ * Origin: the `useMemo` block of `SkillsSection.tsx` (lines ~146-240) plus its
+ * three module-level helpers (`summaryToDraft`, `parseTriggers`,
+ * `skillMatchesSearch`). All of it is pure derivation over a skill list — no
+ * React, no I/O — so it belongs here rather than in the component.
+ *
+ * ONE DELIBERATE DIVERGENCE FROM THE ORIGIN, called out because a reviewer
+ * diffing the two will notice it immediately: the origin repeats the same
+ * "does this skill match every active filter EXCEPT my own dimension"
+ * predicate six times, once inline per `useMemo` (`sourceCounts`,
+ * `modeOptions`, `modeAllCount`, `categoryOptions`, `categoryAllCount`,
+ * `filteredSkills`). Those six copies are not identical by accident — they are
+ * identical by intent, and they had already drifted in one respect: the
+ * category memos skip skills with no category before counting, while the mode
+ * memos do not. Here it is ONE parameterised predicate
+ * (`skillMatchesFilters` + its `except` option), which is why this file is
+ * shorter than the region it replaces while covering the same cases.
+ *
+ * Why counts exclude their own dimension: a filter pill should say what
+ * picking it WOULD yield, not what is showing now. Counting "mode" pills with
+ * the mode filter still applied would show 0 on every unselected pill, which
+ * reads as "empty" when it means "not currently selected".
+ */
+/** The blank draft a "create new skill" form starts from. */
+export const EMPTY_SKILL_DRAFT = {
+    name: '',
+    description: '',
+    triggers: '',
+    body: '',
+};
+/** The two origins the source filter offers explicitly. A host may report
+ *  others (an org registry, a marketplace); those are counted under `all` and
+ *  get no pill of their own, matching the origin's behaviour. */
+const KNOWN_SOURCES = ['all', 'user', 'built-in'];
+/**
+ * The skill's name in `locale`, falling back to the untranslated `name`.
+ *
+ * A host that does not localize its catalog omits `displayName` entirely, so
+ * the fallback is the normal path rather than an error case.
+ */
+export function localizedSkillName(skill, locale) {
+    return skill.displayName?.[locale] ?? skill.name;
+}
+/** The skill's description in `locale`, falling back to the untranslated one. */
+export function localizedSkillDescription(skill, locale) {
+    return skill.descriptionI18n?.[locale] ?? skill.description;
+}
+/**
+ * Whether `skill` matches the free-text `query`.
+ *
+ * Searches the untranslated name AND the localized one, the untranslated
+ * description AND the localized one, every trigger phrase, and the category —
+ * so an operator can find a skill by what their UI calls it or by what its
+ * files call it.
+ *
+ * `query` is normalised (trimmed and lowercased) INSIDE this function. The
+ * origin required callers to pre-lowercase it and silently returned nothing
+ * for a capitalised query if they forgot; every call site there did the
+ * normalisation separately. An empty or whitespace-only query matches
+ * everything, which is what makes an untouched search box a no-op.
+ */
+export function skillMatchesSearch(skill, query, locale) {
+    const needle = query.trim().toLowerCase();
+    if (!needle)
+        return true;
+    const haystack = [
+        skill.name,
+        localizedSkillName(skill, locale),
+        skill.description,
+        localizedSkillDescription(skill, locale),
+        (skill.triggers ?? []).join(' '),
+        skill.category ?? '',
+    ]
+        .join('\n')
+        .toLowerCase();
+    return haystack.includes(needle);
+}
+/**
+ * Whether `skill` survives every active filter, optionally ignoring one
+ * dimension.
+ *
+ * `except` is what makes the filter-pill counts correct — see this file's
+ * header. Omit it and this is the plain "is this row visible" predicate.
+ */
+export function skillMatchesFilters(skill, filters, locale, except) {
+    if (except !== 'source' && filters.source !== 'all' && skill.source !== filters.source)
+        return false;
+    if (except !== 'mode' && filters.mode !== 'all' && skill.mode !== filters.mode)
+        return false;
+    if (except !== 'category' && filters.category !== 'all' && skill.category !== filters.category) {
+        return false;
+    }
+    return skillMatchesSearch(skill, filters.search, locale);
+}
+/** The rows the list should render. */
+export function filterSkills(skills, filters, locale) {
+    return skills.filter((skill) => skillMatchesFilters(skill, filters, locale));
+}
+/**
+ * The pills for one filter dimension, plus the count its "all" pill shows.
+ *
+ * Replaces five of the origin's six memos. Two properties are preserved from
+ * it deliberately:
+ *
+ * - **The option SET comes from every skill, the COUNTS from the filtered
+ *   ones.** So an option can legitimately show 0 — it stays visible, telling
+ *   the operator the value exists but is excluded by their other filters.
+ *   Dropping zero-count options would make pills appear and vanish as filters
+ *   change, which is worse.
+ * - **`source` always offers its two known pills**, even when no skill has
+ *   that origin, because they are a fixed vocabulary rather than data-derived.
+ *   Unknown origins a host may report are counted in `all` but get no pill.
+ *
+ * Options are sorted by value (`localeCompare`) so the row is stable across
+ * renders rather than reflecting list order.
+ */
+export function skillFilterOptions(skills, filters, locale, dimension) {
+    const matching = skills.filter((skill) => skillMatchesFilters(skill, filters, locale, dimension));
+    if (dimension === 'source') {
+        const counts = new Map(KNOWN_SOURCES.filter((s) => s !== 'all').map((s) => [s, 0]));
+        for (const skill of matching) {
+            // `undefined` means an origin outside the known vocabulary (an org
+            // registry, a marketplace). Those are counted in `all` and get no pill.
+            // One lookup, not a `has` + `get` pair with an unreachable `?? 0`.
+            const current = counts.get(skill.source);
+            if (current !== undefined)
+                counts.set(skill.source, current + 1);
+        }
+        return {
+            all: matching.length,
+            options: [...counts.entries()]
+                .map(([value, count]) => [value, count])
+                .sort((a, b) => a[0].localeCompare(b[0])),
+        };
+    }
+    // `mode` is required on every skill; `category` is optional, so a skill
+    // without one contributes to `all` but to no pill.
+    const valueOf = (skill) => dimension === 'mode' ? skill.mode : typeof skill.category === 'string' && skill.category ? skill.category : null;
+    const values = new Set();
+    for (const skill of skills) {
+        const value = valueOf(skill);
+        if (value !== null)
+            values.add(value);
+    }
+    const counts = new Map();
+    for (const skill of matching) {
+        const value = valueOf(skill);
+        if (value === null)
+            continue;
+        counts.set(value, (counts.get(value) ?? 0) + 1);
+    }
+    return {
+        all: matching.length,
+        options: [...values]
+            .map((value) => [value, counts.get(value) ?? 0])
+            .sort((a, b) => a[0].localeCompare(b[0])),
+    };
+}
+/**
+ * Whether the category filter row should render at all.
+ *
+ * Categories are optional per-skill metadata. A host shipping only
+ * uncategorised skills should see no row rather than a row with nothing but
+ * an "all" pill — the origin's own note calls this out.
+ */
+export function hasAnyCategory(skills) {
+    return skills.some((skill) => typeof skill.category === 'string' && skill.category.length > 0);
+}
+/**
+ * Splits the raw triggers textarea into phrases.
+ *
+ * Comma OR newline separated, trimmed, blanks dropped — so trailing
+ * separators and a stray blank line while typing don't produce empty
+ * triggers. The draft keeps the RAW text (see `SkillDraft.triggers`); this
+ * runs at save time, which is why a half-typed entry survives re-renders.
+ */
+export function parseTriggers(raw) {
+    return raw
+        .split(/[,\n]/)
+        .map((trigger) => trigger.trim())
+        .filter(Boolean);
+}
+/**
+ * Seeds the edit form from an existing skill plus its lazily-fetched body.
+ *
+ * `triggers` is joined back to comma-separated text because the form edits the
+ * raw string, not the array. The `Array.isArray` guard is kept from the origin:
+ * `triggers` is optional in the contract and a host can return `null` for it,
+ * which `.join` would throw on.
+ */
+export function summaryToDraft(skill, body) {
+    return {
+        name: skill.name,
+        description: skill.description,
+        triggers: Array.isArray(skill.triggers) ? skill.triggers.join(', ') : '',
+        body,
+    };
+}
+/** Which required field a draft is missing, or `null` when it may be
+ *  submitted. Origin: the two sequential `if (!name) {...}; if (!body)
+ *  {...}` checks inline in `submitDraft` — one pure predicate instead of two
+ *  inline early-returns, so the component only has to pick which message to
+ *  show, not decide whether to. Checked in this order (name before body) to
+ *  match the origin's precedence — an operator who left both blank sees the
+ *  name error first. */
+export function validateSkillDraft(draft) {
+    if (!draft.name.trim())
+        return 'name-required';
+    if (!draft.body.trim())
+        return 'body-required';
+    return null;
+}
+/**
+ * Kebab-case category slugs (`'image-generation'`) rendered as Title Case
+ * (`'Image Generation'`) for the filter pill / category badge. Origin:
+ * `humanizeCategory` inline in `SkillsSection.tsx`.
+ */
+export function humanizeSkillCategory(slug) {
+    if (!slug)
+        return slug;
+    return slug
+        .split('-')
+        .map((word) => (word.length === 0 ? word : word.charAt(0).toUpperCase() + word.slice(1)))
+        .join(' ');
+}
+/** The last `/`-separated segment of a skill file's path — its own file/
+ *  folder name, without the parent tree. Origin: `leafName` inline in
+ *  `SkillsSection.tsx`. Unlike `locationLabel` (project-locations), a skill
+ *  file path is always daemon-reported and forward-slash-only, so this does
+ *  not also handle `\`. */
+export function skillFileLeafName(path) {
+    const idx = path.lastIndexOf('/');
+    return idx >= 0 ? path.slice(idx + 1) : path;
+}
+/**
+ * Indent (in px) for one row of the file-tree list, from its path depth.
+ *
+ * Each `/`-separated segment indents by 12px so a small `assets/` tree reads
+ * as a tree without building a nested list, capped at 4 levels so a deeply
+ * nested bundle does not push the file label out of the panel. Origin:
+ * `depthIndent` inline in `SkillsSection.tsx`.
+ */
+export function skillFileTreeIndent(path) {
+    const depth = Math.min(4, path.split('/').length - 1);
+    return depth * 12;
+}
+/** A byte count as `'N B'` / `'N.N KB'` / `'N.N MB'`. Origin: `formatSize`
+ *  inline in `SkillsSection.tsx`. */
+export function formatSkillFileSize(bytes) {
+    if (bytes < 1024)
+        return `${bytes} B`;
+    if (bytes < 1024 * 1024)
+        return `${(bytes / 1024).toFixed(1)} KB`;
+    return `${(bytes / (1024 * 1024)).toFixed(1)} MB`;
+}
+/** Whether a skill's built-in origin means editing it must go through the
+ *  "creates a user-owned override" confirmation rather than editing in
+ *  place. Origin: `isBuiltIn = skill.source !== 'user'` inline in
+ *  `SkillRow`/`SkillsSection.requestEdit` — matches the origin's own
+ *  (deliberately inclusive) test: anything that is not `'user'` counts,
+ *  including a host's own third-party origins, not only `'built-in'`
+ *  literally. */
+export function isBuiltInSkill(skill) {
+    return skill.source !== 'user';
+}
+/** Whether a skill may be deleted by the operator — only ones they imported
+ *  themselves. Origin: `canDelete = skill.source === 'user'` inline in
+ *  `SkillRow`. */
+export function isDeletableSkill(skill) {
+    return skill.source === 'user';
+}
+//# sourceMappingURL=rules.js.map

package/dist/features/skills/rules.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"rules.js","sourceRoot":"","sources":["../../../src/features/skills/rules.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;GAsBG;AAUH,6DAA6D;AAC7D,MAAM,CAAC,MAAM,iBAAiB,GAAe;IAC3C,IAAI,EAAE,EAAE;IACR,WAAW,EAAE,EAAE;IACf,QAAQ,EAAE,EAAE;IACZ,IAAI,EAAE,EAAE;CACT,CAAC;AAEF;;iEAEiE;AACjE,MAAM,aAAa,GAA4B,CAAC,KAAK,EAAE,MAAM,EAAE,UAAU,CAAC,CAAC;AAE3E;;;;;GAKG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAmB,EAAE,MAAc;IACpE,OAAO,KAAK,CAAC,WAAW,EAAE,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,IAAI,CAAC;AACnD,CAAC;AAED,iFAAiF;AACjF,MAAM,UAAU,yBAAyB,CAAC,KAAmB,EAAE,MAAc;IAC3E,OAAO,KAAK,CAAC,eAAe,EAAE,CAAC,MAAM,CAAC,IAAI,KAAK,CAAC,WAAW,CAAC;AAC9D,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,MAAM,UAAU,kBAAkB,CAAC,KAAmB,EAAE,KAAa,EAAE,MAAc;IACnF,MAAM,MAAM,GAAG,KAAK,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IAC1C,IAAI,CAAC,MAAM;QAAE,OAAO,IAAI,CAAC;IACzB,MAAM,QAAQ,GAAG;QACf,KAAK,CAAC,IAAI;QACV,kBAAkB,CAAC,KAAK,EAAE,MAAM,CAAC;QACjC,KAAK,CAAC,WAAW;QACjB,yBAAyB,CAAC,KAAK,EAAE,MAAM,CAAC;QACxC,CAAC,KAAK,CAAC,QAAQ,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC;QAChC,KAAK,CAAC,QAAQ,IAAI,EAAE;KACrB;SACE,IAAI,CAAC,IAAI,CAAC;SACV,WAAW,EAAE,CAAC;IACjB,OAAO,QAAQ,CAAC,QAAQ,CAAC,MAAM,CAAC,CAAC;AACnC,CAAC;AAMD;;;;;;GAMG;AACH,MAAM,UAAU,mBAAmB,CACjC,KAAmB,EACnB,OAAqB,EACrB,MAAc,EACd,MAA6B;IAE7B,IAAI,MAAM,KAAK,QAAQ,IAAI,OAAO,CAAC,MAAM,KAAK,KAAK,IAAI,KAAK,CAAC,MAAM,KAAK,OAAO,CAAC,MAAM;QAAE,OAAO,KAAK,CAAC;IACrG,IAAI,MAAM,KAAK,MAAM,IAAI,OAAO,CAAC,IAAI,KAAK,KAAK,IAAI,KAAK,CAAC,IAAI,KAAK,OAAO,CAAC,IAAI;QAAE,OAAO,KAAK,CAAC;IAC7F,IAAI,MAAM,KAAK,UAAU,IAAI,OAAO,CAAC,QAAQ,KAAK,KAAK,IAAI,KAAK,CAAC,QAAQ,KAAK,OAAO,CAAC,QAAQ,EAAE,CAAC;QAC/F,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,kBAAkB,CAAC,KAAK,EAAE,OAAO,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;AAC3D,CAAC;AAED,uCAAuC;AACvC,MAAM,UAAU,YAAY,CAC1B,MAA+B,EAC/B,OAAqB,EACrB,MAAc;IAEd,OAAO,MAAM,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,mBAAmB,CAAC,KAAK,EAAE,OAAO,EAAE,MAAM,CAAC,CAAC,CAAC;AAC/E,CAAC;AAED;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,UAAU,kBAAkB,CAChC,MAA+B,EAC/B,OAAqB,EACrB,MAAc,EACd,SAA+B;IAE/B,MAAM,QAAQ,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,mBAAmB,CAAC,KAAK,EAAE,OAAO,EAAE,MAAM,EAAE,SAAS,CAAC,CAAC,CAAC;IAElG,IAAI,SAAS,KAAK,QAAQ,EAAE,CAAC;QAC3B,MAAM,MAAM,GAAG,IAAI,GAAG,CAAiB,aAAa,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QACpG,KAAK,MAAM,KAAK,IAAI,QAAQ,EAAE,CAAC;YAC7B,mEAAmE;YACnE,wEAAwE;YACxE,mEAAmE;YACnE,MAAM,OAAO,GAAG,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;YACzC,IAAI,OAAO,KAAK,SAAS;gBAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,EAAE,OAAO,GAAG,CAAC,CAAC,CAAC;QACnE,CAAC;QACD,OAAO;YACL,GAAG,EAAE,QAAQ,CAAC,MAAM;YACpB,OAAO,EAAE,CAAC,GAAG,MAAM,CAAC,OAAO,EAAE,CAAC;iBAC3B,GAAG,CAAC,CAAC,CAAC,KAAK,EAAE,KAAK,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,EAAE,KAAK,CAAsB,CAAC;iBAC5D,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;SAC5C,CAAC;IACJ,CAAC;IAED,wEAAwE;IACxE,mDAAmD;IACnD,MAAM,OAAO,GAAG,CAAC,KAAmB,EAAiB,EAAE,CACrD,SAAS,KAAK,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,OAAO,KAAK,CAAC,QAAQ,KAAK,QAAQ,IAAI,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,IAAI,CAAC;IAEnH,MAAM,MAAM,GAAG,IAAI,GAAG,EAAU,CAAC;IACjC,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC;QAC7B,IAAI,KAAK,KAAK,IAAI;YAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;IACxC,CAAC;IAED,MAAM,MAAM,GAAG,IAAI,GAAG,EAAkB,CAAC;IACzC,KAAK,MAAM,KAAK,IAAI,QAAQ,EAAE,CAAC;QAC7B,MAAM,KAAK,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC;QAC7B,IAAI,KAAK,KAAK,IAAI;YAAE,SAAS;QAC7B,MAAM,CAAC,GAAG,CAAC,KAAK,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;IAClD,CAAC;IAED,OAAO;QACL,GAAG,EAAE,QAAQ,CAAC,MAAM;QACpB,OAAO,EAAE,CAAC,GAAG,MAAM,CAAC;aACjB,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,KAAK,EAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,CAAsB,CAAC;aACpE,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;KAC5C,CAAC;AACJ,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,cAAc,CAAC,MAA+B;IAC5D,OAAO,MAAM,CAAC,IAAI,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,OAAO,KAAK,CAAC,QAAQ,KAAK,QAAQ,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;AACjG,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,aAAa,CAAC,GAAW;IACvC,OAAO,GAAG;SACP,KAAK,CAAC,OAAO,CAAC;SACd,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC;SAChC,MAAM,CAAC,OAAO,CAAC,CAAC;AACrB,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,cAAc,CAAC,KAAmB,EAAE,IAAY;IAC9D,OAAO;QACL,IAAI,EAAE,KAAK,CAAC,IAAI;QAChB,WAAW,EAAE,KAAK,CAAC,WAAW;QAC9B,QAAQ,EAAE,KAAK,CAAC,OAAO,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,EAAE;QACxE,IAAI;KACL,CAAC;AACJ,CAAC;AAED;;;;;;wBAMwB;AACxB,MAAM,UAAU,kBAAkB,CAAC,KAAiB;IAClD,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE;QAAE,OAAO,eAAe,CAAC;IAC/C,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE;QAAE,OAAO,eAAe,CAAC;IAC/C,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,qBAAqB,CAAC,IAAY;IAChD,IAAI,CAAC,IAAI;QAAE,OAAO,IAAI,CAAC;IACvB,OAAO,IAAI;SACR,KAAK,CAAC,GAAG,CAAC;SACV,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC,IAAI,CAAC,MAAM,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,GAAG,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;SACxF,IAAI,CAAC,GAAG,CAAC,CAAC;AACf,CAAC;AAED;;;;2BAI2B;AAC3B,MAAM,UAAU,iBAAiB,CAAC,IAAY;IAC5C,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAClC,OAAO,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;AAC/C,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,mBAAmB,CAAC,IAAY;IAC9C,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC;IACtD,OAAO,KAAK,GAAG,EAAE,CAAC;AACpB,CAAC;AAED;qCACqC;AACrC,MAAM,UAAU,mBAAmB,CAAC,KAAa;IAC/C,IAAI,KAAK,GAAG,IAAI;QAAE,OAAO,GAAG,KAAK,IAAI,CAAC;IACtC,IAAI,KAAK,GAAG,IAAI,GAAG,IAAI;QAAE,OAAO,GAAG,CAAC,KAAK,GAAG,IAAI,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC;IAClE,OAAO,GAAG,CAAC,KAAK,GAAG,CAAC,IAAI,GAAG,IAAI,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,KAAK,CAAC;AACpD,CAAC;AAED;;;;;;iBAMiB;AACjB,MAAM,UAAU,cAAc,CAAC,KAAmB;IAChD,OAAO,KAAK,CAAC,MAAM,KAAK,MAAM,CAAC;AACjC,CAAC;AAED;;kBAEkB;AAClB,MAAM,UAAU,gBAAgB,CAAC,KAAmB;IAClD,OAAO,KAAK,CAAC,MAAM,KAAK,MAAM,CAAC;AACjC,CAAC"}

package/dist/features/skills/types.d.ts

@@ -0,0 +1,97 @@
+/**
+ * Origin: `SkillsSection.tsx` (994 lines) plus `SkillDetailsModal.tsx`, and the
+ * `SkillSummary` contract they render (`packages/contracts/src/api/registry.ts`).
+ *
+ * What is generic and ported: a skill has an id, a name, a description, trigger
+ * phrases, a mode, an origin (shipped vs. authored here), and optional
+ * category/localization metadata. Filtering, searching, and draft editing over
+ * that shape are the tab's real work and none of it is product-bound.
+ *
+ * What is deliberately NOT ported: the origin's fixed `mode` union
+ * (`'prototype' | 'deck' | 'template' | 'design-system' | ...`) and its
+ * `surface`/`platform`/`scenario` fields. Those name that product's own output
+ * kinds — a "deck" or a "design-system" is not a concept every host has. `mode`
+ * is a plain string here and the host supplies whatever vocabulary it uses,
+ * exactly as `ProviderPreset` lets a host supply its own endpoint catalog.
+ */
+/** Where a skill came from. `'built-in'` ships with the host; `'user'` was
+ *  authored locally. Other values are allowed because a host may have more
+ *  origins (an org registry, a marketplace) — the filter row only ever offers
+ *  the two it knows, and counts the rest under "all". */
+export type SkillSource = 'user' | 'built-in' | (string & {});
+/** One skill as the list renders it. */
+export interface SkillSummary {
+    id: string;
+    name: string;
+    description: string;
+    /** Phrases that activate the skill. */
+    triggers?: readonly string[];
+    /** Free-form kind, host-defined — see this file's header. */
+    mode: string;
+    source: SkillSource;
+    /** Optional grouping slug. The category filter row renders only when at
+     *  least one listed skill carries one, so a host with none sees no empty
+     *  row rather than a dead control. */
+    category?: string | null;
+    /** Locale-keyed display name / description, when the host localizes its
+     *  catalog. Absent means "use `name`/`description` as written". */
+    displayName?: Readonly<Record<string, string>>;
+    descriptionI18n?: Readonly<Record<string, string>>;
+}
+/** One entry in a skill's file tree. */
+export interface SkillFileEntry {
+    path: string;
+    kind: 'file' | 'directory';
+    /** `null` for directories, and for hosts that do not report sizes. */
+    size: number | null;
+}
+/** A skill plus the body text the list does not carry. Fetched lazily per row
+ *  so the initial listing payload stays small. */
+export interface SkillDetail extends SkillSummary {
+    body: string;
+}
+/** The origin filter. `'all'` disables the filter rather than naming a
+ *  source, which is why it is part of this type and not a separate flag. */
+export type SourceFilter = 'all' | 'user' | 'built-in';
+/** The editable fields of a skill, as strings — `triggers` is the raw comma/
+ *  newline-separated text the operator typed, not a parsed array, so a
+ *  half-typed entry survives a re-render. `parseTriggers` converts on save. */
+export interface SkillDraft {
+    name: string;
+    description: string;
+    triggers: string;
+    body: string;
+}
+/** Everything narrowing the visible list. Grouped into one object so the pure
+ *  filter/count helpers take a single argument instead of four positional
+ *  ones that are easy to transpose. */
+export interface SkillFilters {
+    search: string;
+    source: SourceFilter;
+    /** `'all'` disables the filter. */
+    mode: string;
+    /** `'all'` disables the filter. */
+    category: string;
+}
+/** A filter option and how many skills it would leave visible. Counts are
+ *  computed with that option's OWN dimension excluded, so a pill always shows
+ *  what picking it would yield rather than what is showing now. */
+export type SkillFilterOption = readonly [value: string, count: number];
+/**
+ * Why the create/edit form can't submit right now, if at all. The client-side
+ * checks (`validateSkillDraft`'s two cases) and a failed
+ * `SkillsPort.createSkill`/`updateSkill` call collapse into one shape here —
+ * same idiom as `ConnectionTestState` in the execution tab (§3.2 of this
+ * package's error-reporting contract): reuse the "async edge result" shape
+ * rather than a separate validation-error type plus a separate raw-message
+ * string.
+ */
+export type SkillDraftError = {
+    kind: 'name-required';
+} | {
+    kind: 'body-required';
+} | {
+    kind: 'submit-failed';
+    message: string;
+};
+//# sourceMappingURL=types.d.ts.map

package/dist/features/skills/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../../src/features/skills/types.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH;;;yDAGyD;AACzD,MAAM,MAAM,WAAW,GAAG,MAAM,GAAG,UAAU,GAAG,CAAC,MAAM,GAAG,EAAE,CAAC,CAAC;AAE9D,wCAAwC;AACxC,MAAM,WAAW,YAAY;IAC3B,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,uCAAuC;IACvC,QAAQ,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC7B,6DAA6D;IAC7D,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,WAAW,CAAC;IACpB;;0CAEsC;IACtC,QAAQ,CAAC,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB;uEACmE;IACnE,WAAW,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC;IAC/C,eAAe,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC;CACpD;AAED,wCAAwC;AACxC,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,GAAG,WAAW,CAAC;IAC3B,sEAAsE;IACtE,IAAI,EAAE,MAAM,GAAG,IAAI,CAAC;CACrB;AAED;kDACkD;AAClD,MAAM,WAAW,WAAY,SAAQ,YAAY;IAC/C,IAAI,EAAE,MAAM,CAAC;CACd;AAED;4EAC4E;AAC5E,MAAM,MAAM,YAAY,GAAG,KAAK,GAAG,MAAM,GAAG,UAAU,CAAC;AAEvD;;+EAE+E;AAC/E,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,MAAM,CAAC;IACjB,IAAI,EAAE,MAAM,CAAC;CACd;AAED;;uCAEuC;AACvC,MAAM,WAAW,YAAY;IAC3B,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,YAAY,CAAC;IACrB,mCAAmC;IACnC,IAAI,EAAE,MAAM,CAAC;IACb,mCAAmC;IACnC,QAAQ,EAAE,MAAM,CAAC;CAClB;AAED;;mEAEmE;AACnE,MAAM,MAAM,iBAAiB,GAAG,SAAS,CAAC,KAAK,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,CAAC,CAAC;AAExE;;;;;;;;GAQG;AACH,MAAM,MAAM,eAAe,GACvB;IAAE,IAAI,EAAE,eAAe,CAAA;CAAE,GACzB;IAAE,IAAI,EAAE,eAAe,CAAA;CAAE,GACzB;IAAE,IAAI,EAAE,eAAe,CAAC;IAAC,OAAO,EAAE,MAAM,CAAA;CAAE,CAAC"}

package/dist/features/skills/types.js

@@ -0,0 +1,18 @@
+/**
+ * Origin: `SkillsSection.tsx` (994 lines) plus `SkillDetailsModal.tsx`, and the
+ * `SkillSummary` contract they render (`packages/contracts/src/api/registry.ts`).
+ *
+ * What is generic and ported: a skill has an id, a name, a description, trigger
+ * phrases, a mode, an origin (shipped vs. authored here), and optional
+ * category/localization metadata. Filtering, searching, and draft editing over
+ * that shape are the tab's real work and none of it is product-bound.
+ *
+ * What is deliberately NOT ported: the origin's fixed `mode` union
+ * (`'prototype' | 'deck' | 'template' | 'design-system' | ...`) and its
+ * `surface`/`platform`/`scenario` fields. Those name that product's own output
+ * kinds — a "deck" or a "design-system" is not a concept every host has. `mode`
+ * is a plain string here and the host supplies whatever vocabulary it uses,
+ * exactly as `ProviderPreset` lets a host supply its own endpoint catalog.
+ */
+export {};
+//# sourceMappingURL=types.js.map

package/dist/features/skills/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../../src/features/skills/types.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG"}

package/dist/features/source-config-list/agent-handles.d.ts

@@ -0,0 +1,121 @@
+/**
+ * @module agent-handles
+ *
+ * How this feature turns the ONE base handle a host passes as `agentHandle` into the
+ * `data-agent-*` markup for every sub-element it publishes.
+ *
+ * A host names a component (`agentHandle="mcp-add"`); the component names its own parts. That
+ * split is deliberate: only the component knows it has a "Show" toggle next to the password box,
+ * what verb applies to each control, and what each one is called — a caller supplying an
+ * id-per-element map would have to re-derive all of it, and re-derive it again on every internal
+ * change. See `@jini-ai/agentic`'s `element-handles.ts` for the markup convention itself.
+ *
+ * ## The scheme
+ *
+ * | element | handle |
+ * |---|---|
+ * | the component itself | `<base>` |
+ * | an action on it | `<base>-<action>` — e.g. `mcp-add-submit`, `mcp-server-1-remove` |
+ * | one host-described field | `<base>-field-<kebab-cased spec.key>` — e.g. `mcp-add-field-oauth-client-id` |
+ * | a control belonging to a field | `<field handle>-<action>` — e.g. `mcp-add-field-api-key-reveal` |
+ *
+ * Fields sit under their own `-field-` namespace rather than directly under the base so that a
+ * host whose source shape happens to contain a field keyed `remove` or `save` cannot collide with
+ * this feature's own action handles. A duplicate handle is not a cosmetic problem: it makes
+ * `page.fill`/`page.click` ambiguous for the very element a caller was most likely to want.
+ *
+ * ## The list-level scheme (`SourceConfigList`/`SourceConfigListView`)
+ *
+ * A list mounts one add form and N item cards from ONE base, the same "caller names one thing,
+ * this feature names its own parts" split as every card/form above:
+ *
+ * | element | handle |
+ * |---|---|
+ * | the add form | `<base>-add` (that base's own scheme then applies) |
+ * | one item card, keyed by the source's own stable id | `<base>-item-<slug of source.id>` (that base's own scheme then applies) |
+ *
+ * Item cards sit under `<base>-item-` rather than directly under `<base>` so that a source id
+ * cannot collide with the add form's `<base>-add` handle. See {@link sourceConfigItemHandles} for
+ * why cards are keyed by id rather than list position.
+ */
+import { type AgentElementRole, type AgentHandleProps } from '@jini-ai/agentic';
+/**
+ * The handle for one action or sub-control published under `base`.
+ *
+ * Exported because a nested component that publishes handles of its own (`SourceConfigField`,
+ * `SourceConfigTestControl`) is handed a base rather than props — its parent derives that base
+ * with this, and the child then derives its own parts from it the same way.
+ *
+ * @param base - The component's own handle, as published by the host. Must already be a valid
+ *   element handle — see {@link sanitizeHandleSegment} for why this one is not sanitized.
+ * @param action - The sub-element's name. Always a literal chosen by this feature, never host data.
+ * @returns `<base>-<action>`.
+ * @complexity O(1).
+ */
+export declare function sourceConfigActionHandle(base: string, action: string): string;
+/**
+ * The handle for one host-described field rendered under `base`.
+ *
+ * @param base - The component's own handle, as published by the host.
+ * @param fieldKey - The `SourceFieldSpec.key` this field renders. Host data, so it is sanitized.
+ * @returns `<base>-field-<kebab-cased key>`.
+ * @complexity O(n) in `fieldKey.length`.
+ */
+export declare function sourceConfigFieldHandle(base: string, fieldKey: string): string;
+/**
+ * The add form's own base, derived from a list's base — see this module's "list-level scheme" doc.
+ *
+ * @param listBase - The list's own handle, as published by the host to `SourceConfigList`/`SourceConfigListView`.
+ * @returns `<listBase>-add`.
+ * @complexity O(1).
+ */
+export declare function sourceConfigAddFormHandle(listBase: string): string;
+/**
+ * One distinct, stable base handle per rendered item card, positionally aligned with `sourceIds` —
+ * what `SourceConfigListView` hands each `SourceConfigItemCard` as its own `agentHandle`.
+ *
+ * Derived from each source's own stable id, never its position in the list: an agent reading
+ * `page.find_elements` should keep seeing the same card handle for the same source after a
+ * refresh reorders or filters the list. Delegates the actual slugify-and-deduplicate mechanics to
+ * `@jini-ai/agentic`'s {@link buildAgentListHandles} — see that function's own doc for why
+ * uniqueness needs a suffix search rather than a single `-<index>` append.
+ *
+ * @param listBase - The list's own handle, as published by the host.
+ * @param sourceIds - The rendered sources' own stable ids, in render order.
+ * @returns One base handle per id, positionally aligned with `sourceIds`, all distinct, and none
+ *   colliding with {@link sourceConfigAddFormHandle}'s `<listBase>-add` (see the `-item-`
+ *   namespace in this module's doc).
+ * @complexity See {@link buildAgentListHandles}.
+ */
+export declare function sourceConfigItemHandles(listBase: string, sourceIds: readonly string[]): string[];
+/** What to publish about one sub-element — see {@link sourceConfigAgentProps}. */
+export interface SourceConfigAgentPropsOptions {
+    /** Which verb applies to this element. */
+    readonly role: AgentElementRole;
+    /**
+     * Stable ontology for this element — what the control IS, never what it currently says. A
+     * reveal toggle is labelled by the field it reveals, not by the live "Show"/"Hide" on its face,
+     * so a caller can address the same element across the whole session.
+     */
+    readonly label: string;
+    /**
+     * The sub-element's name, appended to `base`. Omit when the element IS `base` — the component's
+     * own root.
+     */
+    readonly action?: string;
+}
+/**
+ * Builds the `data-agent-*` attribute props for one sub-element, or nothing at all when the host
+ * published no base handle.
+ *
+ * Returning `{}` rather than throwing is what keeps `agentHandle` an opt-in prop: a host with no
+ * agent-control surface renders exactly the markup it did before this existed.
+ *
+ * @param base - The component's own handle from the host, or `undefined` when it published none.
+ * @param options - Role, stable label, and the optional action naming this sub-element.
+ * @returns Spreadable attribute props, or `{}` when `base` is `undefined`.
+ * @throws If `base` is not a valid element handle — via `agentHandle()`, deliberately unguarded.
+ * @complexity O(1).
+ */
+export declare function sourceConfigAgentProps(base: string | undefined, options: SourceConfigAgentPropsOptions): AgentHandleProps | Record<string, never>;
+//# sourceMappingURL=agent-handles.d.ts.map

package/dist/features/source-config-list/agent-handles.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"agent-handles.d.ts","sourceRoot":"","sources":["../../../src/features/source-config-list/agent-handles.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GAuCG;AACH,OAAO,EAAsC,KAAK,gBAAgB,EAAE,KAAK,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AAwCpH;;;;;;;;;;;;GAYG;AACH,wBAAgB,wBAAwB,CAAC,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE7E;AAED;;;;;;;GAOG;AACH,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,GAAG,MAAM,CAK9E;AAED;;;;;;GAMG;AACH,wBAAgB,yBAAyB,CAAC,QAAQ,EAAE,MAAM,GAAG,MAAM,CAElE;AAED;;;;;;;;;;;;;;;;GAgBG;AACH,wBAAgB,uBAAuB,CAAC,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,SAAS,MAAM,EAAE,GAAG,MAAM,EAAE,CAEhG;AAED,kFAAkF;AAClF,MAAM,WAAW,6BAA6B;IAC5C,0CAA0C;IAC1C,QAAQ,CAAC,IAAI,EAAE,gBAAgB,CAAC;IAChC;;;;OAIG;IACH,QAAQ,CAAC,KAAK,EAAE,MAAM,CAAC;IACvB;;;OAGG;IACH,QAAQ,CAAC,MAAM,CAAC,EAAE,MAAM,CAAC;CAC1B;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,sBAAsB,CACpC,IAAI,EAAE,MAAM,GAAG,SAAS,EACxB,OAAO,EAAE,6BAA6B,GACrC,gBAAgB,GAAG,MAAM,CAAC,MAAM,EAAE,KAAK,CAAC,CAI1C"}