Skip to content

fix: correct workspace license declaration to MIT OR Apache-2.0#5865

Merged
bug-ops merged 1 commit into
mainfrom
fix/5855-license-mismatch-zeph-bench
Jul 10, 2026
Merged

fix: correct workspace license declaration to MIT OR Apache-2.0#5865
bug-ops merged 1 commit into
mainfrom
fix/5855-license-mismatch-zeph-bench

Conversation

@bug-ops

@bug-ops bug-ops commented Jul 10, 2026

Copy link
Copy Markdown
Owner

Summary

  • Root Cargo.toml's [workspace.package] license field was "MIT", inherited by every crate via license.workspace = true. This was the actual bug: scripts/add-spdx-headers.sh and .github/scripts/add-spdx-headers.sh have stamped SPDX-License-Identifier: MIT OR Apache-2.0 onto every new .rs file since v0.12.0, and 1100+ existing source files across the workspace already carry that header — zeph-bench (the crate originally reported in license mismatch: workspace declares MIT, zeph-bench declares MIT OR Apache-2.0 #5855) was not the anomaly, the manifest was.
  • Corrected license to "MIT OR Apache-2.0" and added the missing LICENSE-APACHE file at the repo root.
  • Updated the root README.md and all 31 crate READMEs whose license badge/section still said plain MIT (excluding zeph-bench, which already correctly said MIT OR Apache-2.0) to match the corrected manifest.
  • crates/zeph-bench itself is untouched — its original SPDX headers and README were already correct.

Follow-up (out of scope here)

  • 5 pre-existing files under crates/zeph-worktree/src/ (error.rs, git_runner.rs, handle.rs, manager.rs, sanitize.rs) still carry plain-MIT SPDX headers while zeph-worktree/src/lib.rs says dual — an internal inconsistency that predates this issue and is a different defect class (not the cross-file "which do I read" contradiction license mismatch: workspace declares MIT, zeph-bench declares MIT OR Apache-2.0 #5855 reported). Will file a separate follow-up issue.

Test plan

  • cargo +nightly fmt --check
  • cargo clippy --profile ci --workspace --all-targets --features "desktop,ide,server,chat,pdf,scheduler,testing" -- -D warnings
  • cargo nextest run --config-file .github/nextest.toml --workspace --features "desktop,ide,server,chat,pdf,scheduler" --lib --bins — 12374 passed, 35 skipped
  • RUSTFLAGS="-D warnings" RUSTDOCFLAGS="--deny rustdoc::broken_intra_doc_links" cargo doc --no-deps --workspace --features "desktop,ide,server,chat,pdf,scheduler"
  • cargo metadata confirms all 33 workspace crates report license: "MIT OR Apache-2.0"
  • .github/deny.toml license allowlist confirmed compatible

Closes #5855

@github-actions github-actions Bot added documentation Improvements or additions to documentation llm zeph-llm crate (Ollama, Claude) skills zeph-skills crate memory zeph-memory crate (SQLite) channels zeph-channels crate (Telegram) rust Rust code changes core zeph-core crate dependencies Dependency updates bug Something isn't working size/L Large PR (201-500 lines) labels Jul 10, 2026
@bug-ops
bug-ops enabled auto-merge (squash) July 10, 2026 14:00
@bug-ops
bug-ops force-pushed the fix/5855-license-mismatch-zeph-bench branch from 95735f6 to 89dba52 Compare July 10, 2026 14:21
Cargo.toml's workspace.package.license was set to "MIT" while
scripts/add-spdx-headers.sh has stamped "MIT OR Apache-2.0" onto every
new source file since v0.12.0, and 1100+ existing files already carry
that SPDX header. The manifest, README badges, and per-crate READMEs
were the odd ones out, not the source headers. Add LICENSE-APACHE and
align every README's license badge/section with the manifest.

Closes #5855
@bug-ops
bug-ops force-pushed the fix/5855-license-mismatch-zeph-bench branch from 89dba52 to 3e920e9 Compare July 10, 2026 14:31
@bug-ops
bug-ops merged commit 9d255b3 into main Jul 10, 2026
47 checks passed
@bug-ops
bug-ops deleted the fix/5855-license-mismatch-zeph-bench branch July 10, 2026 14:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working channels zeph-channels crate (Telegram) core zeph-core crate dependencies Dependency updates documentation Improvements or additions to documentation llm zeph-llm crate (Ollama, Claude) memory zeph-memory crate (SQLite) rust Rust code changes size/L Large PR (201-500 lines) skills zeph-skills crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

license mismatch: workspace declares MIT, zeph-bench declares MIT OR Apache-2.0

1 participant