Various linting enhancements

.remarkrc.mjs

@@ -1,7 +1,13 @@
 import remarkPresetLintRecommended from 'remark-preset-lint-recommended';
 import remarkFrontmatter from 'remark-frontmatter';
 import remarkMdx from 'remark-mdx';
+import remarkNoInlineCodeFences from './src/plugins/remark-no-inline-code-fences.mjs';
 
 export default {
-    plugins: [remarkFrontmatter, remarkMdx, remarkPresetLintRecommended],
+    plugins: [
+        remarkFrontmatter,
+        remarkMdx,
+        remarkPresetLintRecommended,
+        remarkNoInlineCodeFences,
+    ],
 };

eslint.config.mjs

@@ -17,7 +17,7 @@ export default [
                 'warn',
                 { argsIgnorePattern: '^_' },
             ],
-            '@typescript-eslint/no-explicit-any': 'warn',
+            '@typescript-eslint/no-explicit-any': 'error',
         },
     },
 

src/components/Slides.astro

@@ -878,9 +878,19 @@ const slideId = `slides-${Math.random().toString(36).substring(2, 11)}`;
 </style>
 
 <script>
+    interface SlideClientData {
+        url: string;
+        type: string;
+        alt: string;
+        caption: string;
+        width: number | null;
+        height: number | null;
+        attrs: Record<string, string | undefined>;
+    }
+
     function initSlides() {
         document.querySelectorAll('.slides-component').forEach((component) => {
-            const slidesData = JSON.parse(
+            const slidesData: SlideClientData[] = JSON.parse(
                 component.getAttribute('data-slides') || '[]',
             );
             // Skip if already initialized
@@ -924,7 +934,7 @@ const slideId = `slides-${Math.random().toString(36).substring(2, 11)}`;
             let lightboxIndex = 0;
             let imagesLoaded = 0;
             const totalImages = slidesData.filter(
-                (s: any) => s.type === 'image',
+                (s) => s.type === 'image',
             ).length;
 
             // Track image dimensions for scaling
@@ -950,7 +960,7 @@ const slideId = `slides-${Math.random().toString(36).substring(2, 11)}`;
                 });
 
                 // Also consider YouTube videos as 16:9
-                slidesData.forEach((slide: any) => {
+                slidesData.forEach((slide) => {
                     if (slide.type === 'youtube') {
                         const youtubeAspectRatio = 16 / 9;
                         if (youtubeAspectRatio > widestAspectRatio) {

src/content/docs/contribution/methodsOfContributing.mdx

@@ -12,6 +12,8 @@ import Aside from '../../../components/Aside.astro';
 FRCSoftware.org is just getting started, so the easiest way to propose work is by opening a GitHub issue in the main repository.
 Use this template in your issue:
 
+{/* rli:ignore */}
+
 ```
 Issue/content:
 Solution or Notes about the execution of the content:

src/content/docs/contribution/styleguide.mdx

@@ -110,8 +110,18 @@ Enforced by `prettier-plugin-sentences-per-line`.
 
 Run all checks locally before pushing:
 
+{/* rli:ignore */}
+
 ```bash
 pnpm lint && pnpm format:check
 ```
 
 If `pnpm format:check` fails, run `pnpm prettier --write .` to auto-fix.
+
+#### RLI (region-based code fences)
+
+Code fences must use the RLI system: ` ```language path/to/file#regionName ` with an empty body.
+The referenced file lives under `examples/` and uses `// [regionName]` / `// [/regionName]` markers to delimit the snippet.
+This keeps example code in real source files that are compiled and tested.
+
+If you need an inline code fence that doesn't reference an example file, suppress the rule by placing `{/* rli:ignore */}` on the line immediately before it.

src/content/docs/intro-to-java/java-fundamentals.mdx

@@ -33,6 +33,8 @@ This could be a variable that holds the temperature or a variable that holds the
 
 Variable declarations have the following syntax:
 
+{/* rli:ignore */}
+
 ```java
 Datatype name = value;
 ```

src/plugins/remark-code-region.ts

@@ -7,7 +7,7 @@ export default function remarkCodeRegion() {
     return (tree: Root) => {
         const examplesDir = resolve(process.cwd(), 'examples');
 
-        visit(tree, 'code', (node: any) => {
+        visit(tree, 'code', (node) => {
             const meta: string = node.meta || '';
 
             const token = meta.match(/^(\S+)/);

src/plugins/remark-image-attributes.ts

@@ -14,21 +14,22 @@
  */
 
 import { visit } from 'unist-util-visit';
-import type { Root, Image } from 'mdast';
+import type { Root } from 'mdast';
 
-interface ImageNode extends Image {
+interface ImageNode {
+    url: string;
     data?: {
         hName?: string;
-        hProperties?: Record<string, any>;
+        hProperties?: Record<string, unknown>;
     };
 }
 
 interface ParentNode {
     type: string;
-    children: any[];
+    children: { type: string }[];
     data?: {
         hName?: string;
-        hProperties?: Record<string, any>;
+        hProperties?: Record<string, unknown>;
     };
 }
 
@@ -51,7 +52,7 @@ export function remarkImageAttributes() {
                     const child = children[i];
 
                     if (child.type === 'image') {
-                        const imageNode = child as ImageNode;
+                        const imageNode = child as unknown as ImageNode;
                         const url = imageNode.url;
 
                         // Check for hash in URL

src/plugins/remark-mdx-global-imports.ts

@@ -11,6 +11,16 @@
 import type { Root } from 'mdast';
 import type { VFile } from 'vfile';
 
+declare module 'mdast' {
+    interface RootContentMap {
+        mdxjsEsm: {
+            type: 'mdxjsEsm';
+            value: string;
+            data?: { estree?: unknown };
+        };
+    }
+}
+
 const GLOBAL_IMPORTS = [
     { name: 'ContentFigure', path: '@components/ContentFigure.astro' },
     { name: 'Aside', path: '@components/Aside.astro' },
@@ -54,10 +64,8 @@ export function remarkMdxGlobalImports() {
 
         const existingNames = new Set<string>();
         for (const node of tree.children) {
-            if ((node as any).type === 'mdxjsEsm') {
-                const match = (node as any).value?.match(
-                    /\bimport\s+(\w+)\s+from\b/,
-                );
+            if (node.type === 'mdxjsEsm') {
+                const match = node.value?.match(/\bimport\s+(\w+)\s+from\b/);
                 if (match) existingNames.add(match[1]);
             }
         }
@@ -67,7 +75,7 @@ export function remarkMdxGlobalImports() {
         ).map(({ name, path }) => makeImportNode(name, path));
 
         if (toInsert.length > 0) {
-            tree.children.unshift(...(toInsert as any[]));
+            tree.children.unshift(...toInsert);
         }
     };
 }

src/plugins/remark-no-inline-code-fences.mjs

@@ -0,0 +1,29 @@
+import { visit } from 'unist-util-visit';
+
+const IGNORE_RE = /rli:\s*ignore/;
+
+export default function remarkNoInlineCodeFences() {
+    return (tree, file) => {
+        visit(tree, 'code', (node, index, parent) => {
+            if (index !== undefined && parent) {
+                const prev = parent.children[index - 1];
+                if (
+                    prev?.type === 'mdxFlowExpression' &&
+                    IGNORE_RE.test(prev.value)
+                ) {
+                    return;
+                }
+            }
+
+            if (node.meta?.trim()) return;
+
+            if (node.value?.trim()) {
+                const msg = file.message(
+                    'Code fences must use the RLI system: ```language path/to/file#regionName',
+                    node,
+                );
+                msg.fatal = true;
+            }
+        });
+    };
+}