Skip to content

fix: align provider routing and archive export safety#590

Open
AndrewDongminYoo wants to merge 2 commits into
nashsu:mainfrom
AndrewDongminYoo:fix/provider-routing-and-export-safety
Open

fix: align provider routing and archive export safety#590
AndrewDongminYoo wants to merge 2 commits into
nashsu:mainfrom
AndrewDongminYoo:fix/provider-routing-and-export-safety

Conversation

@AndrewDongminYoo

Copy link
Copy Markdown
Contributor

Summary

  • Keep task routing, project overrides, and API chat on the same current provider configuration without persisting API keys in task routing.
  • Stop saved task routes from bypassing the global provider-disabled state, including clip ingestion and API chat.
  • Resolve archive export destinations before checking project containment, preventing lexical traversal and symlink escapes.

Validation

  • npx vitest run src/lib/llm-task-routing.test.ts
  • npx --yes -p typescript@5.7.3 tsc --build --pretty
  • CARGO_TARGET_DIR=/Volumes/donminzzi/Development/01_personal/llm_wiki/.codex/cache/provider-routing-cargo-target cargo test --manifest-path src-tauri/Cargo.toml --lib (341 passed, 1 ignored)

The full Vitest suite remains blocked in this isolated worktree because upstream lockfile/package metadata cannot install cleanly, while linked dependencies expose unrelated YAML/frontmatter and generated MCP test failures.

@AndrewDongminYoo
AndrewDongminYoo marked this pull request as ready for review July 16, 2026 13:01
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.

1 participant