skills: v6.6.2 catch-up - #17
Draft
seidroid[bot] wants to merge 1 commit into
Draft
Conversation
…ed nodes for known public networks (pacific-1, atlantic-2), and the --chain-id flag is now documented as required. (sei-protocol/sei-chain#3954)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Skill catch-up for v6.6.2.
1 source PR(s) produced changes across 1 commit(s). Each source PR is a separate commit, so this reviews commit-by-commit.
skill/references/cli/seid-cli.mdReviewer notes
release/v6.6: feat(seeds): ship Sei Labs seeds as the default bootstrap-peers sei-chain#3954 — The core behavior change (auto-populated bootstrap-peers + required --chain-id) is operator-facing and best documented in node-operations.md, which already covers seid init usage. The seid-cli.md reference does not currently document seid init at all — this is more of an add_section than a targeted update, but flagged as update since the file exists and lacks any init coverage; a reviewer may prefer to fold it into node-operations.md only. The new app/seeds Go package is internal (built-in seed data) and does not itself warrant a standalone skill file, since operators interact with it only indirectly via seid init.Flagged by a review of every touched page's final text. Each source PR was analysed in isolation, so these were invisible to the per-PR checks. Verify before merging — the checker cannot tell which side of a contradiction is correct.
Comparing the new text against the untouched surrounding text:
Contradiction on
--modevalues: The newseid initsynopsis lists[--mode full|validator|seed|archive](witharchive), but the following bullet describes onlyvalidator/seed/fulland says "Default mode isfull" — while the untouched section header shows--mode full|validator|seed|archive. (Minor, but the bullet's enumeration omitsarchive; not a hard contradiction.)Contradiction on IAVL/SeiDB backend vs. configurability: The untouched "Proofs" bullet says "The legacy IAVL backend has been fully removed and SeiDB SC is mandatory — a node started with
sc-enable = falsepanics." This describessc-enableas both a removed/mandatory feature and a still-settable config option (sc-enable = falsecan be set to trigger the panic). This is the "removed and configurable" pattern.NONE of the newly inserted
Node Initializationtext itself contradicts the untouched text, is misplaced under a contradicting heading, or misspells identifiers consistently used elsewhere.Given the criteria, the only clear qualifying item is:
sc-enable(legacy IAVL "fully removed"/"mandatory" yetsc-enable = falseis a settable value).Generated by sei-docs-bridge. Every change is a proposal — verify against the source PRs before merging.