Skip to content

docs(cron): keep PR screening prompt concise#819

Closed
shaun-agent wants to merge 3 commits into
mainfrom
fix/cron-long-trigger
Closed

docs(cron): keep PR screening prompt concise#819
shaun-agent wants to merge 3 commits into
mainfrom
fix/cron-long-trigger

Conversation

@shaun-agent
Copy link
Copy Markdown
Contributor

@shaun-agent shaun-agent commented May 14, 2026

Updates the cron docs with a compact OpenAB PR-screening prompt that stays under Discord's 2,000 character message limit.\n\nThis replaces the earlier generic cron-code approach: the correct fix is to keep the scheduled instruction concise, because cron posts the configured message into Discord before the agent runs.\n\nPrompt length: 855 chars before the cron sender prefix.\n\nLive deployment note: the running pod's /etc/openab/config.toml is mounted read-only, and the pod service account cannot patch ConfigMaps/Helm releases from inside the pod. Apply this concise prompt in the deployed Helm values/config and restart the OpenAB pod.

@shaun-agent shaun-agent requested a review from thepagent as a code owner May 14, 2026 12:49
@github-actions
Copy link
Copy Markdown

⚠️ This PR is missing a Discord Discussion URL in the body.

All PRs must reference a prior Discord discussion to ensure community alignment before implementation.

Please edit the PR description to include a link like:

Discord Discussion URL: https://discord.com/channels/...

This PR will be automatically closed in 3 days if the link is not added.

@github-actions github-actions Bot added closing-soon PR missing Discord Discussion URL — will auto-close in 3 days pending-screening PR awaiting automated screening labels May 14, 2026
@shaun-agent shaun-agent changed the title fix(cron): truncate visible trigger messages docs(cron): keep PR screening prompt concise May 14, 2026
@shaun-agent
Copy link
Copy Markdown
Contributor Author

Closing: this should be fixed in the deployed workflow prompt/config, not as a repo documentation PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

closing-soon PR missing Discord Discussion URL — will auto-close in 3 days pending-screening PR awaiting automated screening

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant