Skip to content

History / Review command

Revisions

  • Document commit-level filters and path denylists Filtering now has two axes: which commits are reviewed (--author, --committer, --start-date, --end-date, --text) and which files within them (the ignore layers, the --file/--dir allowlist, and the new --exclude-file/--exclude-dir denylist). Restructure Filtering.md around that split and add the details a user can only get wrong once: --end-date includes the day named, branch-name matching is best-effort because a squash merge destroys the association, and the reviewed diff is a concatenation of per-commit patches so one file can appear several times. Also record where the filters do NOT apply and why — `commit` describes the staged index, `remote pr` reads its diff from `gh` — and note that `remote pr` now honours --file/--dir on the posting path, not only under --no-post. Global-flags, Review, Diff, Summary, Dry-run, Guard, MCP-server, Commit, Remote-pr, JSON-schema updated to match.

    @muhammetsafak muhammetsafak committed Jul 26, 2026
    8e38464

  • docs: sandbox rerun skips test files the toolchain would not compile Unsatisfied //go:build constraints, GOOS-suffixed names, and testdata/ files resolve to no test name — including tags the sandbox_command itself passes, so -tags=integration in the command does not make a build-tagged file resolvable. Those tests keep the static finding and skip confirmation.

    @muhammetsafak muhammetsafak committed Jul 25, 2026
    ce3e92a

  • docs: sandbox-rerun execution boundary and policy sharing (v1.14.0) Document review.sandbox_command: the argv-list form, the double opt-in (--sandbox-rerun plus a non-empty command), config set rejection, the 2-minute per-attempt bound, and the stderr notice. Test-name resolution is Go-only: non-Go files take the skip-and-warn path, so Python/JS/PHP/Java tests keep full static flaky detection but get no rerun confirmation. mcp and guard never execute the command at all. Also explain how to share .commitbrief/policy.yml past the wholesale .commitbrief/ gitignore, and why baseline.json deliberately stays ignored.

    @muhammetsafak muhammetsafak committed Jul 25, 2026
    726fa51

  • docs: sandbox-rerun flag + config for the flaky detector (v1.12.0)

    @muhammetsafak muhammetsafak committed Jun 21, 2026
    66acf53

  • Document architecture-aware review (archlint architecture.json bridge)

    @muhammetsafak muhammetsafak committed Jun 20, 2026
    502076b

  • Document signal control, pre-commit hooks, and new flaky detectors

    @muhammetsafak muhammetsafak committed Jun 20, 2026
    9a0ec25

  • docs: flaky-test detector — config (review.flaky), --no-flaky flag, review command (ADR-0022)

    @muhammetsafak muhammetsafak committed Jun 18, 2026
    3cb8a48

  • Add new global flags and a provider for OpenAI-compatible APIs

    @muhammetsafak muhammetsafak committed May 28, 2026
    2f0c60d

  • Shrink `COMMITBRIEF.md` via `commitbrief compress` — [Compress command]

    @muhammetsafak muhammetsafak committed May 28, 2026
    0b2b29f

  • Add new documentation files

    @muhammetsafak muhammetsafak committed May 27, 2026
    30bcbc4