npm - lula2 - Versions diffs - 0.5.1 → 0.6.1-nightly.0 - Mend

lula2 0.5.1 → 0.6.1-nightly.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (28) hide show

package/dist/index.html CHANGED Viewed

@@ -6,28 +6,28 @@
 		<link rel="icon" href="/lula.png" />
 		<meta name="viewport" content="width=device-width, initial-scale=1" />
-		<link rel="modulepreload" href="/_app/immutable/entry/start.CXGxkyju.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/BNbzyHlf.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/Bvop-7hR.js">
-		<link rel="modulepreload" href="/_app/immutable/entry/app.DMBX4AyV.js">
+		<link rel="modulepreload" href="/_app/immutable/entry/start.CikNPmTv.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/DYjeAJVm.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/DTWPdvjs.js">
+		<link rel="modulepreload" href="/_app/immutable/entry/app.BMX3XH9B.js">
 		<link rel="modulepreload" href="/_app/immutable/chunks/DsnmJJEf.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/R1gz3SOr.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/zYrdvxnm.js">
-		<link rel="modulepreload" href="/_app/immutable/chunks/CPEw6sZY.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/BXUi170M.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/WlyXjfrM.js">
+		<link rel="modulepreload" href="/_app/immutable/chunks/CRa0j_Fx.js">
 	</head>
 	<body data-sveltekit-preload-data="hover">
 		<div style="display: contents">
 			<script>
 				{
-					__sveltekit_zave4b = {
+					__sveltekit_9ui1cm = {
 						base: ""
 					};
 					const element = document.currentScript.parentElement;
 					Promise.all([
-						import("/_app/immutable/entry/start.CXGxkyju.js"),
-						import("/_app/immutable/entry/app.DMBX4AyV.js")
+						import("/_app/immutable/entry/start.CikNPmTv.js"),
+						import("/_app/immutable/entry/app.BMX3XH9B.js")
 					]).then(([kit, app]) => {
 						kit.start(app, element);
 					});

package/dist/index.js CHANGED Viewed

@@ -5628,6 +5628,18 @@ function getChangedBlocks(oldText, newText) {
   }
   return changed;
 }
+function getRemovedBlocks(oldText, newText) {
+  const oldBlocks = extractMapBlocks(oldText);
+  const newBlocks = extractMapBlocks(newText);
+  const removed = [];
+  for (const oldBlock of oldBlocks) {
+    const newMatch = newBlocks.find((b) => b.uuid === oldBlock.uuid);
+    if (!newMatch) {
+      removed.push(oldBlock);
+    }
+  }
+  return removed;
+}
 function containsLulaAnnotations(text) {
   const lines = text.split("\n");
   return lines.some((line) => line.includes("@lulaStart") || line.includes("@lulaEnd"));
@@ -5708,6 +5720,7 @@ Please review whether:
           fetchRawFileViaAPI({ octokit, owner, repo, path: file.filename, ref: prBranch })
         ]);
         const changedBlocks = getChangedBlocks(oldText, newText);
+        const removedBlocks = getRemovedBlocks(oldText, newText);
         for (const block of changedBlocks) {
           console.log(`Commenting regarding \`${file.filename}\`.`);
           leavePost = true;
@@ -5723,6 +5736,43 @@ Please review whether:
 > **uuid**-\`${block.uuid}\`
  **sha256** \`${blockSha256}\`
+`;
+        }
+        if (removedBlocks.length > 0) {
+          leavePost = true;
+          console.log(`Found removed annotations in \`${file.filename}\`.`);
+          commentBody += `
+**Compliance Warning: Lula annotations were removed from \`${file.filename}\`**
+`;
+          commentBody += `The following compliance annotation blocks were present in the original file but are missing in the updated version:
+`;
+          commentBody += `| File | Original Lines | UUID |
+`;
+          commentBody += `| ---- | -------------- | ---- |
+`;
+          for (const block of removedBlocks) {
+            const oldBlockText = oldText.split("\n").slice(block.startLine, block.endLine).join("\n");
+            const blockSha256 = createHash2("sha256").update(oldBlockText).digest("hex");
+            commentBody += `| \`${file.filename}\` | \`${block.startLine + 1}\u2013${block.endLine}\` | \`${block.uuid}\` |
+`;
+            commentBody += `> **sha256** \`${blockSha256}\`
+`;
+          }
+          commentBody += `Please review whether:
+`;
+          commentBody += `- The removal of these compliance annotations is intentional
+`;
+          commentBody += `- Alternative compliance measures have been implemented
+`;
+          commentBody += `- The compliance coverage is still adequate
+`;
+          commentBody += `---
 `;
         }
       } catch (err) {
@@ -5826,7 +5876,8 @@ async function dismissOldReviews({
     if (!reviews.length) break;
     for (const r of reviews) {
       const hasSignature = (r.body ?? "").includes(LULA_SIGNATURE);
-      if (hasSignature) {
+      const isAlreadyDismissed = r.state === "DISMISSED";
+      if (hasSignature && !isAlreadyDismissed) {
         await octokit.pulls.dismissReview({
           owner,
           repo,

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "lula2",
-  "version": "0.5.1",
+  "version": "0.6.1-nightly.0",
   "description": "A tool for managing compliance as code in your GitHub repositories.",
   "bin": {
     "lula2": "./dist/lula2"
@@ -33,25 +33,6 @@
     "!dist/**/*.test.js*",
     "!dist/**/*.test.d.ts*"
   ],
-  "scripts": {
-    "dev": "vite dev --port 5173",
-    "dev:api": "tsx --watch index.ts --debug ui --port 3000 --no-open-browser",
-    "dev:full": "concurrently \"npm run dev:api\" \"npm run dev\"",
-    "build": "npm run build:svelte && npm run build:cli && npm run postbuild:cli",
-    "build:svelte": "vite build",
-    "build:cli": "esbuild index.ts cli/**/*.ts --bundle --platform=node --target=node22 --format=esm --outdir=dist --external:express --external:commander --external:js-yaml --external:yaml --external:isomorphic-git --external:glob --external:open --external:ws --external:cors --external:multer --external:@octokit/rest --external:undici --external:exceljs --external:csv-parse",
-    "postbuild:cli": "cp cli-wrapper.mjs dist/lula2 && chmod +x dist/lula2",
-    "preview": "vite preview",
-    "prepare": "svelte-kit sync || echo ''",
-    "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json && tsc --noEmit",
-    "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
-    "format": "prettier --write 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
-    "format:check": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
-    "lint": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts' && eslint src cli",
-    "test": "npm run test:unit -- --run --coverage",
-    "test:integration": "vitest --config integration/vitest.config.integration.ts",
-    "test:unit": "vitest"
-  },
   "dependencies": {
     "@octokit/rest": "^22.0.0",
     "@types/ws": "^8.18.1",
@@ -124,5 +105,23 @@
       "main",
       "next"
     ]
+  },
+  "scripts": {
+    "dev": "vite dev --port 5173",
+    "dev:api": "tsx --watch index.ts --debug ui --port 3000 --no-open-browser",
+    "dev:full": "concurrently \"npm run dev:api\" \"npm run dev\"",
+    "build": "npm run build:svelte && npm run build:cli && npm run postbuild:cli",
+    "build:svelte": "vite build",
+    "build:cli": "esbuild index.ts cli/**/*.ts --bundle --platform=node --target=node22 --format=esm --outdir=dist --external:express --external:commander --external:js-yaml --external:yaml --external:isomorphic-git --external:glob --external:open --external:ws --external:cors --external:multer --external:@octokit/rest --external:undici --external:exceljs --external:csv-parse",
+    "postbuild:cli": "cp cli-wrapper.mjs dist/lula2 && chmod +x dist/lula2",
+    "preview": "vite preview",
+    "check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json && tsc --noEmit",
+    "check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch",
+    "format": "prettier --write 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
+    "format:check": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts'",
+    "lint": "prettier --check 'src/**/*.{ts,js,svelte}' 'cli/**/*.ts' 'index.ts' 'tests/**/*.ts' && eslint src cli",
+    "test": "npm run test:unit -- --run --coverage",
+    "test:integration": "vitest --config integration/vitest.config.integration.ts",
+    "test:unit": "vitest"
   }
-}
+}

package/src/lib/components/controls/ControlsList.svelte CHANGED Viewed

@@ -119,15 +119,66 @@
 				results = results.filter((control) => {
 					// Control must match all filters
 					return $activeFilters.every(filter => {
+						// Handle mapping-related fields specially
+						if (filter.fieldName === 'has_mappings') {
+							const hasMappings = control.mappings && control.mappings.length > 0;
+							const filterValue = getFilterValue(filter.value);
+							switch (filter.operator) {
+								case 'equals':
+									return hasMappings === (filterValue === 'true' || filterValue === true);
+								case 'not_equals':
+									return hasMappings !== (filterValue === 'true' || filterValue === true);
+								case 'exists':
+									return hasMappings;
+								case 'not_exists':
+									return !hasMappings;
+								default:
+									return true;
+							}
+						}
+						if (filter.fieldName === 'mapping_status') {
+							if (!control.mappings || control.mappings.length === 0) {
+								return filter.operator === 'not_exists';
+							}
+							const filterValue = getFilterValue(filter.value);
+							const hasStatus = control.mappings.some(mapping => {
+								switch (filter.operator) {
+									case 'equals':
+										return mapping.status === filterValue;
+									case 'not_equals':
+										return mapping.status !== filterValue;
+									case 'includes':
+										return (typeof mapping.status === 'string' ? mapping.status.toLowerCase() : '').includes(String(filterValue).toLowerCase());
+									case 'not_includes':
+										return !(typeof mapping.status === 'string' ? mapping.status.toLowerCase() : '').includes(String(filterValue).toLowerCase());
+									default:
+										return true;
+								}
+							});
+							switch (filter.operator) {
+								case 'exists':
+									return control.mappings.some(m => m.status !== undefined && m.status !== null);
+								case 'not_exists':
+									return !control.mappings.some(m => m.status !== undefined && m.status !== null);
+								default:
+									return hasStatus;
+							}
+						}
+						// Handle regular control fields
 						const dynamicControl = control as Record<string, unknown>;
 						const fieldValue = dynamicControl[filter.fieldName];
+						const filterValue = getFilterValue(filter.value);
 						switch (filter.operator) {
 							case 'equals':
-								return fieldValue === filter.value;
+								return fieldValue === filterValue;
 							case 'not_equals':
-								return fieldValue !== filter.value;
+								return fieldValue !== filterValue;
 							case 'exists':
 								return fieldValue !== undefined && fieldValue !== null && fieldValue !== '';
@@ -137,20 +188,20 @@
 							case 'includes':
 								if (typeof fieldValue === 'string') {
-									return fieldValue.toLowerCase().includes(String(filter.value).toLowerCase());
+									return fieldValue.toLowerCase().includes(String(filterValue).toLowerCase());
 								} else if (Array.isArray(fieldValue)) {
 									return fieldValue.some(item =>
-										String(item).toLowerCase().includes(String(filter.value).toLowerCase())
+										String(item).toLowerCase().includes(String(filterValue).toLowerCase())
 									);
 								}
 								return false;
 							case 'not_includes':
 								if (typeof fieldValue === 'string') {
-									return !fieldValue.toLowerCase().includes(String(filter.value).toLowerCase());
+									return !fieldValue.toLowerCase().includes(String(filterValue).toLowerCase());
 								} else if (Array.isArray(fieldValue)) {
 									return !fieldValue.some(item =>
-										String(item).toLowerCase().includes(String(filter.value).toLowerCase())
+										String(item).toLowerCase().includes(String(filterValue).toLowerCase())
 									);
 								}
 								return true;
@@ -198,6 +249,28 @@
 		return 'No description available';
 	}
+	function formatFilterValue(value: any): string {
+		if (value === null || value === undefined) {
+			return '';
+		}
+		if (typeof value === 'object') {
+			if (Array.isArray(value)) {
+				return value.join(', ');
+			}
+			if ('value' in value) {
+				return String((value as any).value);
+			}
+			return JSON.stringify(value);
+		}
+		return String(value);
+	}
+	function getFilterValue(filterValue: any): any {
+		if (typeof filterValue === 'object' && filterValue !== null && 'value' in filterValue) {
+			return filterValue.value;
+		}
+		return filterValue;
+	}
 	// Get truncation length based on field type
 	function getTruncationLength(field: FieldSchema | undefined): number {
 		if (field?.ui_type === 'textarea' || field?.ui_type === 'long_text') {
@@ -258,11 +331,11 @@
 							{#if filter.operator === 'exists' || filter.operator === 'not_exists'}
 								<span>{getOperatorLabel(filter.operator).toLowerCase()}</span>
 							{:else if filter.operator === 'equals'}
-								<span>= {filter.value}</span>
+								<span>= {formatFilterValue(filter.value)}</span>
 							{:else if filter.operator === 'not_equals'}
-								<span>≠ {filter.value}</span>
+								<span>≠ {formatFilterValue(filter.value)}</span>
 							{:else}
-								<span>{getOperatorLabel(filter.operator).toLowerCase()} "{filter.value}"</span>
+								<span>{getOperatorLabel(filter.operator).toLowerCase()} "{formatFilterValue(filter.value)}"</span>
 							{/if}
 							<button
 								onclick={() => complianceStore.removeFilter(index)}

package/src/lib/components/ui/FilterBuilder.svelte CHANGED Viewed

@@ -9,7 +9,8 @@
 		type FilterValue,
 		activeFilters,
 		FILTER_OPERATORS,
-		getOperatorLabel
+		getOperatorLabel,
+		MAPPING_STATUS_OPTIONS
 	} from '$stores/compliance';
 	import { appState } from '$lib/websocket';
 	import { Filter, Add, TrashCan, ChevronDown, ChevronUp } from 'carbon-icons-svelte';
@@ -37,13 +38,18 @@
 	// Get field type for the selected field
 	$: selectedFieldSchema = $fieldSchema[newFilterField] || null;
-	$: selectedFieldType = selectedFieldSchema?.type || 'string';
-	$: selectedFieldUiType = selectedFieldSchema?.ui_type || 'short_text';
+	$: selectedFieldType =
+		getMappingFieldType(newFilterField) || selectedFieldSchema?.type || 'string';
+	$: selectedFieldUiType =
+		getMappingFieldUiType(newFilterField) || selectedFieldSchema?.ui_type || 'short_text';
 	$: isSelectField = selectedFieldUiType === 'select';
-	$: fieldOptions = isSelectField ? selectedFieldSchema?.options || [] : [];
+	$: fieldOptions =
+		getMappingFieldOptions(newFilterField) ||
+		(isSelectField ? selectedFieldSchema?.options || [] : []);
-	// Force equals operator for select fields
-	$: if (isSelectField) {
+	// Force equals operator for select fields (but allow operators for mapping fields)
+	// Also force equals operator for has_mappings field, but allow operators for mapping_status
+	$: if (isSelectField && newFilterField !== 'mapping_status') {
 		newFilterOperator = 'equals';
 	}
@@ -51,6 +57,7 @@
 	$: fieldsByTab = {
 		overview: [] as string[],
 		implementation: [] as string[],
+		mappings: [] as string[],
 		custom: [] as string[]
 	};
@@ -58,19 +65,25 @@
 		// Reset arrays before populating
 		fieldsByTab.overview = [];
 		fieldsByTab.implementation = [];
+		fieldsByTab.mappings = [];
 		fieldsByTab.custom = [];
 		// Group fields by tab
 		availableFields.forEach((field) => {
-			const schema = $fieldSchema[field];
-			if (schema) {
-				const tab = schema.tab || getDefaultTabForCategory(schema.category);
-				if (tab === 'overview') fieldsByTab.overview.push(field);
-				else if (tab === 'implementation') fieldsByTab.implementation.push(field);
-				else fieldsByTab.custom.push(field);
+			// Handle mapping-related fields specially
+			if (field === 'has_mappings' || field === 'mapping_status') {
+				fieldsByTab.mappings.push(field);
 			} else {
-				// If no schema, default to custom
-				fieldsByTab.custom.push(field);
+				const schema = $fieldSchema[field];
+				if (schema) {
+					const tab = schema.tab || getDefaultTabForCategory(schema.category);
+					if (tab === 'overview') fieldsByTab.overview.push(field);
+					else if (tab === 'implementation') fieldsByTab.implementation.push(field);
+					else fieldsByTab.custom.push(field);
+				} else {
+					// If no schema, default to custom
+					fieldsByTab.custom.push(field);
+				}
 			}
 		});
 	}
@@ -81,12 +94,23 @@
 	// Create a new array from the readonly constant to make it mutable for Svelte
 	const operatorOptions = FILTER_OPERATORS.map((op) => ({ value: op.value, label: op.label }));
+	// Limited operators for mapping status field
+	const mappingStatusOperatorOptions = [
+		{ value: 'equals' as const, label: 'Equals' },
+		{ value: 'not_equals' as const, label: 'Not equals' }
+	];
 	// Add a new filter
 	function addFilter() {
 		if (!newFilterField) return;
 		let value = newFilterValue;
+		// Extract value from dropdown objects if necessary
+		if (typeof value === 'object' && value !== null && 'value' in value) {
+			value = (value as any).value;
+		}
 		// Convert value based on field type
 		let processedValue: FilterValue = value;
 		if (selectedFieldType === 'boolean' && typeof value === 'string') {
@@ -125,8 +149,54 @@
 		}
 	}
+	// Centralized mapping field metadata
+	const mappingFieldConfig: Record<
+		string,
+		{
+			type: string;
+			ui_type: string;
+			options?: Array<{ value: string; label: string }>;
+		}
+	> = {
+		has_mappings: {
+			type: 'boolean',
+			ui_type: 'select',
+			options: [
+				{ value: 'true', label: 'Yes' },
+				{ value: 'false', label: 'No' }
+			]
+		},
+		mapping_status: {
+			type: 'string',
+			ui_type: 'select',
+			options: MAPPING_STATUS_OPTIONS
+		}
+	};
+	function getMappingFieldType(fieldName: string): string | null {
+		return mappingFieldConfig[fieldName]?.type ?? null;
+	}
+	function getMappingFieldUiType(fieldName: string): string | null {
+		return mappingFieldConfig[fieldName]?.ui_type ?? null;
+	}
+	function getMappingFieldOptions(
+		fieldName: string
+	): Array<{ value: string; label: string }> | null {
+		return mappingFieldConfig[fieldName]?.options ?? null;
+	}
 	// Get display name for a field
 	function getFieldDisplayName(fieldName: string): string {
+		// Handle mapping fields specially
+		switch (fieldName) {
+			case 'has_mappings':
+				return 'Has Mappings';
+			case 'mapping_status':
+				return 'Mapping Status';
+		}
 		const schema = $fieldSchema[fieldName];
 		// Use schema names if available
@@ -145,9 +215,27 @@
 		if (filter.operator === 'exists') return 'exists';
 		if (filter.operator === 'not_exists') return 'does not exist';
+		// Convert value to string, handling objects and arrays properly
+		let displayValue = '';
+		if (filter.value === null || filter.value === undefined) {
+			displayValue = '';
+		} else if (typeof filter.value === 'object') {
+			// Handle objects by converting to JSON or getting a meaningful representation
+			if (Array.isArray(filter.value)) {
+				displayValue = filter.value.join(', ');
+			} else if ('value' in filter.value) {
+				// Handle dropdown option objects
+				displayValue = String((filter.value as any).value);
+			} else {
+				displayValue = JSON.stringify(filter.value);
+			}
+		} else {
+			displayValue = String(filter.value);
+		}
 		// Use the shared getOperatorLabel function
 		const operatorText = getOperatorLabel(filter.operator).toLowerCase();
-		return `${operatorText} "${filter.value}"`;
+		return `${operatorText} "${displayValue}"`;
 	}
 	// Handle click outside to close the panel
@@ -306,6 +394,33 @@
 										{/each}
 									{/if}
+									<!-- Mapping fields -->
+									{#if fieldsByTab.mappings.length > 0}
+										<!-- Divider -->
+										<div class="border-t border-gray-200 dark:border-gray-700 my-1"></div>
+										<div
+											class="px-3 py-1 text-xs font-semibold text-gray-500 dark:text-gray-400 bg-gray-50 dark:bg-gray-800/80"
+										>
+											Mapping Fields
+										</div>
+										{#each fieldsByTab.mappings as field (field)}
+											<button
+												class={twMerge(
+													'w-full text-left px-3 py-2 text-sm',
+													newFilterField === field
+														? 'bg-blue-50 dark:bg-blue-900/20 text-blue-700 dark:text-blue-300'
+														: 'hover:bg-gray-100 dark:hover:bg-gray-700/50'
+												)}
+												onclick={() => {
+													newFilterField = field;
+													showFieldDropdown = false;
+												}}
+											>
+												{getFieldDisplayName(field)}
+											</button>
+										{/each}
+									{/if}
 									<!-- Custom fields -->
 									{#if fieldsByTab.custom.length > 0}
 										<!-- Divider -->
@@ -343,8 +458,8 @@
 							>Operator</label
 						>
-						{#if isSelectField}
-							<!-- Disabled dropdown for select fields (always equals) -->
+						{#if (isSelectField && !['has_mappings', 'mapping_status'].includes(newFilterField)) || newFilterField === 'has_mappings'}
+							<!-- Disabled dropdown for select fields and has_mappings (always equals) -->
 							<div
 								class="px-3 py-2 text-sm rounded-md border border-gray-300 dark:border-gray-600 bg-gray-100 dark:bg-gray-800 text-gray-500 dark:text-gray-400"
 							>
@@ -354,7 +469,9 @@
 							<!-- Custom Operator Dropdown -->
 							<CustomDropdown
 								bind:value={newFilterOperator}
-								options={operatorOptions}
+								options={newFilterField === 'mapping_status'
+									? mappingStatusOperatorOptions
+									: operatorOptions}
 								getDisplayValue={getOperatorLabel}
 								labelId="filter-operator"
 							/>
@@ -367,7 +484,30 @@
 							<label for="filter-value" class="block text-xs text-gray-600 dark:text-gray-400 mb-1"
 								>Value</label
 							>
-							{#if isSelectField}
+							{#if newFilterField === 'mapping_status'}
+								<!-- Special dropdown for mapping status -->
+								<select
+									id="filter-value"
+									bind:value={newFilterValue}
+									class="w-full rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 text-gray-900 dark:text-white text-sm"
+								>
+									<option value="">Select status...</option>
+									{#each MAPPING_STATUS_OPTIONS as status}
+										<option value={status.value}>{status.label}</option>
+									{/each}
+								</select>
+							{:else if newFilterField === 'has_mappings'}
+								<!-- Special dropdown for has_mappings -->
+								<select
+									id="filter-value"
+									bind:value={newFilterValue}
+									class="w-full rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 text-gray-900 dark:text-white text-sm"
+								>
+									<option value="">Select...</option>
+									<option value="true">Yes</option>
+									<option value="false">No</option>
+								</select>
+							{:else if isSelectField}
 								<!-- Custom dropdown for select fields -->
 								<CustomDropdown
 									bind:value={newFilterValue}
@@ -377,15 +517,15 @@
 								/>
 							{:else if selectedFieldType === 'boolean'}
 								<!-- Boolean field with CustomDropdown -->
-								<CustomDropdown
+								<select
+									id="filter-value"
 									bind:value={newFilterValue}
-									options={[
-										{ value: 'true', label: 'Yes' },
-										{ value: 'false', label: 'No' }
-									]}
-									placeholder="Select yes/no"
-									labelId="filter-value"
-								/>
+									class="w-full rounded-md border border-gray-300 dark:border-gray-600 bg-white dark:bg-gray-700 text-gray-900 dark:text-white text-sm"
+								>
+									<option value="">Select...</option>
+									<option value="true">Yes</option>
+									<option value="false">No</option>
+								</select>
 							{:else}
 								<!-- Text input for other fields -->
 								<input

package/src/lib/types.ts CHANGED Viewed

@@ -1,5 +1,6 @@
 // SPDX-License-Identifier: Apache-2.0
 // SPDX-FileCopyrightText: 2023-Present The Lula Authors
 export interface Control {
 	id: string;
 	title: string;

package/src/stores/compliance.ts CHANGED Viewed

@@ -5,6 +5,15 @@ import type { Control, Mapping } from '$lib/types';
 import { appState } from '$lib/websocket';
 import { get, writable } from 'svelte/store';
+/**
+ * Shared mapping status options used across the application
+ */
+export const MAPPING_STATUS_OPTIONS = [
+	{ value: 'planned', label: 'Planned' },
+	{ value: 'implemented', label: 'Implemented' },
+	{ value: 'verified', label: 'Verified' }
+];
 /**
  * Shared filter operator options used across the application
  */
@@ -133,6 +142,10 @@ export const complianceStore = {
 			});
 		}
+		// Add mapping-related fields for filtering
+		fieldSet.add('has_mappings');
+		fieldSet.add('mapping_status');
 		return Array.from(fieldSet).sort();
 	}
 };