Repository navigation
Conversation
orgtrack-graph and orgtrack-sync were never wired into the app. Drop them and the unused sync_export adapters that only fed that pipeline.
Collaborator
Author
|
Superseded by #1960, which stacks this commit with the other unused-code cleanups. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
orgtrack-graphandorgtrack-syncwere workspace members with no production consumer. The only callers weresync_exportadapters inorgtrack-coreandproject-management, which themselves had no production callers — only in-module tests. The unused crates still compiled and appeared in the workspace graph.Solution
Remove the two crates, drop
sync_exportfromorgtrack-coreandproject-management, and clear the corresponding workspace/Cargo.lockedges. Live orgtrack commands such asorgtrack_sync_core_repoare unchanged; they are a different path.Potential risks
If AI-Blame graph sync is revived, these crates must come back from git history. Local SQLite tables that
orgtrack-graphwould have created were never written by the app, so there is no production data to migrate.orgtrack-clinever depended onorgtrack_sync; only its publishing notes mentioned it.Verification
cargo check -p orgtrack_core -p project_management— finisheddevprofile in 2m 07sorgtrack_core,project_management— passedcargo test --workspace/ app binary rebuild (no runtime wiring existed)