Repository navigation
v0.3.0: local mode, centered UI, loading states, explanation rendering, expanded license - #10
Merged
Merged
Conversation
- CHECKS_PENDING evidence now lists real check names instead of the state string - skip check-derived flags for not_applicable (local) analysis; adjust PASS reasoning - ignore '\ No newline at end of file' markers in changed_line_numbers - replace SystemExit string-matching with typed GitHubApiError in post_comment
- new 'diffly local' command triages git changes fully offline (private/removed repos) with --base branch comparison and untracked-file inclusion - center wizard, setup, interactive, doctor, and version screens in the terminal - add loading spinners with progress messaging during fetch phases - render generated explanations inside the interactive review as a toggleable section - prompt for Repository URL with format hints; accept pasted PR URLs without number - guard bare 'diffly' against non-TTY hangs; lazy termios import keeps Windows imports safe - add RepoRef type, -V/--version flag, python -m diffly_cli entry point - keep JSON output pure: suppress banners when --json is set - bump version to 0.3.0
- LICENSE: documented four-part license keeping the MIT grant with definitions, contribution terms, verdict disclaimers, extended warranty/liability, general terms - add CONTRIBUTING.md, SECURITY.md, CHANGELOG.md - move AUDIT_REPORT/visual-verification to docs/, capture scripts to scripts/ - document local mode and live example PRs in README
Diffly verdict: BLOCK#10 · 21 files · 1115 lines changed · checks: PENDING Risk flags: Risk flags and reasoning
Blast-radius summary
Deterministic triage is authoritative; any optional LLM explanation cannot change the verdict. |
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.
What this ships
Three commits, 46/46 tests passing (run three times), verified live against real GitHub PRs and real local repos.
New:
diffly local— offline triage for any folderdiffly localtriages uncommitted working-tree edits;diffly local --base maincompares a branchUX overhaul
owner/repoor paste any GitHub / PR URL)Fixed: explanations were invisible
The interactive view never rendered the generated explanation even when produced. It is now a toggleable section in the interactive screen, with clear setup guidance when no LLM key is configured.
Engine fixes
CHECKS_PENDINGevidence lists real check names (was the literal string "pending")\\ No newline at end of filemarkers no longer skew changed-line attributionGitHubApiErrorinstead of catchingSystemExitand string-matching error text--jsonLicense & community files
Verification evidence
pytest -q: 46 passed (three separate runs)