Skip to content

Separate JavaScript and Rust package roots - #161

Merged
konard merged 6 commits into
mainfrom
issue-160-a2a2313cf693
Jun 8, 2026
Merged

Separate JavaScript and Rust package roots#161
konard merged 6 commits into
mainfrom
issue-160-a2a2313cf693

Conversation

@konard

@konard konard commented Jun 8, 2026

Copy link
Copy Markdown
Member

Summary

Fixes #160.

  • split JavaScript package metadata, release scripts, docs, lockfiles, and Changesets state into js/
  • added Rust-specific README, changelog, release scripts, and rust.yml CI/release workflow under rust/
  • replaced the combined release workflow with separate js.yml and rust.yml workflows using js-v and rust-v release tag prefixes
  • documented and tested that the former root scripts/ directory was fully removed, with JS scripts under js/scripts/ and Rust scripts under rust/scripts/
  • added docs/case-studies/issue-160/ with issue/PR data, template snapshots, verification logs, and implementation notes
  • fixed Windows Rust test portability found in PR CI (resolve_path absolute paths and streaming cwd)
  • reported an upstream Rust template follow-up: Release script should remove consumed changelog fragments rust-ai-driven-development-pipeline-template#65

Testing

  • bun test js/tests/repository-layout.test.mjs
  • cd js && bun run lint
  • cd js && bun run format:check
  • ./js/node_modules/.bin/prettier --config js/.prettierrc --check docs/case-studies/issue-160/README.md js/tests/repository-layout.test.mjs
  • cd js && bun run check:duplication
  • cd js && PATH=/tmp/issue-160-bin:$PATH bun run test (684 pass, 5 skip)
  • cd rust && cargo fmt --all -- --check
  • cd rust && cargo clippy --all-targets --all-features
  • cd rust && cargo test --all-features --verbose
  • cd rust && cargo test --doc --all-features --verbose
  • cd rust && cargo package --allow-dirty
  • rust-script --test rust/scripts/publish-crate.rs
  • rust-script --test rust/scripts/version-and-commit.rs
  • Ruby YAML parse for .github/workflows/js.yml and .github/workflows/rust.yml
  • PR CI on 82c1a5a: JavaScript checks and release passed; Rust checks and release passed

Notes

Local sudo was unavailable, so the full JavaScript suite used a temporary downloaded jq binary, matching the CI job's jq dependency setup.

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

Issue: #160
@konard konard self-assigned this Jun 8, 2026
@konard konard changed the title [WIP] We should clearly separate JavaScript and Rust Separate JavaScript and Rust package roots Jun 8, 2026
@konard
konard marked this pull request as ready for review June 8, 2026 13:26
@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Working session summary

Done. PR 161 is updated and ready:

#161

Current head is e8ac08d on issue-160-a2a2313cf693, working tree is clean, and PR is not draft.

Both latest CI runs passed on that head:

  • JavaScript checks and release: success
  • Rust checks and release: success

Implemented the JS/Rust package-root split, separate workflows/releases/docs/scripts, the issue-160 case study, and Windows-safe Rust test fixes found during CI. I also filed the Rust template follow-up here:

link-foundation/rust-ai-driven-development-pipeline-template#65


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

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

🤖 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: $33.695573

📊 Context and tokens usage:

  • 1.0M / 1.1M (96%) input tokens, 79.2K / 128K (62%) output tokens

Total: (1.0M + 20.0M cached) input tokens, 79.2K output tokens, $33.695573 cost

🤖 Models used:

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

📎 Log file uploaded as Repository (85349KB)


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

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

✅ 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 commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Did we process all the scripts at root ./scripts folder as per requirements?

@konard
konard marked this pull request as draft June 8, 2026 14:12
@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

🤖 AI Work Session Started

Starting automated work session at 2026-06-08T14:12:50.044Z

The PR has been converted to draft mode while work is in progress.

This comment marks the beginning of an AI work session. Please wait for the session to finish, and provide your feedback.

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Yes. I audited the former root ./scripts contents and pushed follow-up commit 82c1a5a to make that explicit.

Current disposition:

  • former JavaScript release scripts were moved to js/scripts/
  • scripts/publish-to-crates.mjs was replaced by rust/scripts/publish-crate.rs
  • scripts/sync-rust-version.mjs was removed because JavaScript and Rust now release independently through their own version-and-commit scripts
  • the repository root no longer has a scripts/ directory

I also added regression coverage in js/tests/repository-layout.test.mjs and documented this in docs/case-studies/issue-160/README.md.

@konard
konard marked this pull request as ready for review June 8, 2026 14:20
@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

Follow-up is complete and PR 161 is ready for review again.

PR: #161

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

🤖 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: $4.864889

📊 Context and tokens usage:

  • 191.1K / 1.1M (18%) input tokens, 13.4K / 128K (10%) output tokens

Total: (191.1K + 2.4M cached) input tokens, 13.4K output tokens, $4.864889 cost

🤖 Models used:

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

📎 Log file uploaded as Gist (19677KB)


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

@konard

konard commented Jun 8, 2026

Copy link
Copy Markdown
Member Author

✅ 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

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.

We should clearly separate JavaScript and Rust

1 participant