Skip to content

(feat/change-tracking) Change Tracking Modes#1445

Merged
nickscamara merged 9 commits intomainfrom
devin/1744423210-add-git-diff-to-change-tracking
Apr 12, 2025
Merged

(feat/change-tracking) Change Tracking Modes#1445
nickscamara merged 9 commits intomainfrom
devin/1744423210-add-git-diff-to-change-tracking

Conversation

@devin-ai-integration
Copy link
Contributor

Implements git diffs for the change tracking format feature. The change tracking response now includes both text and structured object diff representations when both changeTracking format and changeTracking@diff-git are specified in the formats array.

Added corresponding tests to verify the functionality.

Link to Devin run: https://app.devin.ai/sessions/11a60f0860e844a489e3c1e636450d9f

Requested by: Nicolas Camara

@devin-ai-integration devin-ai-integration bot requested a review from mogery as a code owner April 12, 2025 02:03
@devin-ai-integration
Copy link
Contributor Author

devin-ai-integration bot commented Apr 12, 2025

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@devin-ai-integration
Copy link
Contributor Author

The CI failures in the crawl tests appear to be unrelated to this PR's changes, which only affect the change tracking format feature. The git diff implementation has been tested locally and works correctly with both text and structured object representations.

devin-ai-integration bot and others added 2 commits April 12, 2025 06:22
* Refactor change tracking to use modes array instead of separate formats

Co-Authored-By: Nicolas Camara <[email protected]>

* Implement schema-based change tracking with old/new value comparison

Co-Authored-By: Nicolas Camara <[email protected]>

* Nick:

* Nick: .json

* Update diff.ts

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-authored-by: Nicolas Camara <[email protected]>
Co-authored-by: Nicolas <[email protected]>
@nickscamara nickscamara changed the title Add git-diff support to change tracking format (feat/change-tracking) Change Tracking Modes Apr 12, 2025
Copy link
Member

@mogery mogery left a comment

Choose a reason for hiding this comment

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

this is great!

@devin-ai-integration
Copy link
Contributor Author

The field was added as part of the structured output/prompt support for change tracking in a later commit (2da991b). It's intentional and needed for the enhanced change tracking functionality that supports different modes including 'git-diff'.

@devin-ai-integration
Copy link
Contributor Author

The changeTrackingOptions field was added as part of the structured output/prompt support for change tracking in a later commit (2da991b). It's intentional and needed for the enhanced change tracking functionality that supports different modes including 'git-diff'.

@nickscamara nickscamara merged commit 138a975 into main Apr 12, 2025
0 of 10 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