also skip required entity checks on derivative files

Author: rwblair

changelog.d/20250605_155304_rosswilsonblair_valid_missing_entities.md

@@ -11,16 +11,16 @@ For top level release notes, leave all the headers commented out.
 - A bullet item for the Added category.
 
 -->
+<!--
 ### Changed
 
-- Categorically prevent required entites on metadata files to more closely follow inheritance principle.
 
-<!--
+-->
 ### Fixed
 
-- A bullet item for the Fixed category.
+- Categorically prevent required entites on metadata files to more closely follow inheritance principle.
+- Also skip required entities on all derivative files.
 
--->
 <!--
 ### Deprecated
 

src/validators/filenameValidate.test.ts

@@ -2,7 +2,7 @@ import type { FileTree } from '../types/filetree.ts'
 import type { GenericSchema } from '../types/schema.ts'
 import { assertEquals } from '@std/assert'
 import { BIDSContext } from '../schema/context.ts'
-import { type atRoot, type entityLabelCheck, missingLabel } from './filenameValidate.ts'
+import { type entityLabelCheck, missingLabel } from './filenameValidate.ts'
 import type { BIDSFileDeno } from '../files/deno.ts'
 import { pathToFile } from '../files/filetree.ts'
 import type { FileIgnoreRules } from '../files/ignore.ts'

src/validators/filenameValidate.ts

@@ -187,7 +187,8 @@ function entityRuleIssue(
   const ruleEntities = Object.keys(rule.entities).map((key) => lookupEntityLiteral(key, schema))
 
   // skip required entity checks for 'metadata' per inheritance rules 1 + 2
-  if (!sidecarExtensions.includes(context.extension)) {
+  // Also skip for all deriv files.
+  if (!sidecarExtensions.includes(context.extension) && !path.includes('rules.deriv.')) {
     const ruleEntitiesRequired = Object.entries(rule.entities)
       .filter(([_, v]) => v === 'required')
       .map(([k, _]) => lookupEntityLiteral(k, schema))