Skip to content

fix: split language package layout#78

Merged
konard merged 3 commits into
mainfrom
issue-77-388911d8eea6
May 12, 2026
Merged

fix: split language package layout#78
konard merged 3 commits into
mainfrom
issue-77-388911d8eea6

Conversation

@konard
Copy link
Copy Markdown
Member

@konard konard commented May 12, 2026

Fixes #77

Summary

  • Move the WebAssembly Rust wrapper crate from the repository root into rust/wasm/, including Cargo.toml, Cargo.lock, src/, and tests/.
  • Split release automation by package: C# Node scripts now live in csharp/scripts/; Rust automation now lives in rust/scripts/ as rust-script scripts with crates.io publish/release checks.
  • Update C#, Rust, and WebAssembly workflows, npm scripts, docs, package READMEs, registry/release badges, and release metadata for the split layout.
  • Preserve issue/PR/template/CI evidence in docs/case-studies/issue-77/.

Reproduction and Regression Coverage

  • Added web/test/repositoryLayout.test.mjs, which fails on the old layout by requiring root Cargo.toml, Cargo.lock, scripts/, src/, and tests/ to be absent and requiring csharp/scripts/, rust/scripts/, and rust/wasm/ to exist.
  • The regression also checks the relocated WebAssembly npm commands and the corrected wasm-pack test --node rust/wasm argument order.

Verification

  • node --test web/test/repositoryLayout.test.mjs
  • npm run test:js
  • npm run test:wasm
  • npm run build
  • npm test
  • cargo fmt --manifest-path rust/Cargo.toml --all -- --check
  • cargo fmt --manifest-path rust/wasm/Cargo.toml --all -- --check
  • rustfmt rust/scripts/*.rs --check
  • cargo test --manifest-path rust/Cargo.toml --all-features
  • cargo test --manifest-path rust/wasm/Cargo.toml --all-features
  • cargo clippy --manifest-path rust/Cargo.toml --all-targets --all-features
  • cargo package --manifest-path rust/Cargo.toml --list --allow-dirty
  • rust-script rust/scripts/get-version.rs --rust-root rust
  • rust-script rust/scripts/get-bump-type.rs --rust-root rust
  • rust-script rust/scripts/check-file-size.rs
  • rust-script rust/scripts/check-changelog-fragment.rs
  • rust-script --test rust/scripts/check-file-size.rs
  • rust-script --test rust/scripts/create-github-release.rs
  • dotnet test csharp/Foundation.Data.Doublets.Cli.sln --configuration Release
  • git diff --cached --check

Notes

  • This is not a UI change, so no screenshots are included.
  • Existing stale CI failures investigated for this issue are preserved under docs/case-studies/issue-77/evidence/; latest main runs were passing before this branch change.

Adding .gitkeep for PR creation (default mode).
This file will be removed when the task is complete.

Issue: #77
@konard konard self-assigned this May 12, 2026
@konard konard changed the title [WIP] Fixes and improvements fix: split language package layout May 12, 2026
@konard konard marked this pull request as ready for review May 12, 2026 11:58
@konard
Copy link
Copy Markdown
Member Author

konard commented May 12, 2026

Working session summary

Implemented and pushed the fix for issue 77.

PR 78 is ready: #78
Head commit: b4b0f02 (fix: split language package layout)

What changed:

  • Moved the WASM crate from root into rust/wasm/.
  • Split release scripts into csharp/scripts/ and rust/scripts/.
  • Updated C#, Rust, and WebAssembly workflows and package scripts.
  • Added package READMEs, badges, changelog/changeset entries, and issue evidence.
  • Added web/test/repositoryLayout.test.mjs to prevent root Cargo.toml, scripts/, src/, or tests/ from coming back.

Verification:

  • Local JS, WASM, Rust, C#, rust-script, package, and build checks passed.
  • Fresh PR CI passed for C#, Rust, and WebAssembly.
  • PR is no longer draft and merge state is clean.
  • Working tree is clean.

This summary was automatically extracted from the AI working session output.

@konard
Copy link
Copy Markdown
Member Author

konard commented May 12, 2026

🤖 Solution Draft Log

This log file contains the complete execution trace of the AI solution draft process.

💰 Cost estimation:

  • Model: GPT-5.5
  • Provider: OpenAI
  • Public pricing estimate: $25.256997

📊 Context and tokens usage:

  • 541.3K / 1.1M (52%) input tokens, 61.8K / 128K (48%) output tokens

Total: (541.3K + 17.1M cached) input tokens, 61.8K output tokens, $25.256997 cost

🤖 Models used:

  • Tool: OpenAI Codex
  • Requested: gpt-5.5
  • Model: GPT-5.5 (gpt-5.5)

📎 Log file uploaded as Repository (86485KB)


Now working session is ended, feel free to review and add any feedback on the solution draft.

@konard
Copy link
Copy Markdown
Member Author

konard commented May 12, 2026

✅ Ready to merge

This pull request is now ready to be merged:

  • All CI checks have passed
  • No merge conflicts
  • No pending changes

Monitored by hive-mind with --auto-restart-until-mergeable flag

@konard konard merged commit f942d30 into main May 12, 2026
20 checks passed
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.

Fixes and improvements

1 participant