axstack 0.30.0 → 0.31.0

package/README.md

@@ -15,7 +15,7 @@ coordination. You can start at the phase you need.
 | Failure mode | How Axstack responds |
 | --- | --- |
 | Wrong thing built | Align rounds clarify the request; every brainstorm runs a light arena across configured families. Use judges only for Rung 2 hard-to-reverse choices. |
-| Nobody really reviewed it | Strict TDD checks behavior first; mixed authored review is cross-provider, and mixed peer review uses two isolated Sol-high sessions at the exact revision. |
+| Nobody really reviewed it | Strict TDD checks behavior first; mixed authored review is cross-provider, and mixed peer review pairs Sol high with Opus low in isolated sessions at the exact revision. |
 | Design rot | The design lens sketches boundaries before a build; Improve surfaces evidenced changes later. |
 | Agents left a mess | T3 makes delegation visible, one writer owns each PR, cleanup stays bounded, and watched own PRs merge under guarded rules. |
 

package/docs/workflows.md

@@ -119,7 +119,7 @@ role.
 
 | Preset | Author | Ordered peer reviewers | Astra / Opus advisers | Auditor |
 | --- | --- | --- | --- | --- |
-| `mixed` | Sol high | Sol high; Sol high | Astra high / Opus xhigh | Sonnet high + Sol high |
+| `mixed` | Sol high | Sol high; Opus low | Astra high / Opus xhigh | Sonnet high + Sol high |
 | `codex-only` | Sol high | Sol high; Luna xhigh | Astra high / unavailable | Luna xhigh + Sol high |
 | `claude-only` | Opus medium | Opus medium; Sonnet high | unavailable / Opus xhigh | Sonnet high (Sol absent) |
 
@@ -147,7 +147,8 @@ later installation changes without re-resolution.
 
 Peer roles use the stable IDs `axstack-reviewer-primary` and
 `axstack-reviewer-peer`; their provider/class mappings come only from the
-selected preset. The mixed peer coordinator and review-lane binding
+selected preset. The mixed cross-provider peer pair is Sol high followed by
+Opus low. The mixed peer coordinator and review-lane binding
 `axstack-owner` uses Sol high and never reviews. `axstack-reviewer-secondary`
 is used in the authored mapping and, alongside `axstack-reviewer-primary`,
 in codebase findings mode.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "axstack",
-  "version": "0.30.0",
+  "version": "0.31.0",
   "description": "Axstack installer and setup CLI: installs owned chat skills and role data, configures supported harness settings, and checks T3 Code capabilities.",
   "keywords": [
     "claude-code",

package/profiles/presets/mixed.json

@@ -43,7 +43,7 @@
       "provider": "codex",
       "modeId": "full-access",
       "thinkingOptionId": "high",
-      "notes": "Primary reviewer in the ordered mixed peer pair: Sol high followed by Sol high. Eligible authored reviewer for an Opus-authored candidate. Never author or owner; review exact SHA and base across all six angles and acceptance. Peer first pass stays isolated.",
+      "notes": "Primary reviewer in the ordered mixed peer pair: Sol high followed by Opus low. Eligible authored reviewer for an Opus-authored candidate. Never author or owner; review exact SHA and base across all six angles and acceptance. Peer first pass stays isolated.",
       "modelClass": "sol"
     },
     {
@@ -58,11 +58,11 @@
     {
       "id": "axstack-reviewer-peer",
       "name": "Axstack reviewer (peer)",
-      "provider": "codex",
-      "modeId": "full-access",
-      "thinkingOptionId": "high",
-      "notes": "Second reviewer in the ordered mixed peer pair: Sol high followed by Sol high. Never author or owner; review exact SHA and base across all six angles and acceptance. Peer first pass stays isolated.",
-      "modelClass": "sol"
+      "provider": "claude",
+      "modeId": "bypassPermissions",
+      "thinkingOptionId": "low",
+      "notes": "Second reviewer in the ordered mixed peer pair: Sol high followed by Opus low. Never author or owner; review exact SHA and base across all six angles and acceptance. Peer first pass stays isolated. T3 resolves Claude classes from saved capabilities; a Claude rejection holds.",
+      "modelClass": "opus"
     },
     {
       "id": "axstack-diligence",

package/skills/axstack/references/role-roster.md

@@ -7,7 +7,8 @@
   Only the recorded owning watch thread merges, including `axstack-owner`
   for authorized standalone maintenance. Other roles never acquire merge authority.
 - `axstack-reviewer-primary` and `axstack-reviewer-peer` are the ordered
-  peer pair. Peer review uses both; authored review uses this table:
+  peer pair. In `mixed`, the cross-provider pair is Sol high followed by Opus low.
+  Peer review uses both; authored review uses this table:
 
   | Preset | Author class | Reviewer (class/effort) |
   | --- | --- | --- |

package/skills/axstack/scripts/dispatch-plan.js

@@ -0,0 +1,134 @@
+import { readFileSync } from 'node:fs';
+
+const flags = ['--role', '--roles', '--capabilities', '--picker', '--kind', '--verify'];
+const nonempty = (value) => typeof value === 'string' && value.trim().length > 0;
+const help = `Usage: bun dispatch-plan.js --role <id> --roles <roles.json>
+  --capabilities <saved.json> --picker <picker.json> --kind task|launch
+  [--verify <configuration.json>]
+
+--role          Role ID from the supplied snapshot
+--roles         Installed-style roles.json (version 1, roles array)
+--capabilities  Saved orchestrator_capabilities JSON; reuse the run's saved snapshot
+--picker        Saved picker JSON; uses chosenInstanceId
+--kind          task emits target; launch emits modelSelection
+--verify        Compare t3_thread_configuration JSON; print MATCH or each mismatch
+--help          Show all flags and exit codes
+
+Read-only: reads supplied JSON, spawns adjacent resolve-models.js, writes stdout/stderr.
+Never dispatches or changes snapshots. Reuse recorded exact model pins as given.
+Exit codes: 0 ok, 1 error, 2 hold (including incomplete read-back).
+`;
+
+function hold(message) {
+  throw Object.assign(new Error(message), { exitCode: 2 });
+}
+
+function argumentsFrom(args) {
+  const options = {};
+  for (let i = 0; i < args.length; i += 2) {
+    const flag = args[i];
+    if (!flags.includes(flag)) throw new Error(`unexpected argument ${flag}`);
+    const value = args[i + 1];
+    if (!nonempty(value) || value.startsWith('--')) throw new Error(`expected value after ${flag}`);
+    if (flag in options) throw new Error(`duplicate argument ${flag}`);
+    options[flag] = value;
+  }
+  for (const flag of flags.filter((flag) => flag !== '--verify')) {
+    if (!options[flag]) throw new Error(`expected ${flag}`);
+  }
+  if (!['task', 'launch'].includes(options['--kind'])) throw new Error('expected --kind task|launch');
+  return options;
+}
+
+const readJSON = (path) => JSON.parse(readFileSync(path, 'utf8'));
+
+function normalizedOptions(options) {
+  const entries = Array.isArray(options) ? options.map((option) => [option?.id, option?.value])
+    : options && typeof options === 'object' ? Object.entries(options) : null;
+  if (!entries || entries.some(([id, value]) => typeof id !== 'string' || !id.trim()
+    || !['string', 'boolean'].includes(typeof value))
+    || new Set(entries.map(([id]) => id)).size !== entries.length) return null;
+  return JSON.stringify(entries.sort(([left], [right]) => left.localeCompare(right)));
+}
+
+function verify(configuration, instanceId, provider, model, options) {
+  const mismatches = [];
+  const selection = configuration?.modelSelection;
+  const compare = (field, expected, observed, equal = expected === observed) => {
+    if (!equal) mismatches.push(`${field}: ${observed == null ? 'unknown' : 'fail'}; `
+      + `expected ${JSON.stringify(expected)}; observed ${JSON.stringify(observed) ?? 'missing'}`);
+  };
+  // T3 may omit the legacy provider field: the exact instance ID is its binding.
+  if (selection?.provider != null) {
+    compare('provider', provider, selection.provider,
+      (selection.provider === 'claude' ? 'claudeAgent' : selection.provider) === provider);
+  }
+  compare('instanceId', instanceId, selection?.instanceId);
+  compare('model', model, selection?.model);
+  compare('options', options, selection?.options,
+    normalizedOptions(options) === normalizedOptions(selection?.options));
+  compare('runtimeMode', 'full-access', configuration?.runtimeMode);
+  console.log(mismatches.length ? mismatches.join('\n') : 'MATCH');
+  return mismatches.length ? 2 : 0;
+}
+
+function main() {
+  if (process.argv.slice(2).includes('--help')) {
+    process.stdout.write(help);
+    return 0;
+  }
+  const args = argumentsFrom(process.argv.slice(2));
+  const snapshot = readJSON(args['--roles']);
+  if (snapshot?.version !== 1 || !Array.isArray(snapshot.roles)) {
+    throw new Error('roles snapshot must have version 1 and a roles array');
+  }
+  const roles = snapshot.roles.filter((role) => role?.id === args['--role']);
+  if (roles.length !== 1) hold(`role ${args['--role']} unavailable or ambiguous`);
+  const role = roles[0];
+  if (!nonempty(role.provider)) hold(`provider unavailable for role ${role.id}`);
+  if (role.modeId !== 'full-access' && !(role.provider === 'claude' && role.modeId === 'bypassPermissions')) {
+    hold(`mode ${role.modeId ?? 'missing'} unavailable for role ${role.id}`);
+  }
+  if (!nonempty(role.thinkingOptionId)) hold(`effort unavailable for role ${role.id}`);
+  if (!nonempty(role.model) && !nonempty(role.modelClass) && role.model !== null) {
+    hold(`model unavailable for role ${role.id}`);
+  }
+  const picker = readJSON(args['--picker']);
+  if (!nonempty(picker?.chosenInstanceId)) hold('picker chosen instance unavailable');
+  const modelArgs = typeof role.model === 'string' ? ['--model', role.model]
+    : role.modelClass ? ['--class', role.modelClass] : ['--model', String(role.model)];
+  const resolved = Bun.spawnSync([process.execPath, `${import.meta.dir}/resolve-models.js`,
+    '--provider', role.provider, '--capabilities', args['--capabilities'],
+    '--effort', role.thinkingOptionId, ...modelArgs], { stdout: 'pipe', stderr: 'pipe' });
+  if (resolved.stderr.length) process.stderr.write(resolved.stderr);
+  if (resolved.exitCode !== 0) return resolved.exitCode === 1 ? 2 : 1;
+  const { model, effortOption, providerInstanceId } = JSON.parse(resolved.stdout.toString());
+  const chosen = readJSON(args['--capabilities']).providers
+    .filter((provider) => provider.providerInstanceId === picker.chosenInstanceId);
+  if (chosen.length !== 1 || chosen[0].driverKind !== providerInstanceId
+    || (['grok', 'antigravity'].includes(role.provider) && picker.chosenInstanceId !== providerInstanceId)) {
+    hold(`picker instance ${picker.chosenInstanceId} does not bind provider ${role.provider}`);
+  }
+  const selectedModel = chosen[0].models?.find((entry) => entry.id === model);
+  if (!selectedModel) hold(`model ${model} unavailable on instance ${picker.chosenInstanceId}`);
+  if (effortOption && !selectedModel.options?.some((option) => option.id === effortOption.id
+    && option.options?.some((value) => value.id === effortOption.value))) {
+    hold(`effort ${effortOption.value} unavailable on instance ${picker.chosenInstanceId}`);
+  }
+  const options = effortOption ? [effortOption] : [];
+  if (args['--verify']) {
+    return verify(readJSON(args['--verify']), picker.chosenInstanceId, providerInstanceId, model, options);
+  }
+  const selection = { [args['--kind'] === 'launch' ? 'instanceId' : 'providerInstanceId']: picker.chosenInstanceId,
+    model, options };
+  console.log(JSON.stringify({ [args['--kind'] === 'launch' ? 'modelSelection' : 'target']: selection,
+    runtimeMode: 'full-access' }));
+  return 0;
+}
+
+try {
+  process.exitCode = main();
+} catch (error) {
+  console.error(`dispatch-plan ${error.exitCode === 2 ? 'hold' : 'error'}: ${error.message}`);
+  process.exitCode = error.exitCode ?? 1;
+}

package/skills/axstack-review/SKILL.md

@@ -223,7 +223,8 @@ This section applies to peer and authored PR modes.
    - **Peer:** exactly two independent final reviewers,
      `axstack-reviewer-primary` and `axstack-reviewer-peer` from the routing snapshot.
      Send both the identical six-angle brief, with no first-pass cross-read or children.
-     Mixed peer reviewers share a model, with independence from separate sessions, the identical brief and an isolated first pass.
+     The mixed peer pair is cross-provider (Sol high + Opus low), with independence
+     still from separate sessions, the identical brief and an isolated first pass.
    - **Authored:** exactly one eligible independent reviewer from this complete
      mapping: