Skip to content

fix: pass --interactive flag to aidevops-update-check.sh in AGENTS.md greeting#2557

Merged
marcusquinn merged 1 commit intomainfrom
bugfix/update-check-interactive
Feb 28, 2026
Merged

fix: pass --interactive flag to aidevops-update-check.sh in AGENTS.md greeting#2557
marcusquinn merged 1 commit intomainfrom
bugfix/update-check-interactive

Conversation

@marcusquinn
Copy link
Owner

@marcusquinn marcusquinn commented Feb 28, 2026

Summary

  • Pass --interactive flag to aidevops-update-check.sh in both AGENTS.md generation sources so the update check actually runs in AI assistant sessions instead of being silently skipped by headless detection

Changes

  • .agents/scripts/generate-opencode-agents.sh:42 — add --interactive to the update check command in the heredoc that generates ~/.config/Claude/AGENTS.md
  • templates/opencode-config-agents.md:6 — same change in the OpenCode config template

Root Cause

is_headless() returns true when [[ ! -t 0 ]] (no TTY on stdin). Every AI coding assistant runs Bash via piped stdin, so the update check was always skipped. The --interactive flag already exists in the script (line 105-110) and is documented for exactly this purpose — it just was never passed.

Closes #2554

Summary by CodeRabbit

  • Improvements
    • Update checks now support interactive mode, allowing users to directly interact with the update process.

… greeting

The update check script's is_headless() detects no TTY on stdin (always true
when called from AI assistant Bash tools) and skips the network call. The
--interactive flag already exists to override this for exactly this use case,
but AGENTS.md never passed it. Add the flag in both source locations.

Closes #2554
@gemini-code-assist
Copy link

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added the bug Auto-created from TODO.md tag label Feb 28, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 28, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 37846f8 and ba2d2e4.

📒 Files selected for processing (2)
  • .agents/scripts/generate-opencode-agents.sh
  • templates/opencode-config-agents.md

Walkthrough

Two files are updated to add the --interactive flag when invoking the update-check script, enabling the existing headless-detection override that the script already supports. This resolves the issue where update availability was never detected when called from interactive AI assistant sessions.

Changes

Cohort / File(s) Summary
Update Check Interactive Flag
.agents/scripts/generate-opencode-agents.sh, templates/opencode-config-agents.md
Added --interactive flag to aidevops-update-check.sh invocation to override headless detection and enable update checks in piped Bash tool contexts.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

bug

Poem

🔧 A silent sentinel now speaks,
With --interactive flag in hand,
Updates once hidden from eager seekers
Now float through the AI's command,
No more headless detection deceiving—
The greeting flow finally believes. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the primary change: adding the --interactive flag to aidevops-update-check.sh invocations in AGENTS.md greeting.
Linked Issues check ✅ Passed The PR fully implements the required fix from issue #2554: passing --interactive flag to aidevops-update-check.sh in both locations (generate-opencode-agents.sh and opencode-config-agents.md template).
Out of Scope Changes check ✅ Passed All changes are directly scoped to the linked issue #2554: only the --interactive flag addition to the two AGENTS.md greeting sources, with no extraneous modifications.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bugfix/update-check-interactive

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

🔍 Code Quality Report

�[0;35m[MONITOR]�[0m Code Review Monitoring Report

�[0;34m[INFO]�[0m Latest Quality Status:
SonarCloud: 0 bugs, 0 vulnerabilities, 103 code smells

�[0;34m[INFO]�[0m Recent monitoring activity:
Sat Feb 28 02:03:18 UTC 2026: Code review monitoring started
Sat Feb 28 02:03:18 UTC 2026: SonarCloud - Bugs: 0, Vulnerabilities: 0, Code Smells: 103

📈 Current Quality Metrics

  • BUGS: 0
  • CODE SMELLS: 103
  • VULNERABILITIES: 0

Generated on: Sat Feb 28 02:03:21 UTC 2026


Generated by AI DevOps Framework Code Review Monitoring

@sonarqubecloud
Copy link

@marcusquinn marcusquinn merged commit 4d3aa35 into main Feb 28, 2026
25 checks passed
@marcusquinn marcusquinn deleted the bugfix/update-check-interactive branch March 3, 2026 03:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Auto-created from TODO.md tag

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: AGENTS.md update check never detects updates (missing --interactive flag)

1 participant