aegisub: Persist config.json

[emotion3459]

Summary

Aegisub uses config.json to store misc settings. The manifest needs to persist this for most settings to carry over versions. We do not need to create the file ourselves as the portable zip comes with a pre-configured config.

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

[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: 21041dea-9987-4c97-92c2-cf79f084e818

📥 Commits

Reviewing files that changed from the base of the PR and between a817c5f and ea6d0d9.

📒 Files selected for processing (1)

• bucket/aegisub.json

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

• bucket/aegisub.json

---

📝 Walkthrough

Walkthrough

Updates the Scoop manifest bucket/aegisub.json: replaces a plain license URL with a structured license object, adds a suggest entry for vcredist2022, and introduces PowerShell lifecycle hooks (pre_install and pre_uninstall) that copy config.json between the app directory and $persist_dir (creating $persist_dir when needed) to preserve user configuration across installs/uninstalls.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• notepad4: Fix persistence #17289: Adds pre_install lifecycle hooks in a Scoop manifest to copy default config files into the app directory and adjust persistence handling.
• prime95@30.19b20: Persist results.txt & prime.txt #16911: Introduces pre-install persistence handling for app config/log files in a Scoop manifest, similar config preservation pattern.
• adventuregamestudio: Update to version 3.6.2.14, fix checkver & autoupdate, add suggest #16436: Converts manifest license to a structured object and adds a suggest entry for vcredist2022, matching the metadata changes here.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely summarizes the main change: persisting the config.json file for the Aegisub package.
Description check	✅ Passed	The description provides a clear summary of the changes and confirms adherence to the repository's conventional PR title and contributing guide requirements.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ 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

_{Review rate limit: 7/8 reviews remaining, refill in 7 minutes and 30 seconds.}

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

[coderabbitai]

🧹 Nitpick comments (1)

bucket/aegisub.json (1)

1-36: Run local manifest lint + a test install.

Before merging, please run Scoop’s manifest lint/verification for aegisub and do at least one install to confirm persistence behavior:

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

Also double-check the committed JSON uses CRLF line endings (Scoop bucket manifests require CRLF). (Based on coding guidelines: Scoop manifest CRLF + local test guidance.)

🤖 Prompt for AI Agents

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

In `@bucket/aegisub.json` around lines 1 - 36, Run Scoop manifest lint and a local
install test for aegisub.json: run .\bin\checkver.ps1 -App aegisub -f, then
.\bin\formatjson.ps1 -App aegisub, and finally verify installation with scoop
install .\bucket\aegisub.json -a 64bit to confirm persistence behavior; also
ensure the committed aegisub.json uses CRLF line endings (convert/save with
CRLF) before committing.

🤖 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/aegisub.json`:
- Around line 1-36: Run Scoop manifest lint and a local install test for
aegisub.json: run .\bin\checkver.ps1 -App aegisub -f, then .\bin\formatjson.ps1
-App aegisub, and finally verify installation with scoop install
.\bucket\aegisub.json -a 64bit to confirm persistence behavior; also ensure the
committed aegisub.json uses CRLF line endings (convert/save with CRLF) before
committing.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 19c40240-5bd7-4167-ad00-08acee815a0d

📥 Commits

Reviewing files that changed from the base of the PR and between d707c61 and 0186faa.

📒 Files selected for processing (1)

• bucket/aegisub.json

[emotion3459]

/verify

[github-actions]

All changes look good.

Wait for review from human collaborators.

aegisub

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

Check the full log for details.

[z-Fng]

/verify

[github-actions]

All changes look good.

Wait for review from human collaborators.

aegisub

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

Check the full log for details.

[z-Fng]

Thanks for your contribution!