fix(security): 2 improvements across 2 files

bin/ensure-ai-cookbook.js

@@ -4,7 +4,13 @@ const { existsSync, mkdirSync, writeFileSync } = require('fs');
 const path = require('path');
 
 const WORKSPACE_ROOT = path.resolve(__dirname, '..');
-const OUTPUT_DIR = path.join(WORKSPACE_ROOT, process.env.AI_COOKBOOK_OUTPUT_DIR ?? 'ai-cookbook');
+const outputDirFromEnv = process.env.AI_COOKBOOK_OUTPUT_DIR ?? 'ai-cookbook';
+const OUTPUT_DIR = path.resolve(WORKSPACE_ROOT, outputDirFromEnv);
+
+if (OUTPUT_DIR !== WORKSPACE_ROOT && !OUTPUT_DIR.startsWith(`${WORKSPACE_ROOT}${path.sep}`)) {
+  throw new Error('[ensure-ai-cookbook] AI_COOKBOOK_OUTPUT_DIR must resolve within the repository root');
+}
+
 const PLACEHOLDER_PATH = path.join(OUTPUT_DIR, 'recipes-not-synced.mdx');
 
 // Ensure ai-cookbook directory exists with a placeholder file.

src/components/elements/CallToAction.js

@@ -1,9 +1,33 @@
 import React from 'react';
 import styles from './call-to-action.module.css';
 
+ const getSafeHref = (href) => {
+   if (typeof href !== 'string') {
+     return '#';
+   }
+
+   const value = href.trim();
+
+   if (!value || value.startsWith('/') || value.startsWith('#') || value.startsWith('?')) {
+     return value || '#';
+   }
+
+   const lowerValue = value.toLowerCase();
+
+   if (
+     lowerValue.startsWith('http://') ||
+     lowerValue.startsWith('https://') ||
+     lowerValue.startsWith('mailto:')
+   ) {
+     return value;
+   }
+
+   return '#';
+ };
+
  export const CallToAction = ({ href, children }) => {
    return (
-     <a href={href} className={styles.cta}>
+     <a href={getSafeHref(href)} className={styles.cta}>
        <div className={styles.content}>
          {children}
        </div>