lhumina_code/hero_aibroker

Fork 0

chore: drop stale unified openrpc.json (phase 9 cleanup) #138

Open

timur wants to merge 1 commit from development_drop_stale_openrpc_artifacts into development

timur commented

2026-05-14 09:55:37 +00:00

Owner

Drops two artifacts that pre-dated the Phase-9 per-domain split (commit 253a994) and are now bloat:

File	Why drop
`crates/hero_aibroker_server/openrpc.json`	Documentary 61-method unified spec from before the split. `specs/IMPLEMENTATION.md` tree diagram literally has `openrpc.json # DELETE in phase 9`; Phase 9 shipped without doing the deletion. The actual source of truth for each per-domain socket is `crates/hero_aibroker_server/specs/<domain>.openrpc.json`, served by `include_spec()` at runtime.
`crates/hero_aibroker_server/openrpc.client.generated.rs`	Macro debug output for an `openrpc_client!` invocation that no longer exists (the SDK now generates per-domain clients from `crates/hero_aibroker_sdk/specs/`).

Kept: crates/hero_aibroker_server/specs/IMPLEMENTATION.md — historical record of the per-domain split design; still useful reference for anyone touching this area. Plus the 10 per-domain specs/*.openrpc.json files (actual source of truth).

README updated to point at specs/admin.openrpc.json as the spec for cascade admin RPC.

cargo build clean for both hero_aibroker_server and hero_aibroker_sdk.

🤖 Generated with Claude Code

Drops two artifacts that pre-dated the Phase-9 per-domain split (commit 253a994) and are now bloat: | File | Why drop | |---|---| | `crates/hero_aibroker_server/openrpc.json` | Documentary 61-method unified spec from before the split. `specs/IMPLEMENTATION.md` tree diagram literally has `openrpc.json # DELETE in phase 9`; Phase 9 shipped without doing the deletion. The actual source of truth for each per-domain socket is `crates/hero_aibroker_server/specs/<domain>.openrpc.json`, served by `include_spec()` at runtime. | | `crates/hero_aibroker_server/openrpc.client.generated.rs` | Macro debug output for an `openrpc_client!` invocation that no longer exists (the SDK now generates per-domain clients from `crates/hero_aibroker_sdk/specs/`). | **Kept**: `crates/hero_aibroker_server/specs/IMPLEMENTATION.md` — historical record of the per-domain split design; still useful reference for anyone touching this area. Plus the 10 per-domain `specs/*.openrpc.json` files (actual source of truth). README updated to point at `specs/admin.openrpc.json` as the spec for cascade admin RPC. `cargo build` clean for both `hero_aibroker_server` and `hero_aibroker_sdk`. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

timur added 1 commit

2026-05-14 09:55:37 +00:00

chore: drop stale unified openrpc.json + IMPLEMENTATION.md

Build and Test / build-and-test (pull_request) Failing after 1m44s

Details

26a6036709

Both artifacts pre-dated the Phase-9 per-domain split (commit 253a994)
and are now bloat:

- crates/hero_aibroker_server/openrpc.json — documentary 61-method unified
  spec from before the split. The actual source of truth for each
  per-domain socket is crates/hero_aibroker_server/specs/<domain>.openrpc.json,
  served by include_spec() at runtime.
- crates/hero_aibroker_server/specs/IMPLEMENTATION.md — design doc from
  2026-05-11 planning the per-domain split. Implementation is done; doc is
  stale.
- crates/hero_aibroker_server/openrpc.client.generated.rs — macro debug
  output for a macro invocation that no longer exists (the SDK now
  generates per-domain clients via crates/hero_aibroker_sdk/specs/).

README updated to point at specs/admin.openrpc.json as the actual
spec for cascade admin RPC.

cargo build clean for both hero_aibroker_server and hero_aibroker_sdk.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

timur force-pushed development_drop_stale_openrpc_artifacts from 26a6036709

Build and Test / build-and-test (pull_request) Failing after 1m44s

Details

to f1db81430d

Build and Test / build-and-test (pull_request) Failing after 1m22s

Details

2026-05-14 10:23:59 +00:00

Compare

timur changed title from ~~chore: drop stale unified openrpc.json + IMPLEMENTATION.md~~ to chore: drop stale unified openrpc.json (phase 9 cleanup)

2026-05-14 10:24:13 +00:00

Build and Test / build-and-test (pull_request) Failing after 1m22s

Details

This pull request can be merged automatically.

This branch is out-of-date with the base branch

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin development_drop_stale_openrpc_artifacts:development_drop_stale_openrpc_artifacts

git switch development_drop_stale_openrpc_artifacts

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_drop_stale_openrpc_artifacts

git switch development_drop_stale_openrpc_artifacts

git rebase development

git switch development

git merge --ff-only development_drop_stale_openrpc_artifacts

git switch development_drop_stale_openrpc_artifacts

git rebase development

git switch development

git merge --no-ff development_drop_stale_openrpc_artifacts

git switch development

git merge --squash development_drop_stale_openrpc_artifacts