Skip to content

Fix dotvvm lint: master pages, --project option, file filtering, metadata caching - #2095

Merged
tomasherceg merged 6 commits into
mainfrom
copilot/dotvvm-cli-lint-issues
Aug 21, 2026
Merged

Fix dotvvm lint: master pages, --project option, file filtering, metadata caching#2095
tomasherceg merged 6 commits into
mainfrom
copilot/dotvvm-cli-lint-issues

Conversation

Copilot AI commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

Three issues with dotvvm lint: master pages were silently skipped, the project path was a positional arg (not AI-friendly), and every run triggered a slow MSBuild metadata rebuild.

Master pages now linted

StaticViewCompiler.CompileAll scans the project directory for .dotmaster files and compiles them alongside registered markup controls and route views. A set tracks already-compiled paths to avoid duplicates.

--project option + per-file argument

The lint command now accepts --project (option, defaults to CWD) instead of the old positional target argument. A new optional positional argument accepts one or more specific .dothtml/.dotcontrol/.dotmaster paths — only diagnostics for those files are reported:

dotvvm lint --project ./MyApp Views/Errors/Bad.dothtml Views/Shared/Site.dotmaster

Other commands retain the existing target positional argument unchanged.

Metadata caching (dotvvm-cli.json)

DotvvmProject.FromCsproj now skips the MSBuild _WriteDotvvmProjectMetadata invocation when obj/dotvvm-cli.json is newer than the .csproj. A --force-build flag forces regeneration when needed.

Compiler-side file filtering

DotVVM.Compiler gained a --files option (passed after the two positional args) that filters diagnostic output to the specified virtual paths. The CLI converts user-supplied absolute paths to project-relative virtual paths before forwarding them.

Copilot AI linked an issue Aug 20, 2026 that may be closed by this pull request
Copilot AI and others added 2 commits August 20, 2026 14:03
…filtering, build cache

Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
…e error message

Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix DotVVM CLI lint issues for master pages and performance Fix dotvvm lint: master pages, --project option, file filtering, metadata caching Aug 20, 2026
Copilot AI requested a review from tomasherceg August 20, 2026 14:09
Comment thread src/Framework/Framework/Compilation/Static/StaticViewCompiler.cs Outdated
…not filesystem scan

Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
Comment thread src/Framework/Framework/Compilation/Static/StaticViewCompiler.cs
Co-authored-by: tomasherceg <5599524+tomasherceg@users.noreply.github.com>
@tomasherceg
tomasherceg marked this pull request as ready for review August 21, 2026 13:03
@tomasherceg
tomasherceg merged commit 66f4395 into main Aug 21, 2026
16 of 17 checks passed
@tomasherceg
tomasherceg deleted the copilot/dotvvm-cli-lint-issues branch August 21, 2026 13:04
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.

DotVVM CLI lint issues

3 participants