Added documentation support to Skyline Tool Store

[brendanx67]

Summary

• Extract tool-inf/docs/ from uploaded tool ZIPs and serve HTML documentation via WebDAV (/_webdav/.../@files/docs/)
• Show "Online Documentation" link in the Documentation box on tool details page when docs/index.html exists
• Carry forward docs from previous version when new ZIP omits them (same pattern as supplementary files)
• Fix icon extraction to skip images under tool-inf/docs/ so screenshot PNGs don't replace the tool icon
• Exclude docs directory from supplementary file listings
• Fall back to /home/support when no tool-specific support board exists, so new tools default to the Skyline support board

Test plan

• Uploaded SkylineMcpConnector ZIP with tool-inf/docs/ — docs extracted, "Online Documentation" link appears, renders correctly in new tab
• Tool icon displays correctly (not replaced by docs screenshot)
• Support Board button navigates to /home/support for tools without a dedicated support board
• Support Board button still navigates to tool-specific board (e.g., AvantGardeDIA) when one exists
• Supplementary files section does not show docs directory

Co-Authored-By: Claude noreply@anthropic.com

[Copilot]

Pull request overview

Adds support for tool-provided HTML documentation in the Skyline Tool Store by extracting tool-inf/docs/ from uploaded ZIPs, serving it via WebDAV, and surfacing an “Online Documentation” link on the tool details page. Also improves upload behavior by carrying documentation forward across versions, prevents docs images from being mistaken as the tool icon, hides docs from supplementary listings, and adjusts Support Board fallback behavior.

Changes:

• Extract tool-inf/docs/ into a docs/ folder under the tool’s file root and (when missing) copy forward docs from the previous version.
• Add “Online Documentation” link in the tool details UI when docs/index.html exists.
• Prevent docs content from affecting icon selection and exclude docs from supplementary file listings; add Support Board fallback to /home/support.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File	Description
SkylineToolsStore/src/org/labkey/skylinetoolsstore/view/SkylineToolDetails.jsp	Adds Support Board fallback selection and conditionally displays an “Online Documentation” link.
SkylineToolsStore/src/org/labkey/skylinetoolsstore/model/SkylineTool.java	Adds helpers to detect docs presence and compute the WebDAV URL to docs/index.html.
SkylineToolsStore/src/org/labkey/skylinetoolsstore/SkylineToolsStoreController.java	Skips tool-inf/docs/ during icon discovery, extracts docs from ZIPs (with zip-slip protection), copies docs forward when absent, and excludes docs from supplementary file basenames.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

[Copilot]

Pull request overview

This PR adds first-class support for tool-provided HTML documentation in the Skyline Tool Store by extracting tool-inf/docs/ from uploaded ZIPs, serving it via WebDAV, and exposing it in the tool details UI, while also preventing docs assets from interfering with icons and supplementary file listings.

Changes:

• Extract tool-inf/docs/ into each tool’s file root docs/ directory (with zip-slip protection) and optionally carry docs forward from the prior version.
• Add an “Online Documentation” link on the tool details page when docs/index.html exists, and exclude the docs directory from supplementary file lists.
• Prevent icon extraction from accidentally selecting images located under tool-inf/docs/, and add a fallback support-board destination (/home/support) when a tool-specific board doesn’t exist.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File	Description
SkylineToolsStore/src/org/labkey/skylinetoolsstore/view/SkylineToolDetails.jsp	Adds support board fallback logic and conditionally renders an “Online Documentation” link in the Documentation box.
SkylineToolsStore/src/org/labkey/skylinetoolsstore/model/SkylineTool.java	Adds hasDocumentation() and WebDAV-based getDocsUrl() to support UI linking to extracted docs.
SkylineToolsStore/src/org/labkey/skylinetoolsstore/SkylineToolsStoreController.java	Implements docs extraction/carry-forward, excludes docs from supplementary listings, and avoids using docs images as tool icons.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.