@jskit-ai/assistant 0.1.42 → 0.1.44

package/package.descriptor.mjs

@@ -1,7 +1,7 @@
 export default Object.freeze({
   packageVersion: 1,
   packageId: "@jskit-ai/assistant",
-  version: "0.1.42",
+  version: "0.1.44",
   kind: "generator",
   description: "Install assistant runtime/config for one surface and scaffold assistant pages at explicit target files.",
   options: {
@@ -49,7 +49,7 @@ export default Object.freeze({
       inputType: "text",
       defaultValue: "",
       promptLabel: "Link placement",
-      promptHint: "Optional host:position target for the generated page link placement."
+      promptHint: "Optional target for the generated page link placement (format: host:position)."
     },
     "link-component-token": {
       required: false,
@@ -158,6 +158,7 @@ export default Object.freeze({
         ]
       },
       page: {
+        requiresShellWeb: true,
         entrypoint: "src/server/subcommands/page.js",
         export: "runGeneratorSubcommand",
         description: "Create an assistant runtime page at an explicit target file relative to src/pages/.",
@@ -194,6 +195,7 @@ export default Object.freeze({
         ]
       },
       "settings-page": {
+        requiresShellWeb: true,
         entrypoint: "src/server/subcommands/settingsPage.js",
         export: "runGeneratorSubcommand",
         description: "Create an assistant settings page at an explicit target file relative to src/pages/.",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jskit-ai/assistant",
-  "version": "0.1.42",
+  "version": "0.1.44",
   "type": "module",
   "scripts": {
     "test": "node --test"
@@ -9,6 +9,6 @@
     "./server/buildTemplateContext": "./src/server/buildTemplateContext.js"
   },
   "dependencies": {
-    "@jskit-ai/kernel": "0.1.33"
+    "@jskit-ai/kernel": "0.1.35"
   }
 }

package/src/server/pageSupport.js

@@ -56,11 +56,11 @@ async function resolveAssistantPageGenerationContext({
   return Object.freeze({
     pageTarget,
     pageLabel: normalizeText(options?.name) || pageTarget.defaultName,
-    linkPlacementHost: String(linkTarget.placementTarget?.host || ""),
-    linkPlacementPosition: String(linkTarget.placementTarget?.position || ""),
+    linkPlacementTarget: String(linkTarget.placementTarget?.id || ""),
     linkComponentToken: String(linkTarget.componentToken || ""),
     linkWorkspaceSuffix: pageTarget.routeUrlSuffix,
     linkNonWorkspaceSuffix: pageTarget.routeUrlSuffix,
+    linkWhenLine: String(linkTarget.whenLine || ""),
     linkToPropLine: resolveLinkToPropLine(linkTarget.linkTo)
   });
 }
@@ -75,8 +75,7 @@ function renderAssistantPageLinkPlacementBlock({
     "{\n" +
     "  addPlacement({\n" +
     `    id: "${String(pageTarget?.placementId || "")}",\n` +
-    `    host: "${String(generationContext?.linkPlacementHost || "")}",\n` +
-    `    position: "${String(generationContext?.linkPlacementPosition || "")}",\n` +
+    `    target: "${String(generationContext?.linkPlacementTarget || "")}",\n` +
     `    surfaces: ["${String(pageTarget?.surfaceId || "")}"],\n` +
     "    order: 155,\n" +
     `    componentToken: "${String(generationContext?.linkComponentToken || "")}",\n` +
@@ -86,7 +85,7 @@ function renderAssistantPageLinkPlacementBlock({
     `      workspaceSuffix: "${String(generationContext?.linkWorkspaceSuffix || "")}",\n` +
     `      nonWorkspaceSuffix: "${String(generationContext?.linkNonWorkspaceSuffix || "")}",\n` +
     `${String(generationContext?.linkToPropLine || "")}    },\n` +
-    "    when: ({ auth }) => Boolean(auth?.authenticated)\n" +
+    `${String(generationContext?.linkWhenLine || "")}` +
     "  });\n" +
     "}\n"
   );

package/test/subcommands.test.js

@@ -25,11 +25,12 @@ function toPagePath(targetFile = "") {
   return path.join("src/pages", targetFile);
 }
 
-async function writeAppFixture(appRoot) {
+async function writeAppFixture(appRoot, { configSource = "" } = {}) {
   await writeFileInApp(
     appRoot,
     "config/public.js",
-    `export const config = {
+    configSource ||
+      `export const config = {
   surfaceDefinitions: {
     admin: {
       id: "admin",
@@ -55,8 +56,8 @@ async function writeAppFixture(appRoot) {
     "src/components/ShellLayout.vue",
     `<template>
   <div>
-    <ShellOutlet host="shell-layout" position="primary-menu" default />
-    <ShellOutlet host="shell-layout" position="top-right" />
+    <ShellOutlet target="shell-layout:primary-menu" default />
+    <ShellOutlet target="shell-layout:top-right" />
   </div>
 </template>
 `
@@ -95,8 +96,7 @@ test("assistant page subcommand creates a runtime page at an explicit target fil
     const placementSource = await readFile(path.join(appRoot, "src/placement.js"), "utf8");
     assert.match(placementSource, /jskit:assistant\.page\.link:admin:\/ops\/copilot/);
     assert.match(placementSource, /id: "ui-generator\.page\.admin\.ops\.copilot\.link"/);
-    assert.match(placementSource, /host: "shell-layout"/);
-    assert.match(placementSource, /position: "primary-menu"/);
+    assert.match(placementSource, /target: "shell-layout:primary-menu"/);
     assert.match(placementSource, /label: "Copilot"/);
     assert.match(placementSource, /workspaceSuffix: "\/ops\/copilot"/);
   });
@@ -111,7 +111,7 @@ test("assistant settings-page subcommand uses the target assistant surface and i
       `<template>
   <SectionContainerShell>
     <template #tabs>
-      <ShellOutlet host="admin-settings" position="sub-pages" />
+      <ShellOutlet target="admin-settings:sub-pages" />
     </template>
     <RouterView />
   </SectionContainerShell>
@@ -137,14 +137,48 @@ test("assistant settings-page subcommand uses the target assistant surface and i
 
     const placementSource = await readFile(path.join(appRoot, "src/placement.js"), "utf8");
     assert.match(placementSource, /jskit:assistant\.settings-page\.link:admin:\/settings\/assistant:console/);
-    assert.match(placementSource, /host: "admin-settings"/);
-    assert.match(placementSource, /position: "sub-pages"/);
+    assert.match(placementSource, /target: "admin-settings:sub-pages"/);
     assert.match(placementSource, /componentToken: "local\.main\.ui\.tab-link-item"/);
     assert.match(placementSource, /to: "\.\/assistant"/);
     assert.match(placementSource, /label: "Assistant"/);
   });
 });
 
+test("assistant page subcommand omits the auth guard for a public surface link", async () => {
+  await withTempApp(async (appRoot) => {
+    await writeAppFixture(appRoot, {
+      configSource: `export const config = {
+  surfaceAccessPolicies: {
+    public: {}
+  },
+  surfaceDefinitions: {
+    home: {
+      id: "home",
+      pagesRoot: "home",
+      enabled: true,
+      requiresWorkspace: false,
+      accessPolicyId: "public"
+    }
+  },
+  assistantSurfaces: {}
+};
+`
+    });
+
+    const targetFile = "home/assistant/index.vue";
+    await runPageSubcommand({
+      appRoot,
+      subcommand: "page",
+      args: [targetFile],
+      options: {}
+    });
+
+    const placementSource = await readFile(path.join(appRoot, "src", "placement.js"), "utf8");
+    assert.match(placementSource, /id: "ui-generator\.page\.home\.assistant\.link"/);
+    assert.doesNotMatch(placementSource, /when: \(\{ auth \}\) => Boolean\(auth\?\.authenticated\)/);
+  });
+});
+
 test("assistant page subcommand refuses to overwrite an existing user-owned page", async () => {
   await withTempApp(async (appRoot) => {
     await writeAppFixture(appRoot);
@@ -213,7 +247,7 @@ test("assistant settings-page subcommand overwrites an existing page when --forc
       `<template>
   <SectionContainerShell>
     <template #tabs>
-      <ShellOutlet host="admin-settings" position="sub-pages" />
+      <ShellOutlet target="admin-settings:sub-pages" />
     </template>
     <RouterView />
   </SectionContainerShell>
@@ -267,19 +301,20 @@ test("assistant settings-page subcommand requires the target assistant surface o
   });
 });
 
-test("assistant page subcommand rejects target files with a src/pages prefix", async () => {
+test("assistant page subcommand accepts target files with a src/pages prefix", async () => {
   await withTempApp(async (appRoot) => {
     await writeAppFixture(appRoot);
 
-    await assert.rejects(
-      () =>
-        runPageSubcommand({
-          appRoot,
-          subcommand: "page",
-          args: ["src/pages/w/[workspaceSlug]/admin/assistant/index.vue"],
-          options: {}
-        }),
-      /must be relative to src\/pages\/, without the src\/pages\/ prefix/
-    );
+    const targetFile = "src/pages/w/[workspaceSlug]/admin/assistant/index.vue";
+    const result = await runPageSubcommand({
+      appRoot,
+      subcommand: "page",
+      args: [targetFile],
+      options: {}
+    });
+
+    assert.deepEqual(result.touchedFiles, [targetFile, "src/placement.js"]);
+    const pageSource = await readFile(path.join(appRoot, targetFile), "utf8");
+    assert.match(pageSource, /<AssistantSurfaceClientElement surface-id="admin" \/>/);
   });
 });

package/README.md

@@ -1,91 +0,0 @@
-# @jskit-ai/assistant
-
-Install assistant runtime/config for one surface, then scaffold assistant pages at explicit page files.
-
-## Mental Model
-
-`@jskit-ai/assistant` is the generator package.
-
-- `setup` installs the assistant runtime/config/env wiring.
-- `page <target-file>` creates the assistant runtime page anywhere relative to `src/pages/...`.
-- `settings-page <target-file>` creates the assistant settings page anywhere relative to `src/pages/...`.
-
-There is no separate server command and no separate client command.
-
-The layering is:
-
-- `@jskit-ai/assistant`: generator package with `setup`, `page`, and `settings-page`
-- `@jskit-ai/assistant-runtime`: runtime package installed by `setup`
-- `@jskit-ai/assistant-core`: lower-level provider/client library pulled in by `assistant-runtime`
-
-So the normal flow is:
-
-- run `setup`
-- optionally create `page`
-- optionally create `settings-page`
-
-Page placement follows the same file-driven model as `@jskit-ai/ui-generator`.
-
-- The surface comes from where the page file lives.
-- The route URL comes from that file path.
-- The default page-link placement comes from the nearest real parent subpages host when there is one.
-
-Only one assistant can be installed per target surface. That rule is enforced by `setup`, which writes the per-surface config entries.
-
-## Setup
-
-```bash
-npx jskit generate @jskit-ai/assistant setup \
-  --surface admin \
-  --settings-surface console \
-  --config-scope global \
-  --ai-provider openai \
-  --ai-api-key "$OPENAI_API_KEY" \
-  --ai-base-url "" \
-  --ai-timeout-ms 120000
-```
-
-`setup` does not create pages.
-
-It only registers:
-
-- `config.assistantSurfaces.<surface>`
-- `config.assistantServer.<surface>`
-- prefixed AI env defaults in `.env`
-- the `@jskit-ai/assistant-runtime` dependency chain, including `@jskit-ai/assistant-core`
-
-## Runtime Page
-
-```bash
-npx jskit generate @jskit-ai/assistant page \
-  admin/ops/copilot/index.vue
-```
-
-The page surface id comes from the target file path.
-
-Optional page-link overrides:
-
-- `--name`
-- `--link-placement`
-- `--link-component-token`
-- `--link-to`
-
-## Settings Page
-
-```bash
-npx jskit generate @jskit-ai/assistant settings-page \
-  admin/settings/index/assistant/index.vue \
-  --surface admin
-```
-
-For `settings-page`, the page location and the assistant target surface are separate concerns:
-
-- `<target-file>` decides where the settings page lives and how its page link is placed
-- `--surface` decides which assistant surface the settings page edits
-
-The same optional page-link overrides are supported:
-
-- `--name`
-- `--link-placement`
-- `--link-component-token`
-- `--link-to`