Skip to content

feat(ingestion): resolve commit baseline context on sync start (#660)#691

Merged
aredenba-rh merged 1 commit into
feature/manage-knowledge-graphfrom
feat/issue-660-ingestion-baseline-resolver
May 20, 2026
Merged

feat(ingestion): resolve commit baseline context on sync start (#660)#691
aredenba-rh merged 1 commit into
feature/manage-knowledge-graphfrom
feat/issue-660-ingestion-baseline-resolver

Conversation

@aredenba-rh
Copy link
Copy Markdown
Collaborator

Summary

  • enrich SyncStarted ingestion handling with Git-backed context preparation before extraction starts
  • resolve baseline commit from data-source commit references and pass it into ingestion as checkpoint state
  • refresh tracked branch head for GitHub sources and inject tenant-scoped credentials for branch-head resolution/adapter execution

Testing

  • uv run pytest tests/unit/ingestion/application/test_ingestion_service.py tests/unit/ingestion/infrastructure/test_ingestion_event_handler.py tests/unit/test_sessioned_ingestion_handler.py -q
  • uv run pytest tests/unit/test_application_lifecycle.py tests/unit/ingestion/test_architecture.py -q

Risks

  • tracked branch refresh currently targets GitHub sources only (other adapters remain no-op until implemented)

Closes #660

Made with Cursor

Prepare Git-backed ingestion context by loading data-source commit references,
refreshing tracked branch head, and passing baseline commit plus resolved
credentials into the ingestion pipeline before packaging begins.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented May 20, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: ASSERTIVE

Plan: Enterprise

Run ID: 7763d626-dc93-428a-9afc-b3ea34e5d848

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/issue-660-ingestion-baseline-resolver

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

@aredenba-rh aredenba-rh merged commit debc505 into feature/manage-knowledge-graph May 20, 2026
1 check was pending
@aredenba-rh aredenba-rh deleted the feat/issue-660-ingestion-baseline-resolver branch May 20, 2026 14:15
aredenba-rh added a commit that referenced this pull request May 26, 2026
Prepare Git-backed ingestion context by loading data-source commit references,
refreshing tracked branch head, and passing baseline commit plus resolved
credentials into the ingestion pipeline before packaging begins.

Co-authored-by: Cursor <cursoragent@cursor.com>
# Conflicts:
#	src/api/ingestion/application/services/ingestion_service.py
#	src/api/ingestion/infrastructure/event_handler.py
#	src/api/ingestion/ports/services.py
#	src/api/tests/unit/ingestion/infrastructure/test_ingestion_event_handler.py
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