Skip to content

docs(plan): Mecatl Studio settings acceptance plan - #1761

Merged
samuv merged 1 commit into
studio/knowledgefrom
plan/studio-settings
Sep 23, 2026
Merged

samuv merged 1 commit into
studio/knowledgefrom
plan/studio-settings

Conversation

@samuv

@samuv samuv commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Adds docs/acceptance/studio-settings.md (Bounded): the safe, capability-gated runtime-settings inventory (management stays deployment-managed), storage health, the eight-section settings workspace, and the restoration of chat's model picker over the live inventory.

Review focus

  • The runtime-settings response is an allowlisted projection: build id, server implementation, provider display endpoint, inventory; never a credential or raw endpoint.
  • Provider and routing configuration are capability-disabled in the UI with the daemon-managed reasons.

Checker: check-acceptance-plan.sh passes; task docs passes. Stacked delivery: nothing in this series merges until the whole stack is reviewed; the plan's approval-by-merge event is deferred to the end, as its Delivery line records.

Contract amendments

  • AC3.3. The models section names each deployment-managed control with the BFF's reason, as it does now, instead of rendering disabled controls. Sign-out is in the about section, where the code puts it.
  • Model picker. The plan now says the chat layer deferred it, not stubbed it, matching the chat plan.

Development stage

  • Plan / Interface
  • Implementation
  • Combined
  • Spike / Routine / Cleanup

Contract linkage

  • Work classification: Bounded
  • Classification rationale: adds the read-only runtime-settings and storage-health surface inside the boundary ADR 0351 fixed
  • Decision record: None — the boundary and the published-SDK rule are ADR 0351's; this plan only fills in a feature contract that ADR anticipated
  • Human waiver of spine: Yes — the directing human chose stacked (gh stack) delivery for the whole Studio series: nothing merges until every layer is reviewed, so the plan's approval-by-merge event is deferred to the end (recorded in each plan's Delivery line), and each implementation layer records the exact plan commit it built against.
  • Acceptance plan: docs/acceptance/studio-settings.md
  • Human decisions resolved and recorded: Yes
  • Plan / Interface PR: this PR
  • Approved commit baseline: absent until the stack root merges (stacking exception above)
  • Combined/exemption rationale: N/A

Interface conformance

  • Interface contract under review: GET /api/v1/settings/runtime (allowlisted, read-only) and GET /api/v1/storage/health; management stays deployment-managed. No gRPC, Go, CLI, or tool-schema change.

Issue relationship

Relates to #1750 (parent #1736; stack #1766). Non-closing: the step completes when both of its layers merge.

Type of change

  • Behavioral/interface plan
  • Bug fix
  • New feature
  • Refactoring (no behavior change)
  • Dependency update
  • Documentation/process

Test plan

Baseline checks

  • Acceptance-plan checker
  • Linting (task lint)
  • Offline test suite (task test)
  • Offline demo (go run ./cmd/mecademo)
  • Markdown changed: docs generation/link checks (task docs)

Plan layer: the acceptance-plan checker and task docs pass. The Studio gates (task studio:check, task ac-trace) run on the implementation layer above, since this layer adds no apps/ code.

User-facing change

None — a plan document and, for the bootstrap, ADR 0351.

Special notes for reviewers

Part of a fourteen-layer stack; review bottom-up. Nothing merges until the whole series is reviewed.

@samuv
samuv added this pull request to stack #1766 September 22, 2026 08:01
@samuv samuv self-assigned this Sep 22, 2026
@samuv
samuv force-pushed the plan/studio-settings branch from 36b9987 to d5972c7 Compare September 22, 2026 09:03
@samuv
samuv force-pushed the plan/studio-settings branch from d5972c7 to fae5537 Compare September 22, 2026 09:13
@samuv
samuv force-pushed the plan/studio-settings branch from fae5537 to 77c6764 Compare September 22, 2026 09:23
@samuv
samuv force-pushed the plan/studio-settings branch from 77c6764 to 337015a Compare September 22, 2026 09:59
@samuv
samuv force-pushed the plan/studio-settings branch from 337015a to d3dadbc Compare September 23, 2026 08:56
@samuv
samuv force-pushed the plan/studio-settings branch from d3dadbc to 5104988 Compare September 23, 2026 09:40
@samuv
samuv force-pushed the plan/studio-settings branch from 5104988 to e000669 Compare September 23, 2026 10:47
@samuv
samuv force-pushed the plan/studio-settings branch from e000669 to f43bcb0 Compare September 23, 2026 11:54
Bounded plan for the fourth Studio feature layer: the safe,
capability-gated runtime-settings inventory (management stays
deployment-managed), storage health, the eight-section settings
workspace with its ported helper tests as proofs, and the restoration of
chat's model picker over the live inventory.

Tracking: #1736

Signed-off-by: Samuele Verzi <samu@stacklok.com>
@samuv
samuv force-pushed the plan/studio-settings branch from f43bcb0 to 3c0ccbb Compare September 23, 2026 12:54
@samuv
samuv merged commit 42484de into main Sep 23, 2026
31 checks passed
@samuv
samuv deleted the plan/studio-settings branch September 23, 2026 13:15
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