f3d: add shell extension registration instructions in notes field

[B67687]

• Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
• I have read the Contributing Guide

Closes #17335

Summary

Adds shell extension registration instructions to the f3d manifest using the notes field for manual registration.

This follows the established pattern from:

• PR rapidcrc@0.3.41: Add shell extension support #16246 (rapidcrc) - closed as not-planned for auto-registration
• PR qbittorrent: Add file associations #14839, qview: Add file associations #16811, geogebra: Fix checkver, add file associations #16797 - merged with notes for manual registration

Changes

• notes: Added instructions for users who want the Windows Explorer shell extension (thumbnail generation). Those who don't need it can install without admin rights.
• autoupdate: Removed hash extraction (no .md5 file in f3d releases)
• Fixed field order per contribution guidelines

The notes field provides registration instructions using elevated regsvr32 commands, allowing users to manually register the shell extension if needed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: ea825618-cdfc-463a-921d-9b4982f82be9

📥 Commits

Reviewing files that changed from the base of the PR and between a094130 and 020b72d.

📒 Files selected for processing (1)

• bucket/f3d.json

🚧 Files skipped from review as they are similar to previous changes (1)

• bucket/f3d.json

---

📝 Walkthrough

Walkthrough

The bucket/f3d.json manifest file is updated to include a new notes field containing instructions for enabling Windows Explorer thumbnails through elevated PowerShell with regsvr32 commands to register and unregister F3DShellExtension.dll. The manifest also removes the MD5 hash URL template ($url.md5) from the 64-bit autoupdate configuration.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and specifically describes the main change: adding shell extension registration instructions to the f3d manifest via the notes field.
Description check	✅ Passed	The description fully complies with the template, including proper checkbox completion, issue reference (Closes #17335), and comprehensive summary of all changes made.
Linked Issues check	✅ Passed	The PR fully addresses issue #17335 by adding shell extension registration instructions via the notes field, removing hash extraction, and reordering fields as required.
Out of Scope Changes check	✅ Passed	All changes are directly related to the linked issue #17335: notes field addition for registration instructions, autoupdate hash removal, and field reordering per guidelines.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[B67687]

Context on previous duplicate PR:

• PR f3d: add shell extension registration instructions in notes field #17665 was auto-closed when we force-pushed to fix the author misattribution (commits showed as Ben <ben+github@basswood.dev> instead of our GitHub account)
• We rewrote the commits with the correct author and rebased onto current master
• Maintainer @z-Fng had already approved the approach: "Yes."

All tests were passing before the close. This new PR contains the same changes.

[github-actions]

All changes look good.

Wait for review from human collaborators.

f3d

• Lint
• Description
• License
• Hashes
• Checkver
• Autoupdate

Check the full log for details.

[coderabbitai]

🧹 Nitpick comments (1)

bucket/f3d.json (1)

39-44: Optional pre-merge manifest validation commands

Before merge, run the standard Scoop checks for this manifest:

.\bin\checkver.ps1 -App f3d -f
.\bin\formatjson.ps1 -App f3d
scoop install .\bucket\f3d.json -a 64bit

Reference: https://github.com/ScoopInstaller/.github/blob/main/.github/CONTRIBUTING.md

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@bucket/f3d.json` around lines 39 - 44, The manifest's autoupdate block
(architecture -> 64bit -> url/extract_dir) should be validated before merging:
run the Scoop validation/format and install checks by executing the checkver,
formatjson, and a test install for 64bit (as described in the contrib guide) to
ensure the URL/extract_dir templating is correct and the manifest parses and
installs cleanly; fix any issues reported by .\bin\checkver.ps1 -App f3d -f,
.\bin\formatjson.ps1 -App f3d, and scoop install .\bucket\f3d.json -a 64bit.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@bucket/f3d.json`:
- Around line 39-44: The manifest's autoupdate block (architecture -> 64bit ->
url/extract_dir) should be validated before merging: run the Scoop
validation/format and install checks by executing the checkver, formatjson, and
a test install for 64bit (as described in the contrib guide) to ensure the
URL/extract_dir templating is correct and the manifest parses and installs
cleanly; fix any issues reported by .\bin\checkver.ps1 -App f3d -f,
.\bin\formatjson.ps1 -App f3d, and scoop install .\bucket\f3d.json -a 64bit.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: d72cd6ab-28d0-4900-bee0-f76e64423523

📥 Commits

Reviewing files that changed from the base of the PR and between b5f4f1f and a094130.

📒 Files selected for processing (1)

• bucket/f3d.json

[B67687]

Validated this before merge: ran ./bin/checkver.ps1 -App f3d -f and ./bin/formatjson.ps1 -App f3d, both clean with no resulting diff. I also verified the release archive layout non-invasively: the zip root matches F3D-$version-Windows-x86_64-raytracing and contains bin\\f3d.exe and bin\\F3DShellExtension.dll, so the autoupdate URL/extract_dir templating is consistent. I did not run a live scoop install .\\bucket\\f3d.json -a 64bit against my local machine because that would mutate the local Scoop install, and the manifest already passes GitHub CI.

[Copilot]

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the Scoop manifest for f3d by adding end-user guidance for enabling Explorer thumbnails and adjusting the autoupdate block.

Changes:

• Added notes instructions for registering/unregistering the shell extension DLL.
• Removed the autoupdate.hash URL-based hash lookup.

---

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