Skip to content

fix: macOS updater archives, AppImage GST registry, mesh max_tokens - #2694

Open
Chessing234 wants to merge 6 commits into
block:mainfrom
Chessing234:fix/updater-console-gst
Open

Chessing234 wants to merge 6 commits into
block:mainfrom
Chessing234:fix/updater-console-gst

Conversation

@Chessing234

Copy link
Copy Markdown
Contributor

Summary

Test plan

  • cargo test --features mesh-llm --lib preserves_user_max_output
  • cargo fmt --check, bash -n fix-appimage, YAML parse release.yml
  • release assemble guard (CI)

Made with Cursor

@bokiko

bokiko commented Jul 24, 2026

Copy link
Copy Markdown

Thanks for the quick fix. I validated the #2558 part of this PR on an independent two-node shared-compute testbed (SmolLM2-135M served over relay-mesh, consumer agent with a per-agent BUZZ_AGENT_MAX_OUTPUT_TOKENS=1024).

Tested at PR head 5c1f22bde279bbdccf86ce55ae84a95bd5e56a91. What I confirmed:

  • The per-agent BUZZ_AGENT_MAX_OUTPUT_TOKENS=1024 now reaches the spawned buzz-acp / buzz-agent (verified on their live process env). Previously the harness always ran 4096 — that clobber is gone.
  • The prior failure is gone: no more 503 … all 0 tunnel(s) from the context fit check. Mesh requests succeed, and a direct completion through the same ingress at max_tokens=1024 returns 200.
  • The added unit test (apply_relay_mesh_env_preserves_user_max_output_tokens) passes.

One caveat so I'm not overstating it: the agent did not post a full channel reply in my run. Inference succeeded (no 503), but SmolLM2-135M didn't produce a postable response — that looks like tiny-model/harness-quality behavior, separate from this PR's fix.

Net: the env-precedence change resolves the clobber and unblocks small-context shared-compute agents at the routing/sizing layer. Happy to run further checks on the testbed if useful.

@Chessing234

Copy link
Copy Markdown
Contributor Author

nice, thanks for validating on the two-node mesh setup

@cameronhotchkies cameronhotchkies added the triage-ready Appropriate for agentic review label Jul 28, 2026
@Chessing234
Chessing234 force-pushed the fix/updater-console-gst branch from 5c1f22b to 7f61936 Compare July 29, 2026 14:37
Signed-off-by: Taksh <takshkothari09@gmail.com>
Signed-off-by: Taksh <takshkothari09@gmail.com>
Signed-off-by: Taksh <takshkothari09@gmail.com>
Signed-off-by: Taksh <takshkothari09@gmail.com>
Signed-off-by: Taksh <takshkothari09@gmail.com>
@Chessing234
Chessing234 force-pushed the fix/updater-console-gst branch from 7f61936 to 01f3136 Compare July 29, 2026 14:46
@Chessing234

Copy link
Copy Markdown
Contributor Author

Rebased onto latest main to pick up nostr 0.44.6 (RUSTSEC-2026-0216) and the file-size ratchet baseline. Local Desktop file-size check is clean against current main.

CI workflows are waiting on maintainer approval for this fork push — please Approve and run workflows when you can.

@Chessing234

Copy link
Copy Markdown
Contributor Author

@tlongwell-block @wesbillman @wpfleger96 mind taking a look when you get a chance?

@Chessing234
Chessing234 force-pushed the fix/updater-console-gst branch from 01f3136 to 400f45c Compare August 5, 2026 10:38
Signed-off-by: Taksh <takshkothari09@gmail.com>
@github-actions

github-actions Bot commented Sep 1, 2026

Copy link
Copy Markdown

🔐 Codex Security Review

Status: review required for the current range.

The current range is 59328d5ae38a51a618dd2fddd7faf1343d42096f...c865c3343843f27666ecb62953ba40a5a5b5ec99.
A new review must complete for this exact range. When manual authorization
is required, a Block organization member must comment exactly
@buzz-security-review c865c3343843f27666ecb62953ba40a5a5b5ec99 to authorize a new review.
Any previous review applies only to its recorded range.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

triage-ready Appropriate for agentic review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants