Skip to content

Remove agent rule text that describes the removed Go Server - #1460

Merged
longquanzheng merged 1 commit into
mainfrom
chore/remove-go-server-rule-text
Nov 10, 2026
Merged

longquanzheng merged 1 commit into
mainfrom
chore/remove-go-server-rule-text

Conversation

@longquanzheng

Copy link
Copy Markdown
Contributor

Summary

Remove text from the agent rule files (CLAUDE.md, AGENTS.md, .cursor/rules) that still describes the removed Go Server and the Temporal and Cadence backends. The three rule sets stay in sync.

  • Drop area:server from the allowed area labels.
  • Rewrite the User-Facing Concept Boundary around server-zd engine internals (apply, commands, partitions, history mechanics). The Continue-As-New exception stays, because the SDKs and server-zd still support it.
  • Remove the Go-specific config injection rules (config.Config, cfg *config.XyzConfig). No remaining Go code uses that pattern. Constructor injection and the no-post-construction-wiring rule stay. The Rust equivalents are in the server-zd conventions.
  • Remove the Temporal Web references from the UI/UX rules.

Why

#1059 removed the Temporal-based Go Server; only server-zd remains. The rules still told agents to name "the backend" in operations docs, to avoid Temporal and Cadence internals, and to inject config.Config sections. A grep for config.Config and XyzConfig across sdk-go, cli, examples and server-zd/integ finds no use. Agents following these rules were being steered by a component that no longer exists. The area:server label itself is being retired separately.

Testing

Rule text only. No code changed.

doc-map: no change needed: agent rule files are not mapped docs or skills.

@longquanzheng
longquanzheng merged commit 8033c81 into main Nov 10, 2026
2 checks passed
@longquanzheng
longquanzheng deleted the chore/remove-go-server-rule-text branch November 10, 2026 03:01
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.

1 participant