Repository navigation
feat(cli): add saved source-change reviews with explicit acknowledgment - #38
Merged
Merged
Conversation
README.md conflicted: this branch and main's "docs: declutter README" (#36) each rewrote it from the same 469-line version. Kept this branch's README, which documents `refract review`, the offline walkthrough and the captured Bitcoin example, and carried over what #36 added that it lacked: the Bun badge, the Python SDK (as a row in "Use it in your application"), the monorepo architecture tree, and the development commands, using `bun run test` (vitest) rather than `bun test`. #36's Semaglutide sample output and its docs link list are not carried over; every link in that list except the docs/README.md hub already appears in this README. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
zz-plant
added a commit
that referenced
this pull request
Oct 13, 2026
packages/cli/src/commands/analyze.ts conflicted on runAnalyze's signature: #38 widened its return type with `coverage`, this branch added the `{ recordObservation }` option. Kept both. CHANGELOG.md had two "## Unreleased" headings on main, from #38's section merging beside the existing one without a conflict. Folded them into one, entries unchanged. Co-Authored-By: Claude Opus 5.5 <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
1 participant
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.
Re-running an analysis does not tell a reviewer which source changes remain unreviewed. This adds
refract reviewwith saved watchlists, resumable revision windows, source links, explicit coverage, and acknowledgment of a saved report. Fetching a newer revision never advances the reviewed baseline.The command saves captured revisions and source-aware event entries, pins each fetch to a source head, and rejects acknowledgment of incomplete, altered, or stale reports. Filters change the displayed view while acknowledgment applies to the complete captured span. A fictional offline sample exercises partial fetch, resume, acknowledgment, and an unchanged rerun.
Analysis and the explorer now retain requested bounds and coverage, avoid diffs across known gaps or unavailable text, and preserve parent/content-availability metadata through ingestion and SQLite. Passage history distinguishes exact, candidate, and absent latest matches; the MCP response uses the same result.
The README and guides lead with source review and inspectable observations. Captured examples are checked in CI.
docs/repositories.mddefines ownership and proposes moving the docs site, browser viewer, and Python adapter into separate subprojects in this repository, with independent builds, licenses, and deployment/install cutovers. No repository migration or archival is performed here.Validation
bun run ci: build, Biome, typecheck, boundaries, 1,109 passing tests (27 skipped), and six-package consumer pack/install check.bun run check:examples: offline onboarding, saved review reports, and pinned Bitcoin examples match their capture scripts.Release and scope
This remains an unreleased source-build feature; package versions are unchanged. Review observes article revisions, not talk-page or move-log events. The documentation distinguishes mechanical observations and captured-window coverage from downstream judgments. Public-docs changes should merge after this PR so their canonical guide links resolve.