Skip to content

Align scouter uploads and command registration safety - #89

Draft
diese-tech wants to merge 1 commit into
mainfrom
agent/command-safety
Draft

Align scouter uploads and command registration safety#89
diese-tech wants to merge 1 commit into
mainfrom
agent/command-safety

Conversation

@diese-tech

Copy link
Copy Markdown
Owner

Summary

  • Enforce the canonical 10 MiB match-screenshot limit.
  • Remove placeholder /rules from runtime dispatch, guild registration, /help, and live-command documentation.
  • Centralize the explicit nine-command manifest so planned ADR-009 commands cannot register without handlers and contracts.
  • Add runtime rejection coverage for admin-only division commands.
  • Include deployment scripts in typechecking and cover the registration manifest with tests.

Why

The storage bucket accepts 10 MiB while SALbot accepted 20 MiB, and /rules exposed placeholder repository content. Registration, runtime dispatch, help, and documentation also needed one canonical command inventory.

Safety and impact

  • The next guild registration run removes /rules.
  • Planned ADR-009 commands remain unregistered.
  • Claude's Railway CI/CD and guild-registration implementation is not duplicated or overwritten.
  • No production deployment or database change is performed by this PR.

Validation

  • 130 tests passed across 34 source suites.
  • Bot, database package, shared package, and deployment-script typechecks passed.
  • Lint passed.
  • Production builds passed.
  • Dependency graph is unchanged.
  • git diff --check 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.

2 participants