fix(jobs): map nu to nushell so action_set deserializes correctly #13

Merged

mahmoud merged 1 commit from development_fix_nu_interpreter into development

2026-04-27 06:03:11 +00:00

mahmoud commented

2026-04-27 06:01:39 +00:00

Owner

Fixes the bug surfaced while verifying issue #9.

What was wrong

hero_codescalers_server was passing interpreter: "nu" to hero_proc.action_set. hero_proc serde rejects that with:

unknown variant `nu`, expected one of `bash`, `sh`, `rhai`, `python3`, `bun`, `node`, `nushell`, `exec`, `ai`, `mcp`

Anyhow context wrapping was hiding this — callers only saw <feature> enqueue failed with no underlying cause.

Fix

Two changes:

jobs::enqueue normalizes "nu" to "nushell" before passing to ActionBuilder. One site catches all 12 callers (users.rs / services.rs / template.rs).
main.rs::rpc_handler returns format!("{:#}", e) instead of e.to_string() so future serde mismatches surface their full chain.

Verified

On kristof4 — user.bridge_ping now succeeds; job ran with the canonical codescaler tags (codescaler, codescaler_, codescaler_kind_, codescaler_target_).

Closes the bug found in #9.

Fixes the bug surfaced while verifying issue #9. ## What was wrong hero_codescalers_server was passing interpreter: "nu" to hero_proc.action_set. hero_proc serde rejects that with: ``` unknown variant `nu`, expected one of `bash`, `sh`, `rhai`, `python3`, `bun`, `node`, `nushell`, `exec`, `ai`, `mcp` ``` Anyhow context wrapping was hiding this — callers only saw `<feature> enqueue failed` with no underlying cause. ## Fix Two changes: - jobs::enqueue normalizes "nu" to "nushell" before passing to ActionBuilder. One site catches all 12 callers (users.rs / services.rs / template.rs). - main.rs::rpc_handler returns format!("{:#}", e) instead of e.to_string() so future serde mismatches surface their full chain. ## Verified On kristof4 — user.bridge_ping now succeeds; job ran with the canonical codescaler tags (codescaler, codescaler_<actor>, codescaler_kind_<kind>, codescaler_target_<t>). Closes the bug found in https://forge.ourworld.tf/lhumina_code/hero_codescalers/issues/9.

mahmoud added 1 commit

2026-04-27 06:01:39 +00:00

fix(jobs): map nu to nushell so action_set deserializes correctly d38491b59a

hero_proc's serde rejects "nu" with: unknown variant `nu`, expected one of
`bash`, `sh`, `rhai`, `python3`, `bun`, `node`, `nushell`, `exec`, `ai`, `mcp`.
Anyhow's context wrapping was hiding this; callers only saw "<feature>
enqueue failed" with no underlying cause.

Two changes:
- jobs::enqueue normalizes "nu" to "nushell" before passing to ActionBuilder.
  Catches all 12 callers (users.rs, services.rs, template.rs) at one site.
- main.rs::rpc_handler returns format!("{:#}", e) instead of e.to_string()
  so future serde mismatches surface their full anyhow chain.

Verified on kristof4: user.bridge_ping now succeeds; job ran with codescaler
tags as designed.

#9