Skip to content

Commit b1b2833

Browse files
devonjonesclaude
andcommitted
fix: exclude deprecated entries from existing_data_map lookup
The existing_data_map was including ALL entries, so when there were two entries with the same full_name (current + deprecated), the last one would overwrite. After sorting deprecated entries last, this meant deprecated entries were overwriting current ones in the lookup map. This caused the scanner to compare against deprecated entries instead of current ones, incorrectly detecting MD5 'changes' for files that hadn't actually changed. Fixed by excluding deprecated entries when building the map - only current (non-deprecated) entries should be used for file comparison. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
1 parent f6d9660 commit b1b2833

1 file changed

Lines changed: 5 additions & 1 deletion

File tree

openforge/data/incremental.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,8 +65,12 @@ def __init__(
6565
self.verbose = verbose
6666
self.skip_schema_validation = skip_schema_validation
6767
self.existing_data = self._load_fixture()
68+
# Only map non-deprecated entries - deprecated entries shouldn't be
69+
# used for comparison since they represent old versions
6870
self.existing_data_map = {
69-
item["file_metadata"]["full_name"]: item for item in self.existing_data
71+
item["file_metadata"]["full_name"]: item
72+
for item in self.existing_data
73+
if not item.get("deprecated")
7074
}
7175
self.subset_path = self._detect_subset_path()
7276

0 commit comments

Comments
 (0)