ShadowsOverWestgate/sow-tools
7
0
Fork 0
Code Issues 2 Pull Requests Actions Packages Projects Releases 38 Activity

Skip area version-only extract churn #28

Merged
archvillainette merged 1 commits from preserve-area-version-on-extract into main 2026-07-01 10:02:14 +00:00
Conversation 1 Commits 1 Files Changed 2
+142
archvillainette commented 2026-07-01 09:57:15 +00:00
Owner
Copy Link
Copy Source

Summary

  • skip overwriting existing .are.json files when only the root GFF Version differs
  • keep real extraction changes writing normally
  • add regression coverage for version-only area extraction

Test Plan

  • nix develop --command make check
  • sample extraction in a clean temp sow-module clone using the patched crucible binary
## Summary - skip overwriting existing `.are.json` files when only the root GFF `Version` differs - keep real extraction changes writing normally - add regression coverage for version-only area extraction ## Test Plan - `nix develop --command make check` - sample extraction in a clean temp `sow-module` clone using the patched `crucible` binary
archvillainette added 1 commit 2026-07-01 09:57:15 +00:00
fix(module): skip area version-only extract churn
build-binaries / build-binaries (pull_request) Successful in 2m23s
Details
test-image / build-image (pull_request) Successful in 37s
Details
test / test (pull_request) Successful in 1m26s
Details
55e595b998
xtul approved these changes 2026-07-01 10:02:01 +00:00
archvillainette merged commit bf787a6458 into main 2026-07-01 10:02:14 +00:00
archvillainette deleted branch preserve-area-version-on-extract 2026-07-01 10:02:14 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
xtul
Labels
Clear labels
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
Chass FallenDabus WinterZen XeroTwo Zariu archvillainette gitea-bot xtul
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ShadowsOverWestgate/sow-tools#28