@cremini/skillpack 1.0.8 → 1.0.9

package/README.md

@@ -1,8 +1,10 @@
 # SkillPack.sh - Pack AI Skills into Standalone Apps
 
+Skillpack by Cremini is built on the idea of distributed intelligence, much like cremini mushrooms that grow from a vast, interconnected mycelial network.
+
 Go to [skillpack.sh](https://skillpack.sh) to pack skills and try existing skill packs.
 
-One command to orchestrate [Skills](https://skills.sh), tools, mcps into a standalone app users can download and use on their own computer!
+One command to orchestrate [Skills](https://skills.sh), tools, mcps into a standalone app users can download and use on their own computer to address a well-defined problem or complete specific tasks!
 
 ```bash
 npx @cremini/skillpack create

package/dist/cli.js

@@ -450,6 +450,25 @@ async function bundle(workDir) {
 function parseSkillNames(value) {
   return value.split(",").map((name) => name.trim()).filter(Boolean);
 }
+function normalizeSourceInput(value) {
+  return value.trim().replace(/^npx\s+skills\s+add\s+/u, "");
+}
+function parseSourceInput(value) {
+  const trimmedValue = normalizeSourceInput(value);
+  const skillFlagIndex = trimmedValue.indexOf(" --skill ");
+  if (skillFlagIndex === -1) {
+    return {
+      source: trimmedValue,
+      inlineSkillNames: []
+    };
+  }
+  const source = trimmedValue.slice(0, skillFlagIndex).trim();
+  const inlineSkillValue = trimmedValue.slice(skillFlagIndex + " --skill ".length).trim();
+  return {
+    source,
+    inlineSkillNames: inlineSkillValue.split(/[,\s]+/).map((name) => name.trim()).filter(Boolean)
+  };
+}
 async function createCommand(directory) {
   const workDir = directory ? path5.resolve(directory) : process.cwd();
   if (directory) {
@@ -493,7 +512,10 @@ async function createCommand(directory) {
     chalk3.dim("  Supported formats: owner/repo, GitHub URL, or local path")
   );
   console.log(chalk3.dim("  Example source: vercel-labs/agent-skills"));
-  console.log(chalk3.dim("  Example skill names: frontend-design, skill-creator\n"));
+  console.log(
+    chalk3.dim("  Example inline skill: vercel-labs/agent-skills --skill find-skills")
+  );
+  console.log();
   while (true) {
     const { source } = await inquirer.prompt([
       {
@@ -505,16 +527,24 @@ async function createCommand(directory) {
     if (!source.trim()) {
       break;
     }
-    const { skillNames } = await inquirer.prompt([
-      {
-        type: "input",
-        name: "skillNames",
-        message: "Skill names (comma-separated):",
-        validate: (value) => parseSkillNames(value).length > 0 ? true : "Enter at least one skill name"
-      }
-    ]);
-    const nextSkills = parseSkillNames(skillNames).map((skillName) => ({
-      source: source.trim(),
+    const parsedSource = parseSourceInput(source);
+    let skillNames = parsedSource.inlineSkillNames;
+    if (skillNames.length === 0) {
+      console.log(
+        chalk3.dim("  Example skill names: frontend-design, skill-creator")
+      );
+      const promptResult = await inquirer.prompt([
+        {
+          type: "input",
+          name: "skillNames",
+          message: "Skill names (comma-separated):",
+          validate: (value) => parseSkillNames(value).length > 0 ? true : "Enter at least one skill name"
+        }
+      ]);
+      skillNames = parseSkillNames(promptResult.skillNames);
+    }
+    const nextSkills = skillNames.map((skillName) => ({
+      source: parsedSource.source,
       name: skillName,
       description: ""
     }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cremini/skillpack",
-  "version": "1.0.8",
+  "version": "1.0.9",
   "description": "Turn Skills into a Standalone App with UI",
   "type": "module",
   "repository": {

package/runtime/server/chat-proxy.js

@@ -12,6 +12,34 @@ const DEBUG = true;
 const log = (...args) => DEBUG && console.log(...args);
 const write = (data) => DEBUG && process.stdout.write(data);
 
+function getAssistantDiagnostics(message) {
+  if (!message || message.role !== "assistant") {
+    return null;
+  }
+
+  const stopReason = message.stopReason;
+  const errorMessage =
+    message.errorMessage ||
+    (stopReason === "error" || stopReason === "aborted"
+      ? `Request ${stopReason}`
+      : "");
+
+  const content = Array.isArray(message.content) ? message.content : [];
+  const text = content
+    .filter((item) => item?.type === "text")
+    .map((item) => item.text || "")
+    .join("")
+    .trim();
+  const toolCalls = content.filter((item) => item?.type === "toolCall").length;
+
+  return {
+    stopReason,
+    errorMessage,
+    hasText: text.length > 0,
+    toolCalls,
+  };
+}
+
 /**
  * Handle incoming WebSocket connection using pi-coding-agent
  * @param {import("ws").WebSocket} ws
@@ -24,6 +52,8 @@ export async function handleWsConnection(
   { apiKey, rootDir, provider = "openai", modelId = "gpt-5.4" },
 ) {
   try {
+    let turnHadVisibleOutput = false;
+
     // Create an in-memory auth storage to avoid touching disk
     const authStorage = AuthStorage.inMemory({
       [provider]: { type: "api_key", key: apiKey },
@@ -78,6 +108,7 @@ export async function handleWsConnection(
 
         case "message_update":
           if (event.assistantMessageEvent?.type === "text_delta") {
+            turnHadVisibleOutput = true;
             write(event.assistantMessageEvent.delta);
             ws.send(
               JSON.stringify({
@@ -86,6 +117,7 @@ export async function handleWsConnection(
               }),
             );
           } else if (event.assistantMessageEvent?.type === "thinking_delta") {
+            turnHadVisibleOutput = true;
             ws.send(
               JSON.stringify({
                 type: "thinking_delta",
@@ -97,6 +129,17 @@ export async function handleWsConnection(
 
         case "message_end":
           log(`\n--- [Message End: ${event.message?.role}] ---`);
+          if (event.message?.role === "assistant") {
+            const diagnostics = getAssistantDiagnostics(event.message);
+            if (diagnostics) {
+              log(
+                `[Assistant Diagnostics] stopReason=${diagnostics.stopReason || "unknown"} text=${diagnostics.hasText ? "yes" : "no"} toolCalls=${diagnostics.toolCalls}`,
+              );
+              if (diagnostics.errorMessage) {
+                log(`[Assistant Error] ${diagnostics.errorMessage}`);
+              }
+            }
+          }
           ws.send(
             JSON.stringify({
               type: "message_end",
@@ -106,6 +149,7 @@ export async function handleWsConnection(
           break;
 
         case "tool_execution_start":
+          turnHadVisibleOutput = true;
           log(`\n>>> [Tool Execution Start: ${event.toolName}] >>>`);
           log("Args:", JSON.stringify(event.args, null, 2));
           ws.send(
@@ -118,6 +162,7 @@ export async function handleWsConnection(
           break;
 
         case "tool_execution_end":
+          turnHadVisibleOutput = true;
           log(`<<< [Tool Execution End: ${event.toolName}] <<<`);
           log(`Error: ${event.isError ? "Yes" : "No"}`);
           ws.send(
@@ -142,8 +187,31 @@ export async function handleWsConnection(
       try {
         const payload = JSON.parse(data.toString());
         if (payload.text) {
+          turnHadVisibleOutput = false;
+
           // Send prompt to the agent, the session will handle message history natively
           await session.prompt(payload.text);
+
+          const lastMessage = session.state.messages.at(-1);
+          const diagnostics = getAssistantDiagnostics(lastMessage);
+          if (diagnostics?.errorMessage) {
+            ws.send(JSON.stringify({ error: diagnostics.errorMessage }));
+            return;
+          }
+
+          if (
+            diagnostics &&
+            !diagnostics.hasText &&
+            diagnostics.toolCalls === 0 &&
+            !turnHadVisibleOutput
+          ) {
+            const emptyResponseError =
+              "Assistant returned no visible output. Check the server logs for stopReason/provider details.";
+            log(`[Assistant Warning] ${emptyResponseError}`);
+            ws.send(JSON.stringify({ error: emptyResponseError }));
+            return;
+          }
+
           ws.send(JSON.stringify({ done: true }));
         }
       } catch (err) {

package/runtime/web/app.js

@@ -181,7 +181,7 @@ function renderMarkdown(mdText, { renderEmbeddedMarkdown = true } = {}) {
     return escapeHtml(mdText);
   }
 
-  const html = marked.parse(mdText);
+  const html = ensureLinksOpenInNewTab(marked.parse(mdText));
   if (!renderEmbeddedMarkdown) {
     return html;
   }
@@ -189,6 +189,18 @@ function renderMarkdown(mdText, { renderEmbeddedMarkdown = true } = {}) {
   return renderEmbeddedMarkdownBlocks(html);
 }
 
+function ensureLinksOpenInNewTab(html) {
+  const template = document.createElement("template");
+  template.innerHTML = html;
+
+  template.content.querySelectorAll("a[href]").forEach((linkEl) => {
+    linkEl.setAttribute("target", "_blank");
+    linkEl.setAttribute("rel", "noopener noreferrer");
+  });
+
+  return template.innerHTML;
+}
+
 function renderEmbeddedMarkdownBlocks(html) {
   const template = document.createElement("template");
   template.innerHTML = html;
@@ -365,7 +377,9 @@ function handleAgentEvent(event) {
 
       let inputHtml = "";
       if (typeof marked !== "undefined") {
-        inputHtml = marked.parse("```json\n" + safeInput + "\n```");
+        inputHtml = ensureLinksOpenInNewTab(
+          marked.parse("```json\n" + safeInput + "\n```"),
+        );
       } else {
         inputHtml = escapeHtml(safeInput);
       }
@@ -438,7 +452,7 @@ function handleAgentEvent(event) {
             : "```json\n" + safeResult + "\n```";
 
         if (typeof marked !== "undefined") {
-          resultEl.innerHTML = marked.parse(mdText);
+          resultEl.innerHTML = ensureLinksOpenInNewTab(marked.parse(mdText));
         } else {
           resultEl.textContent = safeResult;
         }