Add 'Do not show again' checkbox to lint confirmation modals

[jancbeck]

Summary

• Adds a "Do not show this confirmation again" checkbox to the lint confirmation modals (lint-all-files and lint-all-files-in-folder)
• When checked and submitted, persists the preference to data.json so future invocations skip the modal
• Single shared flag for both commands
• No standalone settings toggle — the checkbox on the modal itself is the only UI surface

Motivation

Closes #1474

The confirmation modals block automation workflows — running these commands via Obsidian CLI, MCP, or other programmatic invocations shows a dialog that can't be dismissed without user interaction.

Rather than a standalone settings toggle, the preference is set directly from the modal via a "Do not show again" checkbox, which is more discoverable and follows common UI patterns.

Test plan

• Run "Lint all files" → modal shows checkbox before buttons
• Check the box + click Submit → next invocation skips modal entirely
• Same behavior for "Lint all files in folder"
• Preference persists across plugin reloads
• Checkbox absent when saveSuppressPreference callback not provided
• Setting no longer appears in General settings tab

🤖 Generated with Claude Code

[pjkaufman]

I would prefer this be a "do not show again" checkbox on both the folder and lint all modals. It would get stored in the data.json in order to persist it, but it would not be its own setting.

[pjkaufman]

Going through the CLI will need to be handled via a CLI handler or what have you in order to prevent even asking for the UI confirmation. But that will have to come separately.

[jancbeck]

@pjkaufman updated PR.

[pjkaufman]

Sorry about the delay. I will go ahead and take a look and see how things look and function.

[pjkaufman]

My main concern from just looking at the code is that the code uses the same logic to suppress both modals. But that may not be the intent. I would expect each modal to have its own setting for suppressing seeing it. That hopefully would not be too bad to implement. But I will look at the actual way in which it works in the app.

[pjkaufman]

Well, it looks like it may have just been added to linting all files instead of also for linting a folder. I will double check this though.

Edit: operator error here. I needed to swap to the branch and check. It looks like both modals do indeed use the same setting, so that would need changing. I can see wanting to suppress the modal for folder lints, but not the lint all functionality as well.

[pjkaufman]

I can attest to the logic working. So it just needs to use one boolean for each of the actual modals (folder and all files).

[pjkaufman]

Looks and seems to work good except for one checkbox being used for both modals. If we can get that swapped over to two settings instead of one, this should be good for merging.

[jancbeck]

Split the single suppressLintConfirmationModal boolean into two separate booleans:

• suppressLintAllFilesConfirmationModal — for the lint-all-files modal
• suppressLintAllFilesInFolderConfirmationModal — for the lint-folder modal

Each modal's "Do not show again" checkbox now only affects that specific modal. Added migration in loadSettings() to carry over the old setting for existing users.

[aamikus]

this is a useful change for agentic workflows. let's get this merged

[pjkaufman]

Looks good now that I removed the logic around migrating a setting that does not exist.