Refactor hero proc #42

Open
rawan wants to merge 1 commit from development_refactor into development
Member

Implementation Summary

Files Deleted

  • .claude/ directory (dev-specific Claude Code config)
  • .forgejo/workflows/build-macos.yaml.disabled (dead disabled workflow)
  • crates/hero_proc_factory/ (orphaned crate, functionality already in hero_proc_sdk)
  • .forgejo/workflows/test.yaml (merged into build.yaml)

Files Created

  • docs/cli.md — CLI command reference extracted from README
  • docs/architecture.md — Architecture, crate structure, core concepts extracted from README
  • docs/development.md — SDK usage, env vars, server flags extracted from README
  • .forgejo/workflows/docker.yaml — Docker image build workflow
  • .forgejo/workflows/release.yaml — renamed from build-linux.yaml

Files Modified

  • README.md — Reduced from ~400 to ~86 lines with links to docs
  • Makefile — Simplified from ~218 to ~177 lines, removed unused targets
  • .gitignore — Added .claude/ entry
  • .forgejo/workflows/build.yaml — Consolidated with test.yaml into single CI workflow
  • docker/Dockerfile — Fixed broken hero_proc_pid1 reference, added hero_proc_ui
  • buildenv.sh — Bumped VERSION to 0.5.0
  • Cargo.toml — Bumped workspace version to 0.5.0
  • crates/hero_proc_server/src/main.rs — Fixed clippy redundant_closure warning
  • Various .rs files — Fixed pre-existing cargo fmt issues

CI Fixes

  • Excluded integration tests from default make test (they require a running server)
  • Fixed pre-existing formatting and clippy issues

Edits rather than these are formatting edits.

## Implementation Summary ### Files Deleted - `.claude/` directory (dev-specific Claude Code config) - `.forgejo/workflows/build-macos.yaml.disabled` (dead disabled workflow) - `crates/hero_proc_factory/` (orphaned crate, functionality already in `hero_proc_sdk`) - `.forgejo/workflows/test.yaml` (merged into `build.yaml`) ### Files Created - `docs/cli.md` — CLI command reference extracted from README - `docs/architecture.md` — Architecture, crate structure, core concepts extracted from README - `docs/development.md` — SDK usage, env vars, server flags extracted from README - `.forgejo/workflows/docker.yaml` — Docker image build workflow - `.forgejo/workflows/release.yaml` — renamed from `build-linux.yaml` ### Files Modified - `README.md` — Reduced from ~400 to ~86 lines with links to docs - `Makefile` — Simplified from ~218 to ~177 lines, removed unused targets - `.gitignore` — Added `.claude/` entry - `.forgejo/workflows/build.yaml` — Consolidated with test.yaml into single CI workflow - `docker/Dockerfile` — Fixed broken `hero_proc_pid1` reference, added `hero_proc_ui` - `buildenv.sh` — Bumped VERSION to 0.5.0 - `Cargo.toml` — Bumped workspace version to 0.5.0 - `crates/hero_proc_server/src/main.rs` — Fixed clippy redundant_closure warning - Various `.rs` files — Fixed pre-existing `cargo fmt` issues ### CI Fixes - Excluded integration tests from default `make test` (they require a running server) - Fixed pre-existing formatting and clippy issues **Edits rather than these are formatting edits.**
refactor hero proc
All checks were successful
Build & Test / check (push) Successful in 2m53s
Build & Test / check (pull_request) Successful in 2m52s
1e423fb12a
All checks were successful
Build & Test / check (push) Successful in 2m53s
Build & Test / check (pull_request) Successful in 2m52s
This pull request has changes conflicting with the target branch.
  • tests/integration/tests/action_dependencies.rs
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin development_refactor:development_refactor
git switch development_refactor

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch development
git merge --no-ff development_refactor
git switch development_refactor
git rebase development
git switch development
git merge --ff-only development_refactor
git switch development_refactor
git rebase development
git switch development
git merge --no-ff development_refactor
git switch development
git merge --squash development_refactor
git switch development
git merge --ff-only development_refactor
git switch development
git merge development_refactor
git push origin development
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_proc!42
No description provided.