Skip to content

docs: slim CONTRIBUTING and make extension guides step-by-step - #228

Merged
SaladDay merged 7 commits into
mainfrom
docs/contributor-cleanup
Sep 29, 2026
Merged

SaladDay merged 7 commits into
mainfrom
docs/contributor-cleanup

Conversation

@SaladDay

@SaladDay SaladDay commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

Part 3 of 3 of the documentation cleanup (after #225 and #226).

CONTRIBUTING: 3786 → 385 lines

CONTRIBUTING now keeps only repository-wide rules:

  • documentation ownership
  • repository and product boundary
  • decoupling
  • workflow and review
  • required checks
  • naming

Component rules moved to their owners, merged into existing sections where possible:

Content New owner
Executor/Turn lifetimes, wire rules docs/runtime-protocol.md
Capability preparation (3 copies merged into 1) contracts/agents-api/environments.md#runtime-capability-preparation
Distribution, installer, CI, native installation docs/maintainers.md
Hosted nodes, request handling, implementation constraints services/agents-api/IMPLEMENTATION.md (new)
Harness rules / acceptance harness-onboarding.md / harnesses.md#acceptance-checklist
MiniMax Code and Claude adapter rules their deploy/* READMEs
Console sign-in apps/web/README.md

Every removed block is listed with its reason. Nothing normative was dropped: blocks that might be normative were moved, not deleted.

Adding a Sandbox Provider or a Harness

  • docs/sandbox-provider.md is now a numbered start-to-finish guide.
    • It gives the correct required-operation count, 5: Create, GetInfo, Renew, Kill, RunCommand. Old text said 4 in places.
    • It lists every file where a provider kind is registered, verified in code.
  • harness-onboarding.md is the single numbered guide. harnesses.md keeps qualification and now has a MiniMax Code column.
  • Fixes, each checked against the code:
    • The Claude deploy README described an outdated adapter API.
    • Onboarding wrongly said there is no public harness field.
    • Cancellation ownership was ambiguous: Cancel is on the Session embedded by each Turn.
  • docs/development.md has one short paragraph and a link per extension boundary.

Historical contracts

  • Deleted mcode-workspace-v1.md (marked "historical evidence only") and startup-configuration.md (marked removed), with every inbound link fixed.
  • Removed internal project-board/Feishu process steps from contracts.

Rebase notes

The branch was rebased onto main after #222, and the upstream CONTRIBUTING changes were ported to their new owners:

  • native model configuration → harness-onboarding.md
  • self-hosted one-command installation rules → docs/maintainers.md

Validation

All of the following pass:

  • make check-docs
  • check-names
  • the release-bundle docs check (core-distribution-manifest.py docs)
  • core-distribution-manifest tests
  • the repo-wide link and anchor check: 0 broken across 108 Markdown files

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

CONTRIBUTING keeps only repository-wide rules. Component rules move to
their canonical owners (runtime-protocol, environments, harness docs,
Sandbox Provider guide, maintainers, service and adapter READMEs, and
the new services/agents-api/IMPLEMENTATION.md).

Sandbox Provider and Harness guides become numbered start-to-finish
paths with verified registration files, the five required provider
operations, engine selection and the Turn cancellation owner.
Delete mcode-workspace-v1.md (historical evidence only) and
startup-configuration.md (removed route), fix inbound links and drop
task-board/Feishu process steps from contracts.
The Parsar/Core product boundary is a repository-wide rule; restore it
under Repository boundary instead of the service implementation notes,
and reword two implementation lines to current names.
…zed docs

- The native direct-connection rule replaces the removed model proxy rule in
  services/agents-api/IMPLEMENTATION.md.
- Harness onboarding gains the internal/harnessconfig entry point and the new
  RegisterKind signature.
- User guide, Agents API guide and quickstart state each harness's supported
  protocols; there is no conversion.
- The check-distribution note moves into the maintainer guide.
@SaladDay
SaladDay merged commit af322f2 into main Sep 29, 2026
6 checks passed
@SaladDay
SaladDay deleted the docs/contributor-cleanup branch October 10, 2026 05:06
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