docs + chore: CLAUDE.md v0.35.3 progress + gitignore internal docs#413
Open
Nathan Schram (nathanschram) wants to merge 2 commits intodevfrom
Open
docs + chore: CLAUDE.md v0.35.3 progress + gitignore internal docs#413Nathan Schram (nathanschram) wants to merge 2 commits intodevfrom
Nathan Schram (nathanschram) wants to merge 2 commits intodevfrom
Conversation
- Bump unit-test count 2372 → 2387 (reflects #407 +8 test_build_args tests and prior untracked test additions). - Expand test_build_args.py entry 42 → 56 tests with the new coverage areas. - Add extra_args passthrough feature entry under "Features (vs upstream takopi)" — documents the Claude-in-Chrome motivator, reserved-flag list, and argv placement (#407, shipped in v0.35.3rc1). Issue progress tracked in gh#407 comment. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…dovers Public repo hygiene pass. Three classes of file shouldn't be committed going forward: 1. docs/handover/ (new) — Claude Code handover docs that pass context between sessions. Internal-only by nature. 2. docs/audits/ — incident and security audits referencing production bot names and internal workflows. GitHub issues/milestones are the public tracker. 3. docs/tests/ — per-release integration test plans and execution reports. Contain internal bot references, skipped-test notes, and QA methodology detail that isn't user-facing. docs/reference/ integration-testing.md remains the public playbook. 4. incoming/*.md — draft design and feedback markdown uploaded via Telegram file-transfer. Auto-named file_*.jpg already covered. Files untracked (contents preserved on disk): - docs/audits/pitchdocs-context-guard-interference.md - docs/tests/v0.35.2-integration-test-plan.md - docs/tests/results/v0.35.2-results.md - docs/tests/results/v0.35.2rc3-results.md mkdocs/zensical nav (zensical.toml) doesn't reference any of these paths, so the docs site build is unaffected. History note: these files remain visible in git history; removing them entirely would require a separate BFG/filter-branch pass, which is out of scope for this PR. The forward-going commitment is that internal planning/audit/test-artifact content stops being tracked. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Two-in-one small-PR combining docs note + public-repo hygiene gitignore pass.
Commit 1 —
f440f91docs(claude.md): note v0.35.3rc1 staging + Claude extra_args featuretest_build_argstests + prior untracked additions).test_build_args.pyentry 42 → 56 tests with new coverage areas.extra_argspassthrough (Claude runner: add extra_args for upstream CLI flags (enables --chrome for Claude-in-Chrome) #407, v0.35.3rc1) feature entry under "Features (vs upstream takopi)".Commit 2 —
90a2df6chore(gitignore): untrack internal docsPublic repo hygiene. Three classes of file now gitignored going forward:
docs/handover/(new) — Claude Code handover docs between sessions. Internal-only.docs/audits/— incident and security audits referencing production bot names + internal workflows. GitHub issues/milestones are the public tracker.docs/tests/— per-release integration test plans and execution reports. Contain internal bot refs, skip-notes, QA methodology.docs/reference/integration-testing.mdremains the public playbook.incoming/*.md— draft design/feedback markdown uploaded via Telegram file-transfer.incoming/file_*.jpgauto-uploads also covered.Files untracked from index (contents preserved on disk):
docs/audits/pitchdocs-context-guard-interference.mddocs/tests/v0.35.2-integration-test-plan.mddocs/tests/results/v0.35.2-results.mddocs/tests/results/v0.35.2rc3-results.mdDocs site impact
None.
zensical.tomlnav does not referenceaudits/,tests/, orhandover/. Build unaffected.History caveat
These files remain visible in git history. Full scrub would require BFG/filter-branch — out of scope for this PR. Forward-going commitment is that new internal planning/audit/test-artifact content stops being tracked.
Test plan
git check-ignore -vconfirms all four new patterns match expected pathsdocs/audits/pitchdocs-*.md,docs/tests/*,docs/handover/*,incoming/*.mdall now gitignoredgit rm --cached(contents preserved)zensical.tomlnav audit — no references to removed folders🤖 Generated with Claude Code