feat: add Rust implementation and reorganize codebase #27
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Adding CLAUDE.md with task information for AI processing. This file will be removed when the task is complete. Issue: #26
- Reorganize JavaScript code into js/ folder with dedicated js.yml workflow - Add Rust implementation in rust/ folder with rust.yml workflow - Both implementations provide compatible REST APIs: - GET /html?url=<URL> - Render page as HTML - GET /markdown?url=<URL> - Convert page to Markdown - GET /image?url=<URL> - Screenshot page as PNG - GET /fetch?url=<URL> - Proxy fetch content - GET /stream?url=<URL> - Stream content - Each language version has its own: - README.md for package documentation - Dockerfile for microservice deployment - GitHub workflow for CI/CD and releases - Rust implementation uses axum web framework with tokio async runtime - JavaScript workflow uses changeset-based releases - Rust workflow uses version-tag based releases Closes #26 Co-Authored-By: Claude Opus 4.5 <[email protected]>
konard
changed the title
- Fix Rust workflow to use dtolnay/rust-toolchain instead of dtolnay/rust-action - Add changeset for JavaScript package changes Co-Authored-By: Claude Opus 4.5 <[email protected]>
Co-Authored-By: Claude Opus 4.5 <[email protected]>
Co-Authored-By: Claude Opus 4.5 <[email protected]>
axum 0.8.8 requires Rust 1.78+, updated from 1.75 to 1.83
This reverts commit 80fbdb3.
Collaborator
Author
🤖 Solution Draft LogThis log file contains the complete execution trace of the AI solution draft process. 💰 Cost estimation:
Now working session is ended, feel free to review and add any feedback on the solution draft. |
Collaborator
Author
|
./scripts folder can be left at the root, also let's make sure we have all scripts nessesary for both js.yml and rust.yml there. Ensure all changes are correct, consistent and fully meet the requirements, and both language versions have as similar implementations as possible. |
Collaborator
Author
|
🤖 AI Work Session Started Starting automated work session at 2026-01-23T21:01:04.391Z 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 working session to finish, and provide your feedback. |
- Move JavaScript release scripts from js/scripts/ to root scripts/ - Add Rust-specific release scripts for crates.io publishing - Update js.yml workflow to reference scripts from ../scripts/ - Update rust.yml workflow to use Node.js scripts for consistency - Both workflows now use similar patterns for version management and releases Co-Authored-By: Claude Opus 4.5 <[email protected]>
Collaborator
Author
🤖 AI Work Session CompleteThe following changes have been made based on your feedback: Changes Made
CI Status
The PR is now ready for review. |
Collaborator
Author
🤖 Solution Draft LogThis log file contains the complete execution trace of the AI solution draft process. 💰 Cost estimation:
Now working session is ended, feel free to review and add any feedback on the solution draft. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
js/folder with dedicatedjs.ymlworkflowrust/folder withrust.ymlworkflowscripts/folder (shared between JS and Rust)Changes
Scripts (
scripts/folder)js/scripts/to rootscripts/rust-*.mjs) for crates.io publishingJavaScript (
js/folder)js/folderrelease.ymltojs.ymlwith path filtering forjs/**andscripts/**js/README.mdfor npm package documentationRust (
rust/folder)GET /html?url=<URL>- Render page as HTMLGET /markdown?url=<URL>- Convert page to MarkdownGET /image?url=<URL>- Screenshot page as PNG (requires Chrome)GET /fetch?url=<URL>- Proxy fetch contentGET /stream?url=<URL>- Stream content--serve,--format,--output,--portoptionsrust.ymlworkflow with multi-OS testing matrix and Node.js scripts for releasesrust/README.mdfor crates.io documentationrust/Dockerfilefor microservice deploymentRoot Level
README.mdto document multi-language project structure withscripts/folderTest Plan
-D warningsRelated Issues
Closes #26
🤖 Generated with Claude Code