docs: classify JSON-RPC breaking changes for application developers - #297
docs: classify JSON-RPC breaking changes for application developers#297kutluhaneth46 wants to merge 2 commits into
Conversation
Add an [RPC] flag and For Application Developers sections in BREAKING_CHANGES.md, clarifying that --rpc.gascap is a node-local simulation limit distinct from protocol gas limits. Fixes circlefin#270 and circlefin#271. Co-authored-by: Cursor <cursoragent@cursor.com>
|
The taxonomy change is a solid implementation of #270, and I verified the two new v0.7.2 Two substantive problems, though: 1. The v0.8.0 "For Application Developers" entry restates error-text claims that #291 (open) corrects with live-RPC verification.
2. A missed fact that directly serves this PR's own #271 story: Coordination note: this PR and #291 edit the same v0.8.0 region of Small nit: |
Address review on circlefin#297: scope application-facing notes to v0.7.2 (v0.8.0 error-text matrix is tracked in circlefin#291) and document the gas required exceeds allowance string collision. Co-authored-by: Cursor <cursoragent@cursor.com>
|
Thanks for the detailed review — addressed in 9cf3f04:
|
|
Verified I also checked the one thing that worried me about moving the disambiguation into a v0.7.2 entry: whether the "same allowance string on both paths" behavior actually holds on the reth lineage v0.7.x pins (
So in v0.7.2 the exact string varies by failure mode — which "often surfaces as One operational heads-up: the |
|
nice patch 👍 although again it should probably go into a separate repo with dApp documentation. |
Add [RPC] flag and For Application Developers sections in BREAKING_CHANGES.md for v0.7.2 RPC changes. Clarifies rpc.gascap is a node-local simulation limit and documents the gas required exceeds allowance disambiguation. v0.8.0 error-text classification deferred to #291. Fixes #270, fixes #271.