`wheels validate` incorrectly reports `Model.cfc` as invalid model

[zainforbjs]

Describe the bug
Running:
wheels validatereports an error for the default Model.cfc file:

Validation found 1 issue(s):
  [ERROR] Model.cfc — Model.cfc does not extend Model

However, the default generated Model.cfc already correctly extends wheels.Model. Current Model.cfc

/**
 * This is the parent model file that all your models should extend.
 * You can add functions to this file to make them available in all your models.
 * Do not delete this file.
 */
component extends="wheels.Model" {
}

To Reproduce
Steps to reproduce the behavior:

1. Create a fresh Wheels application
2. Run:

   wheels validate
   

3. Observe validation error for Model.cfc

Expected behavior
One of the following should happen:

Option 1 (preferred)

Model.cfc should be excluded from this validation entirely since it is the framework base model.

Option 2

If validated, the file should pass validation because:

component extends="wheels.Model"

is valid and intentional.

Actual behavior
The validator reports:

Model.cfc does not extend Model

even though it extends:

wheels.Model

Desktop

• OS: Any

[zainforbjs]

Reopening this issue because there still appears to be a false negative case in the validator.

The previous fix correctly stopped flagging the base Model.cfc, but now the validator incorrectly passes models where the valid extends="Model" declaration is commented

Example:

// component extends="Model" {
component {
	function config() {
		validatesPresenceOf("title,body,publishedAt");
	}
}

Running wheels validate returns:

Validation passed — no errors found (0 warnings)

However, the model does not actually extend Model.

If the commented line is completely removed:

component {
	function config() {
		validatesPresenceOf("title,body,publishedAt");
	}
}

Then validation correctly reports:

[ERROR] Post.cfc — Post.cfc does not extend Model

So it appears the validator is still matching commented code when checking inheritance declarations.

[wheels-bot]

Wheels Bot — Triage

Classified as bug.

Suspected layer

cli — the offending logic is in the CLI's analysis service, not framework runtime. See cli/CLAUDE.md for CLI conventions.

Fix sketch

The false negative lives in cli/lucli/services/Analysis.cfc at validateModel() (lines 341–365). The extends-check on line 347 is a raw findNoCase('extends="Model"', content) over the whole file content, so a commented-out line like // component extends="Model" { satisfies the substring match even though the active component { … } declaration below it does not extend anything. Fix shape: strip CFML comments (// …, /* … */, and <\!--- … --->) from content before the findNoCase check, or — more precisely — scan only the component declaration line itself. A small regex pre-pass + a focused test in cli/lucli/tests/specs/services/AnalysisSpec.cfc covering (a) commented-out extends with no real extends, (b) commented-out extends with a real extends="Model" below, and (c) the base Model.cfc exclusion still working, should pin the behavior down.

Confidence: high

Single file, single function, deterministic behavior, no cross-engine surface (LuCLI is Lucee-only), no security / middleware / migrator / DI involvement — none of the auto-downgrade rules fire.

<!-- wheels-bot:triage:2491 -->
<!-- wheels-bot:triage-class:bug -->
<!-- wheels-bot:triage-confidence:high -->

[wheels-bot]

Wheels Bot — Fix proposed

Draft PR: #2595

Spec: cli/lucli/tests/specs/services/AnalysisSpec.cfc — four new BDD specs cover commented-out extends="Model" (line + block forms), the legitimate commented-copy-next-to-real-extends case, and commented-out extends="Controller".
Implementation: cli/lucli/services/Analysis.cfc — validateModel() and validateController() now run their extends= substring checks against a comment-stripped copy of the file content via a small private $stripCfmlComments() helper.

A human review is required before merge. Reviewer A and Reviewer B will weigh in shortly. Doc updates (MDX guides, .ai/wheels/, CLAUDE.md) are handled separately by bot-update-docs.yml.