fix: CI checks before release + primer packaging fix

- Add CI job (fmt, clippy, check, test) to release workflow
- Build job now depends on CI checks passing
- Fix primer.defaults.json packaging for crates.io via build.rs
- Bump version to 0.5.2

Author: ddunnock

.github/workflows/release.yml

@@ -11,8 +11,35 @@ env:
   CARGO_TERM_COLOR: always
 
 jobs:
+  ci:
+    name: CI Checks
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v4
+        with:
+          submodules: recursive
+
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: rustfmt, clippy
+
+      - uses: Swatinem/rust-cache@v2
+
+      - name: Check formatting
+        run: cargo fmt --all -- --check
+
+      - name: Clippy
+        run: cargo clippy --all-features -- -D warnings
+
+      - name: Check
+        run: cargo check --all-features
+
+      - name: Test
+        run: cargo test --all-features
+
   build:
     name: Build ${{ matrix.target }}
+    needs: ci
     strategy:
       fail-fast: false
       matrix:

CHANGELOG.md

@@ -7,7 +7,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
-## [0.5.1] - 2025-12-30
+## [0.5.2] - 2025-12-30
+
+### Fixed
+- Fixed crates.io packaging: primer.defaults.json now included via build.rs
+- Fixed code formatting issues
+
+### Infrastructure
+- Release workflow now runs CI checks (fmt, clippy, test) before building binaries
+
+## [0.5.1] - 2025-12-30 [YANKED]
+
+Yanked due to formatting check failure in CI.
 
 ### Added
 - **RFC-0004: Full primer implementation** with value-based section selection

Cargo.toml

@@ -7,7 +7,7 @@
 # @acp:summary "Package metadata and publishing information"
 [package]
 name = "acp-protocol"
-version = "0.5.1"
+version = "0.5.2"
 edition = "2021"
 authors = ["ACP Contributors"]
 description = "AI Context Protocol - Token-efficient and context enhancing code documentation for AI systems"

build.rs

@@ -8,13 +8,12 @@ use std::path::Path;
 
 fn copy_json_files(src_dir: &Path, dst_dir: &Path) {
     if src_dir.exists() {
-        fs::create_dir_all(dst_dir).unwrap_or_else(|e| {
-            panic!("Failed to create {:?} directory: {}", dst_dir, e)
-        });
+        fs::create_dir_all(dst_dir)
+            .unwrap_or_else(|e| panic!("Failed to create {:?} directory: {}", dst_dir, e));
 
-        for entry in fs::read_dir(src_dir).unwrap_or_else(|e| {
-            panic!("Failed to read {:?}: {}", src_dir, e)
-        }) {
+        for entry in
+            fs::read_dir(src_dir).unwrap_or_else(|e| panic!("Failed to read {:?}: {}", src_dir, e))
+        {
             let entry = entry.expect("Failed to read directory entry");
             let src_path = entry.path();
 
@@ -30,16 +29,10 @@ fn copy_json_files(src_dir: &Path, dst_dir: &Path) {
 
 fn main() {
     // Copy schema files
-    copy_json_files(
-        Path::new("acp-spec/schemas/v1"),
-        Path::new("schemas/v1"),
-    );
+    copy_json_files(Path::new("acp-spec/schemas/v1"), Path::new("schemas/v1"));
 
     // Copy primer files
-    copy_json_files(
-        Path::new("acp-spec/primers"),
-        Path::new("primers"),
-    );
+    copy_json_files(Path::new("acp-spec/primers"), Path::new("primers"));
 
     // Tell Cargo to rerun build.rs if submodule files change
     println!("cargo:rerun-if-changed=acp-spec/schemas/v1");