@chatpanel/events 0.78.0 → 0.78.2

package/index.js

@@ -192,7 +192,7 @@ export { recipeToolProvider, recipeToolSpec, describeRecipeForApproval, RECIPE_T
 // Agent teams (F8): a team is data, a run is a turn of turns under a budget, members talk
 // through a typed board, nothing lands without a person.
 export { createBudget, validateBudget, normalizeBudget, usageOf, BudgetError, BUDGET_DIMENSIONS } from './budget.js';
-export { defineTeam, validateTeam, normalizeTeam, normalizeGrants, grantAllows, describeRole, TeamError, ROLE_MODES, MERGE_POLICIES, PLAN_MODES, GRANTABLE, STARTER_TEAMS, starterTeams, blankTeam, teamFromForm } from './team.js';
+export { defineTeam, validateTeam, normalizeTeam, normalizeGrants, grantAllows, describeRole, TeamError, ROLE_MODES, MERGE_POLICIES, PLAN_MODES, GRANTABLE, STARTER_TEAMS, starterTeams, blankTeam, teamFromForm, slugTeamName } from './team.js';
 export { fixedPlan, parsePlan, plannerPrompt, waves, breakCycles, TEAM_PLAN_SCHEMA } from './team-plan.js';
 export { createBoard, parseFindings, boardText, findingsInstruction, toBriefClaims, FINDINGS_SCHEMA, FINDING_KINDS } from './team-board.js';
 export { runTeam, dryRunTeam, TeamRunError, RUN_STATUSES } from './team-run.js';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chatpanel/events",
-  "version": "0.78.0",
+  "version": "0.78.2",
   "description": "The canonical ChatPanel event-log and capability contracts \u2014 typed durable facts, clock-free deterministic linearization, schema upcasting, and the invariants the replay harness asserts. Pure, dependency-free ESM shared by the ChatPanel extension, gateway and bridge.",
   "type": "module",
   "main": "index.js",

package/team-tool.js

@@ -11,7 +11,7 @@
 // receive the host's toolset narrowed to their grants, and the host builds that from the
 // same providers this tool is a member of.
 
-import { validateTeam, normalizeTeam, describeRole, TEAM_NAME_RE } from './team.js';
+import { validateTeam, normalizeTeam, describeRole, slugTeamName } from './team.js';
 import { dryRunTeam } from './team-run.js';
 
 export const TEAM_TOOL_NAME = 'team';
@@ -31,7 +31,7 @@ export function teamToolSpec(teams) {
       + 'Actions: {"action":"run","name":"<team>","request":"<what to do>"} runs one (streams; may take a while); '
       + '{"action":"dry_run","name":"<team>","request":"…"} shows roles, models, tools and budget without running; '
       + '{"action":"save","team":{…}} proposes a NEW team after a task that would benefit from several roles — the user approves it on a card. '
-      + 'A team: {"name":"research","description":"…","roles":[{"id":"researcher","prompt":"…","prefer":"balanced","grants":["data","web"]},{"id":"writer","prompt":"…","prefer":"strong","grants":["none"]}],"merge":"judge","judge":"writer","budget":{"tokens":40000,"ms":300000}}. '
+      + 'A team: {"name":"research" (a short identifier: letters, digits, - _; used as /research),"description":"…","roles":[{"id":"researcher","prompt":"…","prefer":"balanced","grants":["data","web"]},{"id":"writer","prompt":"…","prefer":"strong","grants":["none"]}],"merge":"judge","judge":"writer","budget":{"tokens":40000,"ms":300000}}. '
       + 'grants: none | data | web | history | mcp | mcp:<server>. merge: judge | converge | concat | first. A budget is required.',
     parameters: {
       type: 'object',
@@ -71,17 +71,20 @@ export function teamToolProvider({ teams = [], run = null, appoint = null, confi
   return {
     id: 'team',
     specs: [teamToolSpec(teams)],
-    system: byName.size ? 'Saved agent teams exist (see the `team` tool). When a request is broad enough that several roles would do it better — research plus writing, several sources to reconcile — run the matching team rather than doing it all in one turn.' : '',
+    system: [
+      byName.size ? 'Saved agent teams exist (see the `team` tool). When a request is broad enough that several roles would do it better — research plus writing, several sources to reconcile — run the matching team rather than doing it all in one turn.' : '',
+      (confirmSave && saveTeam) ? 'When the user asks to create, make, set up or save a team (of agents / roles), do not run one: call the `team` tool with {"action":"save","team":{…}} — pick roles, prompts, grants and a budget from what they said, and ask only for what you cannot infer. The user approves it on a card.' : '',
+    ].filter(Boolean).join(' '),
     bind(toolset) { bound = toolset; },
     async execute(name, input) {
       if (name !== TEAM_TOOL_NAME) return json({ error: `Unknown tool: ${name}` });
       const action = String(input?.action || '');
 
       if (action === 'save') {
-        const team = input?.team;
+        // A name in prose ("Research Team") becomes the /command it will be run by.
+        const team = input?.team && typeof input.team === 'object' ? { ...input.team, name: slugTeamName(input.team.name) } : input?.team;
         const v = validateTeam(team);
-        if (!v.ok) return json({ error: 'The team is not valid.', problems: v.errors, hint: 'A team needs a name, roles with prompts and grants, and a budget.' });
-        if (!TEAM_NAME_RE.test(team.name)) return json({ error: 'name must be a short identifier.' });
+        if (!v.ok) return json({ error: 'The team is not valid.', problems: v.errors, hint: 'A team needs a name (letters, digits, - _), roles with prompts and grants, and a budget such as {"tokens":40000,"ms":300000}.' });
         if (byName.has(team.name)) return json({ error: `A team named "${team.name}" already exists. Pick another name.` });
         if (!confirmSave || !saveTeam) return json({ error: 'Saving a team needs the user\'s approval, which this surface cannot ask for. Describe the team and suggest saving it from the side panel or the desktop.' });
         const norm = normalizeTeam(team);

package/team.js

@@ -152,6 +152,15 @@ export const STARTER_TEAMS = Object.freeze([
   },
 ]);
 
+/**
+ * A name as typed → the identifier a `/command` needs: "Research Team" → "research-team".
+ * Used by the tool's save and by the form, so a model that names a team in prose is not
+ * bounced for it; what cannot be shaped (nothing left) still fails validation.
+ */
+export function slugTeamName(name) {
+  return String(name || '').trim().toLowerCase().replace(/[^a-z0-9_-]+/g, '-').replace(/^[^a-z]+/, '').replace(/-+$/, '').slice(0, 64);
+}
+
 /** Fresh copies — a starter is a template, never the stored record. */
 export function starterTeams() {
   return STARTER_TEAMS.map((t) => ({ ...t, roles: t.roles.map((r) => ({ ...r, grants: [...r.grants] })), budget: { ...t.budget } }));
@@ -183,7 +192,7 @@ export function teamFromForm(form) {
   }
   const merge = form.merge || 'concat';
   const team = {
-    name: String(form.name || '').trim(),
+    name: slugTeamName(form.name),
     description: String(form.description || '').trim(),
     plan: form.plan || 'fixed',
     merge,