Skip to content

feat: review overhaul + auto-update checker#5

Merged
Axect merged 6 commits intomainfrom
worktree-streamed-weaving-hollerith
Apr 7, 2026
Merged

feat: review overhaul + auto-update checker#5
Axect merged 6 commits intomainfrom
worktree-streamed-weaving-hollerith

Conversation

@Axect
Copy link
Copy Markdown
Owner

@Axect Axect commented Apr 7, 2026

Summary

  • Review template overhaul: Rewrote all 12 review prompts with expert-reviewer persona, added 3 new sections (Reproducibility, Impact & Significance, Reading Guide), publication-quality markdown rendering with metadata table, TOC, severity badges, and formal recommendation scoring
  • Auto-update checker: axp now checks for git updates on startup (throttled to 12h), warns about potential conflicts with locally modified files, and prompts user to pull

Test plan

  • All 190 tests pass (171 existing + 19 new for update checker)
  • Manual test: uv run axp daily with pending remote commits
  • Manual test: uv run axp review <id> produces enhanced review output
  • Manual test: conflict detection with locally modified files

🤖 Generated with Claude Code

Axect added 6 commits April 7, 2026 15:11
…ion-quality output

- Add 3 new review sections: Reproducibility Assessment, Impact & Significance, Reading Guide
- Rewrite all 12 existing prompts with senior reviewer persona, structured evaluation criteria,
  evidence requirements, and anti-patterns
- Richer JSON schemas: severity badges on weaknesses, novelty levels on contributions,
  formal recommendation (strong_accept..reject), confidence levels, structured assumptions
- Publication-quality markdown rendering: metadata table, table of contents, severity badges,
  inline tags, structured sub-sections, blockquote highlights
- Backward compatible with existing cached review data
- Add update_checker module: git fetch, compare HEAD vs remote, detect conflicts
- Prompt user with y/n when updates are available, show conflict warnings
- 12-hour throttling to avoid repeated fetches
- Integrate into @app.callback() with --no-update-check escape hatch
- 19 tests covering throttling, conflict detection, network failure
@Axect Axect merged commit 44b2047 into main Apr 7, 2026
4 checks passed
@Axect Axect deleted the worktree-streamed-weaving-hollerith branch April 7, 2026 07:16
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.

1 participant