Skip to content

feat: retry failed job with modified arguments#49

Merged
eclectic-coding merged 3 commits into
mainfrom
feat/retry-with-modified-arguments
May 21, 2026
Merged

feat: retry failed job with modified arguments#49
eclectic-coding merged 3 commits into
mainfrom
feat/retry-with-modified-arguments

Conversation

@eclectic-coding
Copy link
Copy Markdown
Owner

Summary

  • Adds an editable arguments textarea to the job detail page for failed jobs
  • Submitting the form updates job.arguments and calls execution.retry in one step
  • Invalid JSON redirects back to the detail page with an alert, leaving the failed execution untouched
  • All other job statuses continue to show the existing read-only <pre> display

Implementation

  • FailedJobs::ArgumentsController#update — new controller following the existing nested-resource pattern (failed_jobs/arguments)
  • Route: resource :arguments, only: [:update] nested inside resources :failed_jobs
  • sqd-textarea CSS class added to _07_forms.css matching existing input/select styling
  • 6 request specs; 100% line coverage maintained

Test plan

  • Visit a failed job's detail page — Arguments card shows an editable textarea pre-filled with the current JSON
  • Edit the JSON and click "Retry with these arguments" — redirects to /jobs/failed_jobs with a success notice; job appears in the ready queue with updated arguments
  • Submit malformed JSON — redirects back to the detail page with "Invalid JSON" alert; failed execution remains
  • Visit a non-failed job detail page — Arguments card still shows the read-only <pre>

🤖 Generated with Claude Code

eclectic-coding and others added 3 commits May 21, 2026 15:34
Adds an editable textarea to the Arguments card on the job detail page
for failed jobs. Submitting patches the job record then retries in one
step; invalid JSON redirects back with an alert without touching the
failed execution.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@eclectic-coding eclectic-coding merged commit f5e9ebc into main May 21, 2026
7 checks passed
@eclectic-coding eclectic-coding deleted the feat/retry-with-modified-arguments branch May 21, 2026 20:24
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