Skip to content

docs+feat: document A2A_NODE_ID in SKILL.md and warn when unset#164

Closed
WeZZard wants to merge 2 commits intoEvoMap:mainfrom
WeZZard:feat/document-a2a-node-id
Closed

docs+feat: document A2A_NODE_ID in SKILL.md and warn when unset#164
WeZZard wants to merge 2 commits intoEvoMap:mainfrom
WeZZard:feat/document-a2a-node-id

Conversation

@WeZZard
Copy link
Contributor

@WeZZard WeZZard commented Mar 2, 2026

Problem

A2A_NODE_ID is the canonical environment variable read by getNodeId() in src/gep/a2aProtocol.js, but:

  1. It is not mentioned anywhere in SKILL.md -- new agents have no way to discover it without reading source code
  2. When A2A_NODE_ID is unset, getNodeId() silently falls back to a device-fingerprint-based ID that may change across machines or environments, causing identity mixups with no visible signal

Changes

SKILL.md (documentation):

  • Add Setup section: node registration flow, 24-hour claim window, how to set A2A_NODE_ID
  • Add A2A_NODE_ID as first row in the Configuration table, marked required
  • Emphasize: never hardcode node IDs in scripts -- getNodeId() reads the env var automatically

src/gep/a2aProtocol.js (code):

  • Add console.warn when falling back to device-fingerprint generation, directing users to set A2A_NODE_ID and pointing to SKILL.md

Notes

  • Backward-compatible: no change to fallback behavior, only adds a warning
  • Follows CONTRIBUTING.md style: small, reviewable, documentation updated alongside behavior change

WeZZard added 2 commits March 2, 2026 06:51
A2A_NODE_ID is the canonical env var read by getNodeId() in
a2aProtocol.js but was not documented anywhere in SKILL.md.

New agents installing this skill had no way to discover this without
reading source code, leading to hardcoded node IDs in integration
scripts and identity mixups between agents.

Changes:
- Add Setup section: node registration flow, claim URL, env var
- Add A2A_NODE_ID row to Configuration table
- Emphasize: never hardcode node IDs in scripts
…rprint

Without this warning, agents that forget to set A2A_NODE_ID silently
get a device-fingerprint-based node ID that may change across machines
or environments, causing identity mixups with no visible signal.

The warning directs users to set A2A_NODE_ID and points to SKILL.md
for registration instructions.
@WeZZard WeZZard changed the title docs: add Setup section and A2A_NODE_ID to SKILL.md docs+feat: document A2A_NODE_ID in SKILL.md and warn when unset Mar 2, 2026
@WeZZard WeZZard marked this pull request as ready for review March 2, 2026 13:32
@autogame-17
Copy link
Collaborator

Thank you @WeZZard for this contribution! The changes have been reviewed and merged into the upstream codebase with full Co-authored-by attribution.

Your improvements will be included in the next public release:

  • SKILL.md: Setup section and A2A_NODE_ID documentation added
  • a2aProtocol.js: console.warn when A2A_NODE_ID is unset

If you have an EvoMap account, we would like to award you credits for this contribution. Please reply with your node_id or registered email so we can send the reward.

Closing this PR as the changes have been cherry-picked upstream (the public repo is a one-way publish from the private codebase). Thanks again!

@autogame-17 autogame-17 closed this Mar 5, 2026
fmw666 pushed a commit that referenced this pull request Mar 5, 2026
- Sync zh-CN README with cron keepalive best practice
- Add A2A_NODE_ID setup guide to SKILL.md (PR #164, thanks @WeZZard)
- Add cron keepalive best practice to README (PR #167, thanks @Golden-Koi)
- Add acknowledgments for WeZZard and Golden-Koi

Co-authored-by: WeZZard <user@wezzard.com>
Co-authored-by: Golden-Koi <golden-koi@users.noreply.github.com>
@WeZZard
Copy link
Contributor Author

WeZZard commented Mar 6, 2026

Thanks for your appreciation. It is my pleasure to receive the credits. This is my node id: node_56c96e7995bc8723. (^_^)

fmw666 pushed a commit that referenced this pull request Mar 11, 2026
PR #164 (WeZZard): document A2A_NODE_ID setup in SKILL.md, warn when unset
PR #167 (Golden-Koi): cron keepalive best practice in README.md

Co-authored-by: WeZZard <wezzard@users.noreply.github.com>
Co-authored-by: Golden-Koi <263994476+Golden-Koi@users.noreply.github.com>
fmw666 pushed a commit that referenced this pull request Mar 11, 2026
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.

2 participants