Skip to content

refactor: exclude unused crates from workspace, fix flaky tests#631

Merged
AlexMikhalev merged 1 commit intomainfrom
cleanup/phase-2-dead-crates
Mar 5, 2026
Merged

refactor: exclude unused crates from workspace, fix flaky tests#631
AlexMikhalev merged 1 commit intomainfrom
cleanup/phase-2-dead-crates

Conversation

@AlexMikhalev
Copy link
Contributor

Summary

  • Exclude 7 unused crates from workspace builds (keep dirs for future use):
    • terraphim_build_args, terraphim-markdown-parser (planned)
    • haystack_atlassian, haystack_discourse, haystack_jmap, haystack_grepapp (unused providers)
    • terraphim_repl (superseded by terraphim_agent)
  • Comment out grepapp_haystack path dep in terraphim_middleware
  • Remove 10 nested crates/*/crates/terraphim_settings/ duplicate dirs
  • Remove terraphim-repl references from release-minimal.yml
  • Fix 2 flaky tests:
    • test_save_one_memory: handle global storage singleton race
    • test_circuit_breaker_half_open_after_cooldown: increase timing margin

Test plan

  • cargo check --workspace passes
  • cargo test --workspace --lib passes (all tests green)
  • cargo clippy --workspace clean
  • Pre-commit hooks pass

Closes #622, #623, #624

Generated with Terraphim AI

Workspace cleanup:
- Exclude from workspace builds (keep directories for future use):
  terraphim_build_args, terraphim-markdown-parser (planned future),
  haystack_atlassian, haystack_discourse, haystack_jmap, haystack_grepapp
  (unused providers), terraphim_repl (superseded by terraphim_agent)
- Comment out grepapp_haystack path dep in terraphim_middleware
- Remove 10 nested crates/*/crates/terraphim_settings/ duplicate dirs
- Update .release-plz.toml for excluded crates
- Remove terraphim-repl references from release-minimal.yml

Test fixes:
- Fix test_save_one_memory in terraphim_config: handle global storage
  singleton already initialized by other tests with different profiles
- Fix test_circuit_breaker_half_open_after_cooldown in terraphim_spawner:
  increase sleep margin from 5ms to 50ms to prevent flaking under load

Refs: #622, #623, #624

Co-Authored-By: Terraphim AI <noreply@anthropic.com>
@AlexMikhalev AlexMikhalev merged commit 0fa54ba into main Mar 5, 2026
26 of 32 checks passed
@AlexMikhalev AlexMikhalev deleted the cleanup/phase-2-dead-crates branch March 5, 2026 23:08
AlexMikhalev added a commit that referenced this pull request Mar 5, 2026
Removes unused mockall dependency from desktop/src-tauri/Cargo.toml. Flaky test fixes already landed via #631.
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.

[cleanup] Phase 2a: Exclude future-use crates from workspace builds

1 participant