A line-by-line review after the 7-skill restructure (#17) surfaced four high-priority content issues.
-
Dangling security/SKILL.md link — skills/standards/SKILL.md:120 points to ../security/SKILL.md, which no longer exists; security/ was merged into soroban/SKILL.md (Part 3) during the restructure.
-
Inconsistent env.register API in the Soroban skill — skills/soroban/SKILL.md teaches the older env.register_contract(None, X) / env.register_contract_wasm(None, WASM) forms at lines 510, 527, 634, 660, 690, 716, 765, 789, 812, 813, 1134, 1277 and the newer env.register(X, ()) form at lines 1193, 1241, 1278, which will confuse anyone copy-pasting between sections.
-
stellar contract install is deprecated — skills/soroban/SKILL.md:925 uses stellar contract install, which the current Stellar CLI marks as deprecated in favor of stellar contract upload.
-
Freighter API uses pre-v3 surface — skills/dapp/SKILL.md:121 imports getPublicKey from @stellar/freighter-api, which no longer exports it — the current package uses getAddress() / requestAccess() returning { address, error }, and isConnected() / signTransaction() now return object shapes ({ isConnected, error } and { signedTxXdr, signerAddress, error }) instead of bare values.
A line-by-line review after the 7-skill restructure (#17) surfaced four high-priority content issues.
Dangling
security/SKILL.mdlink —skills/standards/SKILL.md:120points to../security/SKILL.md, which no longer exists;security/was merged intosoroban/SKILL.md(Part 3) during the restructure.Inconsistent
env.registerAPI in the Soroban skill —skills/soroban/SKILL.mdteaches the olderenv.register_contract(None, X)/env.register_contract_wasm(None, WASM)forms at lines 510, 527, 634, 660, 690, 716, 765, 789, 812, 813, 1134, 1277 and the newerenv.register(X, ())form at lines 1193, 1241, 1278, which will confuse anyone copy-pasting between sections.stellar contract installis deprecated —skills/soroban/SKILL.md:925usesstellar contract install, which the current Stellar CLI marks as deprecated in favor ofstellar contract upload.Freighter API uses pre-v3 surface —
skills/dapp/SKILL.md:121importsgetPublicKeyfrom@stellar/freighter-api, which no longer exports it — the current package usesgetAddress()/requestAccess()returning{ address, error }, andisConnected()/signTransaction()now return object shapes ({ isConnected, error }and{ signedTxXdr, signerAddress, error }) instead of bare values.