Skip to content

feat: store cli_args and include in prompt for local session refresh#102

Merged
tomasz-tomczyk merged 1 commit intomainfrom
feat/auto-refresh-local-after-web-feedback
Apr 24, 2026
Merged

feat: store cli_args and include in prompt for local session refresh#102
tomasz-tomczyk merged 1 commit intomainfrom
feat/auto-refresh-local-after-web-feedback

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Adds cli_args column (string array) to reviews table via migration
  • API controller extracts and passes cli_args through on create and update
  • upsert_review persists cli_args even when file content hasn't changed
  • Local prompt appends "Then run crit <args> to refresh the local review session" when cli_args are present

Companion PR: tomasz-tomczyk/crit#349

Closes tomasz-tomczyk/crit#342

Test plan

  • API controller tests: cli_args in create (multi-file, single-file, default empty) and update (content changed, content unchanged)
  • LiveView tests: prompt includes cli_args when present, omits when absent
  • All 427 existing tests pass
  • Share integration tests pass (make e2e-share)

🤖 Generated with Claude Code

…fresh

The crit CLI now sends cli_args (the original arguments passed to `crit`)
in POST/PUT /api/reviews payloads. crit-web stores them on the review
and appends a "then run `crit <args>`" instruction to the local prompt,
so the agent knows to refresh the local review session after acting on
web feedback.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tomasz-tomczyk tomasz-tomczyk merged commit 0a3af94 into main Apr 24, 2026
2 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the feat/auto-refresh-local-after-web-feedback branch April 24, 2026 16:17
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.

Auto-refresh local review session after agent acts on web feedback

1 participant