microsoft / microsoft/ApplicationInspector

Paired language rules?

Open
#662 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
C#
Stars
4.4k
Forks
366
Avg merge
6d 7h
Merged PRs (30d)
3

Description

Is your feature request related to a problem? Please describe.
Sometimes there's an overlap where rules could apply to multiple kinds of languages, i.e. C# rules can also apply to Razor pages, but Razor has unique things that don't apply to C#. Maintaining the language lists to always include C# in every razor rule is onerous, could this be solved in the languages.json definition perhaps and resolved by the rule engine when selecting rules by language to include sublangauges. I.e. when hitting a Razor language defined file also pick up C# rules?

Additional context
Not sure how many other similar cases there are/the importance here. If super niche to just Razor maybe not worth fixing.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start with the languages.json definition and the rule engine's language-selection path. Compare how Razor and C# are represented, then determine how paired or sublanguage relationships should affect rule selection. Done means the relationship is represented without duplicating C# in every Razor rule and works for the described Razor case.

Written by the indexing model from the issue text.

Assessment

Tech stack
csharp
Domain
devtools, security
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.