Skip to content

Add configurable commit lookback#455

Merged
ashleyredzko merged 3 commits into
Jellyfish-AI:masterfrom
rikwasmus:configurable-commit-lookback
May 28, 2026
Merged

Add configurable commit lookback#455
ashleyredzko merged 3 commits into
Jellyfish-AI:masterfrom
rikwasmus:configurable-commit-lookback

Conversation

@rikwasmus
Copy link
Copy Markdown
Contributor

Summary

  • Add a commit_lookback_days git config option for commit ingestion.
  • Keep the existing default behaviour of 31 days when the option is omitted.
  • Pass the configured value through GitHub, GitHub GraphQL, GitLab, Bitbucket Cloud, and Bitbucket Server commit collection paths.

Motivation

For large installations that run the agent frequently, always refetching the last 31 days of commits can make each run much longer than necessary. This lets customers tune the steady-state commit lookback window without changing backpopulation behaviour or PR ingestion.

Testing

  • uv run pytest tests/test_config_file.py tests/test_github.py tests/test_bitbucket_cloud_adapter.py tests/test_bitbucket_server.py tests/test_gitlab_adapter.py
  • uv run pre-commit run black --all-files
  • uv run pre-commit run detect-secrets --all-files
  • uv run pre-commit run isort --files jf_agent/config_file_reader.py jf_agent/git/__init__.py jf_agent/git/bitbucket_cloud_adapter.py jf_agent/git/bitbucket_server.py jf_agent/git/github.py jf_agent/git/github_gql_adapter.py jf_agent/git/gitlab_adapter.py tests/test_config_file.py
  • uv run pre-commit run hadolint --all-files

@rikwasmus
Copy link
Copy Markdown
Contributor Author

Added reference to this to https://help.jellyfish.co/hc/en-us/p/feedback-portal as well.

@rikwasmus rikwasmus changed the title CDE-8931: Add configurable commit lookback Add configurable commit lookback May 21, 2026
Copy link
Copy Markdown
Contributor

@ashleyredzko ashleyredzko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for submitting this change @rikwasmus ! This looks really useful for customers with large amounts of data and who don't need such a long lookback for commits. This change looks good to me overall!

Since this is adding an additional configuration option, please add an example of this (using the default 31) to the example.yml file for other customers who might find this configuration helpful in the future.

Once that example is added we'll leave an approval and get this in for use!

Feel free to ignore the failing Endor Labs job, we're working out some kinks with that action wrt external pull requests.

@sonarqubecloud
Copy link
Copy Markdown

@rikwasmus
Copy link
Copy Markdown
Contributor Author

Thanks for considering @ashleyredzko
example.yaml update added to the PR.

@ashleyredzko ashleyredzko merged commit 3688219 into Jellyfish-AI:master May 28, 2026
6 of 7 checks passed
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.

2 participants