Skip to content

Docs: Adjust MOCA to moca as a name, not acronym - #695

Merged
maia-iyer merged 3 commits into
rossoctl:mainfrom
maia-iyer:moca-lowercase-not-acronym
Nov 5, 2026
Merged

maia-iyer merged 3 commits into
rossoctl:mainfrom
maia-iyer:moca-lowercase-not-acronym

Conversation

@maia-iyer

@maia-iyer maia-iyer commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Assisted-By: GLM 5.3

Summary

Team has agreed moca is not an acronym - simply a name. Adjusted docs to reflect this

Checklist

  • Commits are signed off (git commit -s).
  • make lint and the tests for the touched packages pass.
  • Docs: changed behavior, flags or environment variables are reflected in the relevant README or
    docs/ page, with the right maturity label (see docs/CONTRIBUTING-DOCS.md).
  • Docs: make check-links passes if any Markdown changed.
  • A new spec or ADR is registered in docs/specs/README.md or docs/adrs/README.md.

Signed-off-by: Maia Iyer <maia.raj.iyer@gmail.com>
Assisted-By: GLM <noreply@z.ai>
@maia-iyer
maia-iyer force-pushed the moca-lowercase-not-acronym branch from 6356344 to e31115d Compare November 5, 2026 16:28
@maia-iyer
maia-iyer marked this pull request as ready for review November 5, 2026 16:32

@ingpaolodettori-dev ingpaolodettori-dev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Summary

A docs-only rename of the product from MOCA to moca in 28 files. Most of it is fine, but it breaks a regression test: harness/test/config-resolver.test.ts uses the MOCA in CLAUDE.md as its sentinel, and that test is why check is red. Two demo READMEs also lowercased the mock-model marker identifiers, which are case-sensitive and still MOCA-DEMO* in code.

Author: maia-iyer (MEMBER — maintainer)
Areas reviewed: Docs, cross-checked against tests/scripts that read them
Agent/IDE config (.claude/.vscode): none (deploy/claude/commands/promote.md is a shipped template, not repo agent config; a 1-line description change)
Commits: 1 commit, all signed-off: yes
CI status: failing — check (Test step; both attempts) — caused by this PR, see the CLAUDE.md comment. Everything else is green.

Comment thread CLAUDE.md Resolved
Comment thread docs/CONTRIBUTING-DOCS.md Resolved
Swap leak-test sentinel to 'DCO sign-off required', restore
MOCA-DEMO-* marker names in demo READMEs, document moca
capitalization rule.

Assisted-By: GLM 5.3
Signed-off-by: Maia Iyer <maia.raj.iyer@gmail.com>

@ingpaolodettori-dev ingpaolodettori-dev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Re-reviewed at cc39b76. The follow-up commit closes all four e31115d items:

  • The leak regression test now uses 'DCO sign-off required' as its sentinel in both assertions (config-resolver.test.ts:176 and :186) and in the L162 comment, so the negative assertion still means something.
  • Both demo READMEs are back to the case-sensitive MOCA-DEMO-* / MOCA-DEMO2-* markers that mock-anthropic.mjs matches.
  • CONTRIBUTING-DOCS.md now documents the capitalization rule and the ADR/spec exemption, and defers user-visible source strings to a follow-up.

One nit inline. LGTM.

Assisted-By: Claude Code

Comment thread docs/CONTRIBUTING-DOCS.md Resolved
Assisted-By: GLM 5.3
Signed-off-by: Maia Iyer <maia.raj.iyer@gmail.com>
@maia-iyer
maia-iyer merged commit 2f47132 into rossoctl:main Nov 5, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants