@awebai/oats 0.37.0 → 0.38.0

package/lib/workspace.mjs

@@ -18,7 +18,8 @@ import { oatsError } from "./errors.mjs";
 import * as defaultRemote from "./remote.mjs";
 import { bindRemote, classifyPackageValue, lockedPackageRef } from "./packages.mjs";
 import { manifestContractProblems } from "./capability-contract.mjs";
-import { BY_TEAM_REMOVED, TEAM_MEMBERSHIP_MOVED } from "./teams.mjs";
+import YAML from "yaml";
+import { BY_TEAM_REMOVED, TEAM_MEMBERSHIP_MOVED, soulsReplacement } from "./teams.mjs";
 
 /* ───────────────────────────── errors ─────────────────────────────────── */
 
@@ -261,7 +262,7 @@ function sharedLabelProblems(value, problems) {
 }
 export function validateMembership(value) { return withRemovedKeys(validateAgainst(schemaFor("membership"), value), value, REMOVED_KEYS.membership); }
 
-/** Keys 0.30 removed (team model v2): each is a schema problem naming its replacement, in place of
+/** Keys 0.30 (team model v2) and 0.38.0 (team model 3) removed: each is a schema problem naming its replacement, in place of
  *  the schema's generic "unexpected property". `at(value)` → the JSON pointers where it appears. */
 const REMOVED_KEYS = {
   workspace: [
@@ -270,6 +271,11 @@ const REMOVED_KEYS = {
   ],
   membership: [{ at: (v) => (isObject(v) && Object.hasOwn(v, "team") ? ["/team"] : []), message: TEAM_MEMBERSHIP_MOVED }],
   soul: [{ at: (v) => (isObject(v) && Object.hasOwn(v, "team") ? ["/team"] : []), message: TEAM_MEMBERSHIP_MOVED }],
+  // 0.38.0 (team model 3): a soul's teams and default are committed in the workspace's souls:.
+  local: [
+    { at: (v) => (isObject(v?.souls) && Object.hasOwn(v.souls, "teams") ? ["/souls/teams"] : []), message: "souls.teams was removed in 0.38.0 (team model 3): which teams a soul may join is souls: in oats-workspace.yaml" },
+    { at: (v) => (isObject(v?.souls) && Object.hasOwn(v.souls, "default") ? ["/souls/default"] : []), message: "souls.default was removed in 0.38.0 (team model 3): a soul's default team is souls: in oats-workspace.yaml (default:)" },
+  ],
 };
 function withRemovedKeys(problems, value, removed) {
   const found = removed.flatMap((r) => r.at(value).map((path) => ({ path, reason: "removed-key", message: r.message })));
@@ -316,7 +322,7 @@ function fromProblems(remote, capabilities, path, problems, { here }) {
 }
 /** Schema + domain problems for an oats-local.yaml value: settings[<cap>] may not carry the removed `byTeam` key. */
 export function validateLocal(value) {
-  const problems = validateAgainst(schemaFor("local"), value);
+  const problems = withRemovedKeys(validateAgainst(schemaFor("local"), value), value, REMOVED_KEYS.local);
   if (isObject(value) && isObject(value.settings)) for (const [cap, payload] of Object.entries(value.settings)) reservedKeyProblems(payload, `/settings/${pointerKey(cap)}`, problems);
   // runtime → harness (0.27.0, lead call 6): `runtime`, the pre-0.27 name, is still read; both,
   // disagreeing, are refused.
@@ -361,8 +367,12 @@ function schemaError(kind, origin, problems, value) {
   const where = origin.repoKey ? `${origin.repoKey}@${(origin.commit || "").slice(0, 12)}:${origin.path}` : origin.path;
   // A single-cause refusal (reserved-key, …) surfaces its reason on details for callers that branch on it.
   const reasons = new Set(problems.map((p) => p.reason).filter(Boolean));
-  return fail("E_WORKSPACE_SCHEMA", `${FILE_KINDS[kind].file} at ${where} is invalid${schemaHint(kind, value)}: ${problems.map((p) => `${p.path || "/"}: ${p.message}`).join("; ")}`, {
-    path: origin.path, repoKey: origin.repoKey, commit: origin.commit, problems, ...(reasons.size === 1 ? { reason: [...reasons][0] } : {}),
+  // Team model 3: the refused souls.teams / souls.default come with the souls: that replaces them.
+  const moved = kind === "local" && problems.some((p) => p.reason === "removed-key" && /^\/souls\/(teams|default)$/.test(p.path)) ? soulsReplacement(value) : null;
+  const replacement = moved ? YAML.stringify(moved, { lineWidth: 0 }) : null;
+  return fail("E_WORKSPACE_SCHEMA", `${FILE_KINDS[kind].file} at ${where} is invalid${schemaHint(kind, value)}: ${problems.map((p) => `${p.path || "/"}: ${p.message}`).join("; ")}`
+    + (replacement ? `; commit this in oats-workspace.yaml (<member> is the soul's member repository name, as souls.disabled names it), then remove souls.teams and souls.default from oats-local.yaml:\n${replacement}` : ""), {
+    path: origin.path, repoKey: origin.repoKey, commit: origin.commit, problems, ...(reasons.size === 1 ? { reason: [...reasons][0] } : {}), ...(replacement ? { replacement } : {}),
   });
 }
 

package/package-catalog.json

@@ -8,7 +8,7 @@
     },
     "oats.aweb": {
       "url": "https://github.com/awebai/oats-aweb.git",
-      "ref": "v1.18.1",
+      "ref": "v1.19.0",
       "path": "oats-package"
     },
     "oats.jira": {
@@ -33,7 +33,7 @@
     },
     "oats.framework": {
       "url": "https://github.com/awebai/oats.git",
-      "ref": "oats-framework/v1.4.3",
+      "ref": "oats-framework/v1.5.0",
       "path": "oats-package"
     }
   },

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@awebai/oats",
-  "version": "0.37.0",
+  "version": "0.38.0",
   "description": "OATS (Open Agent Team Specification) — durable souls, disposable instances, targetable capability packages, and the runtime-neutral oats CLI/kernel.",
   "keywords": [
     "agents",

package/skills/oats-getting-started/SKILL.md

@@ -60,7 +60,7 @@ members:
   - git:github.com/acme/agents          # the host is a member too
   - git:github.com/acme/platform
 packages:
-  oats.framework: v1.4.3                # bare versions resolve through the official catalog
+  oats.framework: v1.5.0                # bare versions resolve through the official catalog
   oats.okf: v4.1.1
 defaults:
   capabilities: { oats.core: { from: package } }
@@ -135,18 +135,29 @@ membership.
 ## 6. Give the deployment a team (with messaging)
 
 If a messaging capability fills the messaging slot, every instance lives in a
-team, and readiness fails with `E_TEAM_UNCONFIGURED` until this deployment has
-a default team. Create the team with the messaging provider (its own skills say
-how), then record it:
+team, and readiness fails with `E_TEAM_UNCONFIGURED` until there is a default
+team. Create the team with the messaging provider (its own skills say how),
+then commit it in `oats-workspace.yaml` with its provider id, and make it the
+default:
+
+```yaml
+teams:
+  research: { team: <provider team id> }
+defaultTeam: research
+```
+
+Push, then sync and check:
 
 ```bash
-oats teams add research --team <provider team id>   # a local team; the first one becomes the default
-oats teams
+oats sync --dir <dir>
+oats teams --dir <dir>
 ```
 
-A team the whole organisation uses is committed in `oats-workspace.yaml` as
-`teams.<label>` with its provider id; which souls join which team on this
-machine is `oats soul teams` (`/oats-teams`, in `oats.setup`).
+The teams an instance may join, and its default, are the organisation's
+decision, committed in this file: by default a soul joins its default team
+only, and `souls:` entries open other teams to a soul. A deployment declares
+teams of its own (`oats teams add`) only when the workspace says
+`localTeams: true` (`/oats-teams`, in `oats.setup`).
 
 ## 7. Spawn the first soul