Skip to content

feat(admin): OpenShell run facts and a per-profile secrets view (#712) - #726

Merged
edgehero merged 1 commit into
mainfrom
feat/console-openshell-runs-712d
Oct 21, 2026
Merged

edgehero merged 1 commit into
mainfrom
feat/console-openshell-runs-712d

Conversation

@edgehero

Copy link
Copy Markdown
Owner

Part 4 of 4 for #712. Parts 1 to 3 (#722, #724 and #723) are merged.

What changes

  • OpenShell facts on the run record. On the two openshell venues, the record gains openshell: { mode, ruleHosts, ruleHostCount, providers, placeholders }.

    • The rule hosts are the generated policy's endpoints: the egress allowlist, the model endpoints and the fixed forge host. That is operator config, never repository or payload text, so the record stays PII-free.
    • Hosts must fit a host-name charset. At most 32 are listed, and the total is always kept.
    • No values appear.
  • The secrets profile on the run record. The record gains secretsProfile: the name of the profile a run's secrets resolved through.

  • Run views. The console run page and the panel's run detail show the sandbox policy, its rule hosts and the secrets profile.

  • MCP connection state. It is recorded nowhere, so no row claims it.

  • A "Secret profiles in use" view, in the console, the panel and /dispatch secrets list. For each profile it shows:

    • its source;
    • where an overlay resolver stands against PI_SECRET_RESOLVER_ROOTS;
    • the triggers and projects that use it;
    • its last resolve;
    • the live hosts that advertise it.

    The view never shows a path. A refusal whose reason the project bundle's read shares is worded as either read.

  • One rule for resolver roots. resolverRootsStatus is the single roots rule. The add's pre-confirm check reads it and stays strict on a bad seam.

  • Model context. dispatch_runs carries the openshell counts, plus host names only for a run read by id. It never carries resolver paths.

  • Conformance lanes. Through buildRecord, they check the exact facts of a real bundle run:

    • copy mode: 2 providers and 4 placeholders;
    • bind mode: 2 and 2;
    • the host set as networkPolicyFor derives it.

Specs

  • Amended: INT-RUN-HISTORY-FILE-CONTRACT (openshell and secretsProfile), DES-OPENSHELL-VENUE, DES-OPERATOR-CONSOLE, REQ-TRIGGER-SECRETS and REQ-OPERATOR-CONSOLE-SCREENS (g).

Residuals

Verification

  • Review: two adversarial rounds, with every finding fixed.
  • Full suite with Valkey: 9658 tests; only the known macOS mcp test and two timing tests failed.
  • Build and checks: node admin/build.mjs succeeds. The revision-row, test-count, temp-dir and dated-fixture checks pass.

Part 4 of #712, the last.

- The record gains openshell, on the two openshell venues only:
  { mode, ruleHosts, ruleHostCount, providers, placeholders }.
  - The rule hosts come from the generated policy's endpoints: the
    egress allowlist, the model endpoints and the fixed forge host. That
    is operator config, never repository or payload text, so the record
    stays PII-free.
  - Hosts are held to a host-name charset, at most 32 listed, with the
    total kept. Values never appear.
  - None is recorded for a stage that throws or is stopped before the
    exec, or for a detached run.
- The record gains secretsProfile: the profile name a run's secrets
  resolved through, its own or the implicit default.
- The console run page and the panel's run detail show the sandbox
  policy, its rule hosts and the secrets profile.
- MCP connection state is recorded nowhere, and no row claims it.
- A "Secret profiles in use" view (console, panel, /dispatch secrets
  list) gives each profile:
  - its source (env, overlay, or both, which is ambiguous);
  - where an overlay resolver stands against the roots;
  - the triggers and projects that use it;
  - its last resolve, from the newest run that names it;
  - the live hosts that advertise s:<name>.

  The view never shows a path; only the operator's own command prints
  them. A failure that a project bundle's read shares a reason with is
  worded as either read.
- resolverRootsStatus is the one roots rule. The add's
  resolverRootsJudgement reads it and stays strict on a bad seam; the
  view tolerates a row it cannot judge.
- Model context: dispatch_runs carries the openshell counts, and the
  host names only for a run read by its id, never resolver paths.
- The conformance lanes check, through buildRecord, the exact facts of a
  real bundle run: mode, hosts, providers and placeholders.

Specs: INT-RUN-HISTORY-FILE-CONTRACT (openshell, secretsProfile),
DES-OPENSHELL-VENUE, DES-OPERATOR-CONSOLE, REQ-TRIGGER-SECRETS and
REQ-OPERATOR-CONSOLE-SCREENS (g) are amended.

Signed-off-by: Rob Boerman <robboerman@live.nl>
@edgehero
edgehero merged commit 473410b into main Oct 21, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant