Skip to content

fix(github_action): handle synchronize event for push trigger (#2455) - #1

Open
utsab345 wants to merge 2 commits into
mainfrom
fix/scoped-ticket-cache
Open

utsab345 wants to merge 2 commits into
mainfrom
fix/scoped-ticket-cache

Conversation

@utsab345

@utsab345 utsab345 commented Jul 2, 2026

Copy link
Copy Markdown
Owner

Handles the synchronize event in the GitHub Action runner so that handle_push_trigger works for GitHub Actions workflows, not just the GitHub App.

When a PR is updated (new commits pushed), the synchronize event is now checked against github_app.handle_push_trigger. If enabled, the configured push_commands (default: /describe, /review) are executed.

Closes The-PR-Agent#2455

- Skip caching entirely when PR URL is unavailable (avoid global key fallback)
- Add cache_tickets to .pr_agent.toml to mirror configuration.toml
- Wrap overlong cache_key assignment to stay within 120-char line limit
- Update PRReviewer to read PR-scoped cache key for accurate token budgeting
@utsab345
utsab345 force-pushed the fix/scoped-ticket-cache branch from 55a15e8 to fc8ea8c Compare July 2, 2026 05:54
…nt#2383)

- Add cache_tickets check to _load_related_tickets (prevent cache bypass)
- Cache empty extraction results too (avoid repeated API calls)
- Replace Dynaconf cache with in-memory dict + 1h TTL (bounded cache)
- Update tests to use new in-memory cache
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.

synchronize event should be handled for github without app as well

1 participant