Skip to content
This repository was archived by the owner on Jun 8, 2026. It is now read-only.

Retire schema guard from package workspace contract (#148)#152

Merged
fitz123 merged 15 commits into
issue-148-package-cli-workspace-contractfrom
issue-148-control-agent-contract-cleanup
Jun 6, 2026
Merged

Retire schema guard from package workspace contract (#148)#152
fitz123 merged 15 commits into
issue-148-package-cli-workspace-contractfrom
issue-148-control-agent-contract-cleanup

Conversation

@fitz123

@fitz123 fitz123 commented Jun 6, 2026

Copy link
Copy Markdown
Owner

Summary

Stacked on #151 / issue-148-package-cli-workspace-contract.

This follow-up completes the corrected #148 architecture:

  • treats --workspace as the control/app workspace;
  • allows absolute agents.*.workspaceCwd outside the control workspace;
  • retires schema/write-guard package contract (schema.md, MINIME_SCHEMA_PATH, Pi guard extension, Claude guard hooks);
  • keeps Telegram/Discord/Tavily as control-workspace/global secrets;
  • passes non-secret control path refs to Pi parent/cron/subagent children;
  • keeps Tavily on dedicated control-root config/secrets.sops.yaml + tavily.api_key;
  • adds deterministic schema/guard contract removal check and private cleanup artifact.

Validation

Ralphex reported:

  • git diff --check
  • npm run typecheck
  • npm run check:schema-guard-contract
  • npm run validate-config
  • npm test — 1434 pass, 0 fail
  • npm run build
  • npm run workspace:validate -- --workspace ./test-fixtures/minimal-workspace
  • npm pack --dry-run

Notes

@fitz123 fitz123 merged commit e3ef2d4 into issue-148-package-cli-workspace-contract Jun 6, 2026
@fitz123

fitz123 commented Jun 6, 2026

Copy link
Copy Markdown
Owner Author

Superseded by #153. This PR was accidentally marked merged when the stacked base branch was briefly advanced to the cleanup head; the base branch has been restored to #151 head (6ece37c), and #153 is the active stacked PR for this change.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant