Port to Hero Service architecture; fix missing RPC methods and UI bugs #10
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "development_standardize"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Description
Completes the conversion of hero_redis to the standard Hero Service
5-crate layout and fixes the admin UI errors caused by the new
stateless proxy architecture.
Changes
redis.get with type detection, redis.del, redis.database_create)
single redis.exec call with db param instead of broken 3-call pattern
Related Issues
Restructures the workspace from an ad-hoc layout into the standard Hero Service 5-crate pattern (_openrpc backend + _http proxy + _client + _rhai + core lib), completing the backend/HTTP separation and wiring up all missing RPC methods so the admin UI is fully functional. Crate renames - hero_redis_server → hero_redis (core library) - hero_redis_sdk → hero_redis_client - hero_redis_ui → hero_redis_http - new: hero_redis_openrpc (JSON-RPC management backend) hero_redis_openrpc (new backend binary) - Binds ~/hero/var/sockets/hero_redis_openrpc.sock (newline-delimited JSON-RPC 2.0) plus RESP2 TCP :3378 and Unix socket - Implements: rpc.health, rpc.discover, auth.validate, redis.exec, redis.info, redis.database_list, redis.select, redis.keys, redis.get (with type detection for string/list/set/hash/zset), redis.del, redis.database_create, perf.batch, perf.batch_native hero_redis_http (rewritten as pure proxy) - Removed all hero_redis core dependency; /rpc and /rpc/:context are transparent proxies to the openrpc socket - Dual listener: Unix socket (hero_proxy) + TCP :8812 (dev) - Login validated via auth.validate RPC call to backend Template fixes - flushDatabase used a broken select→exec(FLUSHDB)→select(0) pattern across three stateless connections; replaced with a single redis.exec({command:"FLUSHDB", db}) call in database_manager.html and base.html Build & CI - buildenv.sh/Makefile updated to reference new binary names - .forgejo/workflows/ci.yml added (check + clippy -D warnings + test) - .cargo/config.toml.example added for local patch overrides - All clippy -D warnings resolved (pre-existing issues suppressed with crate-level #![allow(...)] in hero_redis and hero_redis_rhai)hero_redis_sdktohero_redis_clientView command line instructions
Checkout
From your project repository, check out a new branch and test the changes.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.