Skip to content

chore: remove CLI audit markdown from git - #65

Merged
echobt merged 3 commits into
mainfrom
cursor/remove-cli-audit-markdown-1152
Sep 10, 2026
Merged

chore: remove CLI audit markdown from git#65
echobt merged 3 commits into
mainfrom
cursor/remove-cli-audit-markdown-1152

Conversation

@echobt

@echobt echobt commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Audits stay in Linear / chat only — not in CortexLM/cli git.

Removes the docs/audits/ dump (CORTEX_CLI_100_AUDIT_2026-09-08.md) and the docs index row that pointed at it.

design/cli-lock-board-index.md is self-contained (no deleted-audit §/P0/P1 citations). tui.key recipes that Greptile proved were no-ops (F2, Shift+Tab, Alt+Enter, Ctrl+x) are replaced with lock.render / cargo tests / Shift+Enter. Same limit is noted in docs/guides/development.md.

Guides, CI_SECRETS.md, CONTRIBUTING.md, and docs/README.md stay.

Test plan

  • Confirmed no remaining links to docs/audits/ or CORTEX_CLI_100_AUDIT_*
  • No tui.key F2 / Shift+Tab / Alt+Enter / Ctrl+x recipes that claim to open Settings, cycle mode, newline, or Shortcuts
  • docs/guides/, docs/CI_SECRETS.md, docs/CONTRIBUTING.md, docs/README.md retained
  • cargo fmt --all -- --check — docs-only; not required
  • ./scripts/clippy.sh — docs-only; not required
  • cargo test --workspace — docs-only; not required
  • TUI / snapshot tests — no TUI surface touched
  • cargo audit — no dependency change
  • Source/dependency policy and generated API contracts — unchanged
  • Local functional/security QA and changed-line coverage — docs-only; no production lines changed

Attestation (required)

I attest that:

  • Security reviewed — auth, exec/sandbox, path traversal, network egress, and secret handling in this diff were reviewed. No secrets, tokens, or keyring dumps are in the change.
  • Product-facing errors — user-visible failures use Cortex product copy. API-down paths say The coding service is temporarily unavailable. No raw provider, SDK, or transport names.
  • TUI verified — no TUI surface touched.
  • Tests added — no logic change; docs-only.
  • No secrets — no API keys, WorkOS secrets, R2/AWS credentials, or .env files are included.

Risk

Docs-only. No auth, exec policy, sandbox, release, or API-contract impact. Merge still needs a non-author human approving review.

Open in Web Open in Cursor 

Audits stay in Linear/chat only. Drop docs/audits/ and the index links.

Co-authored-by: Mathis <echobt@users.noreply.github.com>
@echobt

echobt commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai

@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown

Greptile Summary

This documentation-only update removes the obsolete CLI audit while preserving a self-contained lock-board reference. It replaces former audit citations with current source, test, verifier, and capture-regeneration references, and accurately documents the verification MCP’s supported key-input behavior.

Confidence Score: 5/5

Safe to merge once the required non-author Protect review is complete.

The previously reported missing-audit-context issue is fully addressed: the lock-board index now contains current, tracked references for its flows, verification surface, test coverage, source of truth, and regeneration procedures. The previously reported special-key recipe issue is fully addressed: unsupported key recipes were removed or redirected to applicable rendering and test coverage, and Shift+Enter is documented as the supported newline input.

Reviews (5): Last reviewed commit: "docs(verify): drop false tui.key special..." | Re-trigger Greptile

Comment thread design/cli-lock-board-index.md
Drop leftover audit §/P0/P1 citations. Point at verify MCP, tests, and render scripts.

Co-authored-by: Mathis <echobt@users.noreply.github.com>
@cursor

cursor Bot commented Sep 10, 2026

Copy link
Copy Markdown

@greptileai

1 similar comment
@echobt

echobt commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai

Comment thread design/cli-lock-board-index.md Outdated
tui.key only applies clear, Shift+Enter newline, backspace, and insert.
F2, Shift+Tab, Alt+Enter, and Ctrl+x are no-ops on the verify path.

Co-authored-by: Mathis <echobt@users.noreply.github.com>
@cursor

cursor Bot commented Sep 10, 2026

Copy link
Copy Markdown

@greptileai

1 similar comment
@echobt

echobt commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

@greptileai

@echobt

echobt commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

Closing per standing rule: never merge audit/research .md PRs — remove these files out of band if needed, not via merge to main.

@echobt echobt closed this Sep 10, 2026
@echobt echobt reopened this Sep 10, 2026
@echobt

echobt commented Sep 10, 2026

Copy link
Copy Markdown
Contributor Author

Reopened after accidental close — purge audit md must merge, not close. Still needs non-author Protect review. @greptileai

@echobt
echobt merged commit 0c6f45e into main Sep 10, 2026
37 checks passed
@echobt
echobt deleted the cursor/remove-cli-audit-markdown-1152 branch September 10, 2026 14:42
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