Skip to content

prompt-tutor example: interval = 1 keeps the pending Review timer counting - #22

Merged
shayan-ys merged 1 commit into
mainfrom
docs/prompt-tutor-interval
Oct 12, 2026
Merged

shayan-ys merged 1 commit into
mainfrom
docs/prompt-tutor-interval

Conversation

@shayan-ys

Copy link
Copy Markdown
Owner

📝 Summary

This PR adds interval = 1 to the prompt-tutor integration example. With only watch, devdash reruns prompt-tutor --once when its data files change, so a pending Review's reviewing… Ns timer and spinner stayed frozen.


🛠️ Type of Change

  • 🐛 Bug fix (non-breaking change which fixes an issue)
  • ✨ New feature (non-breaking change which adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • ⚡ Performance / Refactor (code optimization without behavior changes)
  • 📚 Documentation update

🔍 Changes Made

  • README.md: the prompt-tutor example config adds interval = 1, and step 4 explains why.
  • config.example.toml: the commented prompt-tutor example adds # interval = 1, and its comment now explains the interval instead of saying to omit it.

🧪 How to Test

  1. Add interval = 1 to the prompt-tutor integration in your devdash config.
  2. Run devdash.
  3. Verify that the PROMPT TUTOR header shows fetched 0s ago · next fetch 1s and that a pending Review's timer counts up.

✅ Review and verification

  • Review path: low risk self-review; omitted optional reviews: none.
  • Verification: ran devdash with interval = 1 in a pty; the PROMPT TUTOR header showed fetched 0s ago · next fetch 1s. prompt-tutor --once takes about 40 ms per run.
  • Declined findings: none.
  • Remaining non-blocking findings: none.

@shayan-ys
shayan-ys marked this pull request as ready for review October 10, 2026 05:04
@shayan-ys
shayan-ys merged commit 3b18f52 into main Oct 12, 2026
6 checks passed
@shayan-ys
shayan-ys deleted the docs/prompt-tutor-interval branch October 12, 2026 13:56
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