Skip to content

[Cloud] strip UTF-8 BOM from agent-seed.env - #3727

Merged
GuillaumeDSM merged 2 commits into
devfrom
cursor/fix-agent-seed-env-bom-251f
Oct 6, 2026
Merged

GuillaumeDSM merged 2 commits into
devfrom
cursor/fix-agent-seed-env-bom-251f

Conversation

@GuillaumeDSM

@GuillaumeDSM GuillaumeDSM commented Oct 6, 2026 •

Copy link
Copy Markdown
Member

Thank you for sending your pull request.
But first, have you included unit tests, and is your code PEP8 conformant? More details

Summary

Make the Cloud Agent install use the ui-node-web profile, and make .cursor/agent-seed.env sourceable by Bash.

Solve the issue: #___

Changelog

  • Default environment.json install is PANTS_PANTSD=false OCTOBOT_CLOUD_PROFILE=ui-node-web bash .cursor/cloud-install.sh
  • Strip the leading UTF-8 BOM from .cursor/agent-seed.env
  • Document that GitHub cloud-env-validate still installs ci-tentacles

What's new?

Cloud agents that follow environment.json now build the node web UI (ui-node-web) instead of stopping at ci-tentacles. PANTS_PANTSD=false keeps the Pants daemon from reserving extra memory during install.

.cursor/agent-seed.env began with a UTF-8 BOM. Bash treated that byte as part of the first variable name, so source .cursor/agent-seed.env failed and bash .cursor/seed-agent.sh could not resolve user/agent-seed. The variable values are unchanged.

GitHub Actions cloud-env-validate still passes OCTOBOT_CLOUD_PROFILE=ci-tentacles itself.

Validation

Local ui-node-web install passed twice (OCTOBOT_CLOUD_INSTALL_OK). bash .cursor/cloud-validate.sh --profile ui-node-web --json passed. Simulated seed wrote user/agent-seed (Kraken sim accounts). Node UI login with passphrase demodemo showed Seed kraken A and Seed kraken B. Draft build bld-20261006-ca757bec-10d6-4724-8ad6-c0249b2814c5 (from dev) succeeded, and a fresh agent booted from that build passed version, tentacles, UI dist, and pytest tests/unit_tests/test_octobot_version.py (4 passed).

Open in Web Open in Cursor 

Bash treats the BOM as part of the variable name, so source .cursor/agent-seed.env fails and the documented demo seed cannot start.

Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
@GuillaumeDSM
GuillaumeDSM marked this pull request as ready for review October 6, 2026 09:12
@GuillaumeDSM
GuillaumeDSM requested a review from Herklos as a code owner October 6, 2026 09:12
Cloud agents need the node web UI build. GitHub cloud-env-validate still installs ci-tentacles explicitly.

Co-authored-by: Guillaume De Saint Martin <GuillaumeDSM@users.noreply.github.com>
@GuillaumeDSM
GuillaumeDSM enabled auto-merge (rebase) October 6, 2026 09:28
@GuillaumeDSM
GuillaumeDSM merged commit 4792687 into dev Oct 6, 2026
26 checks passed
@GuillaumeDSM
GuillaumeDSM deleted the cursor/fix-agent-seed-env-bom-251f branch October 6, 2026 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants