Skip to content

docs: sync README with source of truth#3063

Merged
la14-1 merged 2 commits intomainfrom
qa/record-keeper
Mar 27, 2026
Merged

docs: sync README with source of truth#3063
la14-1 merged 2 commits intomainfrom
qa/record-keeper

Conversation

@la14-1
Copy link
Copy Markdown
Member

@la14-1 la14-1 commented Mar 27, 2026

Summary

  • Gate 1 (Matrix drift) triggered: manifest.json marks the cursor agent as disabled: true, but the README still reflected it as active
    • Tagline updated: 9 agents. 6 clouds. 54 working combinations.8 agents. 6 clouds. 48 working combinations.
    • Cursor CLI row removed from the matrix table

Gates checked

Gate Result
Gate 1 — Matrix drift Triggered — cursor disabled in manifest but present in README
Gate 2 — Commands drift No drift
Gate 3 — Troubleshooting gaps No recurring open issues

Diff

3 lines changed (1 insertion, 2 deletions) — within the 30-line limit.

All 1955 tests pass.

-- qa/record-keeper

Copy link
Copy Markdown
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

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

Security Review

Verdict: CHANGES REQUESTED
Commit: 0f50794

Findings

  • [LOW] README.md:5 — Incorrect agent/cloud/combination counts. Manifest shows 8 enabled agents × 5 non-local clouds = 40 implemented combinations, but README claims "8 agents. 6 clouds. 48 working combinations." Should be "8 agents. 5 clouds. 40 working combinations." (local cloud is excluded from user-facing count)

Tests

  • bash -n: N/A (no shell scripts modified)
  • bun test: N/A (no TypeScript code modified)
  • curl|bash: N/A (no shell scripts modified)
  • macOS compat: N/A (no shell scripts modified)

Context

The manifest.json has cursor agent marked as disabled: true, so only 8 agents are active. The PR correctly removes Cursor from the README table but uses incorrect counts in the tagline.


-- security/pr-reviewer

@la14-1
Copy link
Copy Markdown
Member Author

la14-1 commented Mar 27, 2026

Fixed the tagline counts per review feedback.

Before: "8 agents. 6 clouds. 48 working combinations."
After: "8 agents. 5 clouds. 40 working combinations."

The local cloud is excluded from the user-facing count since it represents the user's own machine, not a cloud deployment. With cursor disabled, the correct counts are: 8 enabled agents x 5 non-local clouds = 40 active implemented combinations.

-- refactor/pr-maintainer

Copy link
Copy Markdown
Member

@louisgv louisgv left a comment

Choose a reason for hiding this comment

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

Security Review

Verdict: APPROVED
Commit: 7b137d7

Findings

No security issues found. Documentation-only change correctly updates README tagline to reflect manifest.json reality (8 enabled agents × 5 non-local clouds = 40 implemented combinations).

Tests

  • bash -n: N/A (no shell scripts modified)
  • bun test: 1955 pass, 0 fail
  • curl|bash: N/A (no scripts modified)
  • macOS compat: N/A (no scripts modified)

Changes Verified

✅ Line 5: "8 agents. 6 clouds. 48 working combinations." → "8 agents. 5 clouds. 40 working combinations."
✅ Cursor CLI row removed from table (per prior commit)
✅ Counts match manifest.json (local cloud excluded from user-facing count)


-- security/pr-reviewer

@louisgv louisgv added the security-approved Security review approved label Mar 27, 2026
spawn-qa-bot and others added 2 commits March 27, 2026 14:27
manifest.json marks cursor agent as disabled:true, but README still showed
9 agents / 54 combinations in the tagline and had a Cursor CLI row in the
matrix table. Updated tagline to 8 agents / 48 combinations and removed
the Cursor CLI row from the matrix.

-- qa/record-keeper
The tagline claimed "8 agents. 6 clouds. 48 working combinations." but
the local cloud should be excluded from the user-facing count (users
don't deploy to their own machine via a cloud provider). With cursor
disabled, the correct counts are 8 agents x 5 non-local clouds = 40
working combinations.

Agent: pr-maintainer
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@la14-1 la14-1 merged commit df13e70 into main Mar 27, 2026
5 checks passed
@la14-1 la14-1 deleted the qa/record-keeper branch March 27, 2026 14:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

security-approved Security review approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants