Skip to content

docs: correct stale server, permission, and theme documentation - #44

Merged
wayback09 merged 1 commit into
Aether-Launcher:mainfrom
DarkLabsStudio:docs/fix-stale-docs
Nov 1, 2026
Merged

wayback09 merged 1 commit into
Aether-Launcher:mainfrom
DarkLabsStudio:docs/fix-stale-docs

Conversation

@DarkLabsStudio

@DarkLabsStudio DarkLabsStudio commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Corrects stale documentation found while reviewing main at 28b93cc:

  • Removes the contradictory "server hosting is planned for v2 / not available" line; servers:process is implemented and documented.
  • Adds the five implemented permissions missing from the API and extension permission lists: modpacks:install, resourcepacks:install, shaderpacks:install, screenshots:read, screenshots:write.
  • Clarifies that directories under servers/ are shared across extensions; servers:manage can list/delete any valid server directory, not only ones created by the same extension.
  • Documents the theme sanitizer's actual restrictions: url(), @font-face, quoted strings, and full-window overlays. Adds the current font-bundling limitation and notes that sanitization runs at install time.
  • Fixes the server-manager spec's references to missing dist-extensions/ examples and aligns Fabric/any-jar fallback and error text with findServerJar.
  • Updates the SDK docs to mark the TypeScript snippet as a trimmed example and point to the full declarations.
  • Notes the CLI's Go 1.26.2 requirement and corrects the stale GetCSS comment.

Testing

  • go test ./pkg/theme
  • git diff --check

No behavior changes.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

- Remove the stale planned-v2 note for servers:process
- Add missing modpack, resourcepack, shaderpack, and screenshot permissions
- Document shared servers/ directories and theme sanitizer limits
- Fix server-manager example references and Fabric jar fallback docs
- Mark the SDK snippet as trimmed and note the CLI Go requirement
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