Refactor hero proc #42

Closed

rawan wants to merge 5 commits from development_refactor into development

rawan commented

2026-04-07 15:04:24 +00:00

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.** ### Related issues - https://forge.ourworld.tf/lhumina_code/hero_proc/issues/40

rawan added 1 commit

2026-04-07 15:04:24 +00:00