@checkstack/ui 0.2.3 → 0.3.0

package/CHANGELOG.md

@@ -1,5 +1,53 @@
 # @checkstack/ui
 
+## 0.3.0
+
+### Minor Changes
+
+- 83557c7: ## CodeEditor Multi-Language Support
+
+  - **Refactored CodeEditor** into modular architecture with language-specific support
+  - **Added language modes**: JSON, YAML, XML, and Markdown with custom indentation and syntax highlighting
+  - **Smart Enter key behavior**: Bracket/tag splitting (e.g., `<div></div>` → proper split on Enter)
+  - **Autocomplete fix**: Enter key now correctly selects completions instead of inserting newlines
+  - **Click area fix**: Entire editor area is now clickable (per official CodeMirror minHeight docs)
+  - **Line numbers**: Now visible with proper gutter styling
+  - **185 comprehensive tests** for all language indentation and template position validation
+
+- 6dbfab8: Replace react-simple-code-editor with @uiw/react-codemirror for better maintenance and features. Added new `CodeEditor` component as a reusable abstraction for code editing with syntax highlighting.
+
+### Patch Changes
+
+- d316128: Add "None" option to optional Select fields in DynamicForm
+
+  **Bug Fix:**
+
+  - Optional select fields (using `x-options-resolver` or enums) now display a "None" option at the top of the dropdown
+  - Selecting "None" clears the field value, allowing users to unset previously selected values
+  - This fixes the issue where optional fields like `defaultRole` in authentication strategies could not be cleared after selection
+
+- Updated dependencies [83557c7]
+  - @checkstack/common@0.4.0
+  - @checkstack/frontend-api@0.3.1
+
+## 0.2.4
+
+### Patch Changes
+
+- d94121b: Add group-to-role mapping for SAML and LDAP authentication
+
+  **Features:**
+
+  - SAML and LDAP users can now be automatically assigned Checkstack roles based on their directory group memberships
+  - Configure group mappings in the authentication strategy settings with dynamic role dropdowns
+  - Managed role sync: roles configured in mappings are fully synchronized (added when user gains group, removed when user leaves group)
+  - Unmanaged roles (manually assigned, not in any mapping) are preserved during sync
+  - Optional default role for all users from a directory
+
+  **Bug Fix:**
+
+  - Fixed `x-options-resolver` not working for fields inside arrays with `.default([])` in DynamicForm schemas
+
 ## 0.2.3
 
 ### Patch Changes

package/package.json

@@ -1,26 +1,33 @@
 {
   "name": "@checkstack/ui",
-  "version": "0.2.3",
+  "version": "0.3.0",
   "type": "module",
   "main": "src/index.ts",
   "dependencies": {
     "@checkstack/common": "workspace:*",
     "@checkstack/frontend-api": "workspace:*",
+    "@codemirror/autocomplete": "^6.20.0",
+    "@codemirror/lang-json": "^6.0.2",
+    "@codemirror/lang-markdown": "^6.5.0",
+    "@codemirror/lang-xml": "^6.1.0",
+    "@codemirror/lang-yaml": "^6.1.2",
+    "@codemirror/language": "^6.12.1",
+    "@codemirror/state": "^6.5.4",
+    "@codemirror/view": "^6.39.11",
     "@radix-ui/react-accordion": "^1.2.12",
     "@radix-ui/react-dialog": "^1.1.15",
     "@radix-ui/react-select": "^2.2.6",
     "@radix-ui/react-slot": "^1.2.4",
+    "@uiw/react-codemirror": "^4.25.4",
     "ajv": "^8.17.1",
     "ajv-formats": "^3.0.1",
     "class-variance-authority": "^0.7.1",
     "clsx": "^2.1.0",
     "date-fns": "^4.1.0",
     "lucide-react": "0.562.0",
-    "prismjs": "^1.29.0",
     "react": "^18.2.0",
     "react-markdown": "^10.1.0",
     "react-router-dom": "^6.20.0",
-    "react-simple-code-editor": "^0.14.1",
     "recharts": "^3.6.0",
     "tailwind-merge": "^2.2.0"
   },

package/src/components/CodeEditor/CodeEditor.tsx

@@ -0,0 +1,420 @@
+import React from "react";
+import CodeMirror from "@uiw/react-codemirror";
+import {
+  EditorView,
+  ViewPlugin,
+  Decoration,
+  keymap,
+  type ViewUpdate,
+  type DecorationSet,
+} from "@codemirror/view";
+import { RangeSetBuilder, Prec } from "@codemirror/state";
+import {
+  autocompletion,
+  completionStatus,
+  type CompletionContext,
+  type CompletionResult,
+} from "@codemirror/autocomplete";
+import { indentUnit, getIndentUnit, indentString } from "@codemirror/language";
+import {
+  jsonLanguageSupport,
+  yamlLanguageSupport,
+  xmlLanguageSupport,
+  markdownLanguageSupport,
+  isBetweenBrackets,
+  type LanguageSupport,
+} from "./languageSupport";
+
+export type CodeEditorLanguage = "json" | "yaml" | "xml" | "markdown";
+
+/**
+ * A single payload property available for templating
+ */
+export interface TemplateProperty {
+  /** Full path to the property, e.g., "payload.incident.title" */
+  path: string;
+  /** Type of the property, e.g., "string", "number", "boolean" */
+  type: string;
+  /** Optional description of the property */
+  description?: string;
+}
+
+export interface CodeEditorProps {
+  /** Unique identifier for the editor */
+  id?: string;
+  /** Current value of the editor */
+  value: string;
+  /** Callback when the value changes */
+  onChange: (value: string) => void;
+  /** Language for syntax highlighting */
+  language?: CodeEditorLanguage;
+  /** Minimum height of the editor */
+  minHeight?: string;
+  /** Whether the editor is read-only */
+  readOnly?: boolean;
+  /** Placeholder text when empty */
+  placeholder?: string;
+  /**
+   * Optional template properties for autocomplete.
+   * When provided, typing "{{" triggers autocomplete with available template variables.
+   */
+  templateProperties?: TemplateProperty[];
+}
+
+// Language support registry - add new languages here
+const languageRegistry: Record<CodeEditorLanguage, LanguageSupport> = {
+  json: jsonLanguageSupport,
+  yaml: yamlLanguageSupport,
+  xml: xmlLanguageSupport,
+  markdown: markdownLanguageSupport,
+};
+
+/**
+ * Get display type with color info for autocomplete
+ */
+function getTypeInfo(type: string): string {
+  return type.charAt(0).toUpperCase() + type.slice(1).toLowerCase();
+}
+
+/**
+ * Create a ViewPlugin for template-aware syntax highlighting.
+ * Uses the language's buildDecorations function to generate decorations.
+ */
+function createTemplateHighlighter(languageSupport: LanguageSupport) {
+  return ViewPlugin.fromClass(
+    class {
+      decorations: DecorationSet;
+
+      constructor(view: EditorView) {
+        this.decorations = this.buildDecorations(view);
+      }
+
+      buildDecorations(view: EditorView): DecorationSet {
+        const builder = new RangeSetBuilder<Decoration>();
+        const doc = view.state.doc.toString();
+        const ranges = languageSupport.buildDecorations(doc);
+
+        for (const range of ranges) {
+          builder.add(range.from, range.to, range.decoration);
+        }
+
+        return builder.finish();
+      }
+
+      update(update: ViewUpdate) {
+        if (update.docChanged || update.viewportChanged) {
+          this.decorations = this.buildDecorations(update.view);
+        }
+      }
+    },
+    {
+      decorations: (v) => v.decorations,
+    },
+  );
+}
+
+/**
+ * Create a CodeMirror autocomplete extension for template properties.
+ * Triggers when user types "{{" and offers completions from templateProperties.
+ */
+function createTemplateAutocomplete(
+  templateProperties: TemplateProperty[],
+  languageSupport: LanguageSupport,
+): ReturnType<typeof autocompletion> {
+  return autocompletion({
+    override: [
+      (context: CompletionContext): CompletionResult | null => {
+        // Look for {{ pattern before cursor
+        const textBefore = context.state.sliceDoc(0, context.pos);
+        const recentText = context.state.sliceDoc(
+          Math.max(0, context.pos - 50),
+          context.pos,
+        );
+
+        // Find the last {{ that doesn't have a matching }}
+        const lastOpenBrace = recentText.lastIndexOf("{{");
+        const lastCloseBrace = recentText.lastIndexOf("}}");
+
+        if (lastOpenBrace === -1 || lastOpenBrace < lastCloseBrace) {
+          // eslint-disable-next-line unicorn/no-null -- CodeMirror API requires null
+          return null;
+        }
+
+        // Validate position based on text BEFORE the {{ started
+        // We exclude the {{ from validation because it confuses most parsers
+        const positionOfTemplateStart =
+          context.pos - recentText.length + lastOpenBrace;
+        const textBeforeTemplate = textBefore.slice(0, positionOfTemplateStart);
+        if (!languageSupport.isValidTemplatePosition(textBeforeTemplate)) {
+          // eslint-disable-next-line unicorn/no-null -- CodeMirror API requires null
+          return null;
+        }
+
+        // Calculate the position in the document where {{ starts
+        const startOffset = context.pos - recentText.length + lastOpenBrace;
+        const query = recentText.slice(lastOpenBrace + 2).toLowerCase();
+
+        // Check for auto-closed braces after cursor position
+        // When user types {{ with bracket auto-close, it becomes {{}}
+        // We need to consume any trailing }} when completing
+        const textAfter = context.state.sliceDoc(context.pos, context.pos + 4);
+        let endOffset = context.pos;
+        if (textAfter.startsWith("}}")) {
+          endOffset += 2;
+        } else if (textAfter.startsWith("}")) {
+          // Just one } from first auto-close
+          endOffset += 1;
+        }
+
+        // Filter properties based on query
+        const filtered = templateProperties.filter((prop) =>
+          prop.path.toLowerCase().includes(query),
+        );
+
+        if (filtered.length === 0 && query.length > 0) {
+          // eslint-disable-next-line unicorn/no-null -- CodeMirror API requires null
+          return null;
+        }
+
+        return {
+          from: startOffset,
+          to: endOffset,
+          options: filtered.map((prop) => ({
+            label: `{{${prop.path}}}`,
+            displayLabel: prop.path,
+            type: "variable",
+            detail: getTypeInfo(prop.type),
+            info: prop.description,
+            boost: prop.path.toLowerCase().startsWith(query) ? 1 : 0,
+          })),
+          validFor: /^\{\{[\w.]*$/,
+        };
+      },
+    ],
+    activateOnTyping: true,
+    icons: false,
+  });
+}
+
+/**
+ * A code editor component with syntax highlighting and optional template autocomplete.
+ * Wraps @uiw/react-codemirror for consistent styling and API across the platform.
+ */
+export const CodeEditor: React.FC<CodeEditorProps> = ({
+  id,
+  value,
+  onChange,
+  language = "json",
+  minHeight = "100px",
+  readOnly = false,
+  placeholder,
+  templateProperties,
+}) => {
+  const extensions = React.useMemo(() => {
+    const languageSupport = languageRegistry[language];
+    const hasTemplates = templateProperties && templateProperties.length > 0;
+
+    const exts = [
+      EditorView.lineWrapping,
+      EditorView.theme({
+        "&": {
+          fontSize: "14px",
+          fontFamily: "ui-monospace, monospace",
+          backgroundColor: "transparent",
+        },
+        ".cm-scroller": {
+          backgroundColor: "transparent",
+          overflow: "auto",
+        },
+        // minHeight must be on .cm-content and .cm-gutter, not the wrapper (per CM docs)
+        ".cm-content, .cm-gutter": {
+          minHeight: minHeight,
+        },
+        ".cm-content": {
+          padding: "10px",
+        },
+        // Cursor/caret styling
+        ".cm-cursor, .cm-dropCursor": {
+          borderLeftColor: "hsl(var(--foreground))",
+          borderLeftWidth: "2px",
+        },
+        ".cm-line": {
+          color: "hsl(var(--foreground))",
+        },
+        ".cm-gutters": {
+          backgroundColor: "transparent",
+          color: "hsl(var(--muted-foreground))",
+          border: "none",
+        },
+        "&.cm-focused": {
+          outline: "none",
+        },
+        // JSON syntax highlighting for dark/light mode
+        ".cm-string": {
+          color: "hsl(142.1, 76.2%, 36.3%)",
+        },
+        ".cm-number": {
+          color: "hsl(217.2, 91.2%, 59.8%)",
+        },
+        ".cm-propertyName": {
+          color: "hsl(280, 65%, 60%)",
+        },
+        ".cm-keyword": {
+          color: "hsl(280, 65%, 60%)",
+        },
+        ".cm-punctuation": {
+          color: "hsl(var(--muted-foreground))",
+        },
+        // Placeholder styling
+        ".cm-placeholder": {
+          color: "hsl(var(--muted-foreground))",
+        },
+        // JSON syntax highlighting via decorations (overrides Lezer parser)
+        ".cm-json-property": {
+          color: "hsl(280, 65%, 60%)",
+        },
+        ".cm-json-string": {
+          color: "hsl(142.1, 76.2%, 36.3%)",
+        },
+        ".cm-json-number": {
+          color: "hsl(217.2, 91.2%, 59.8%)",
+        },
+        ".cm-json-keyword": {
+          color: "hsl(280, 65%, 60%)",
+        },
+        // Template expression highlighting
+        ".cm-template-expression": {
+          color: "hsl(190, 70%, 50%)",
+          fontWeight: "500",
+        },
+        // Style for autocomplete popup
+        ".cm-tooltip.cm-tooltip-autocomplete": {
+          backgroundColor: "hsl(var(--popover))",
+          border: "1px solid hsl(var(--border))",
+          borderRadius: "0.375rem",
+          boxShadow:
+            "0 10px 15px -3px rgba(0, 0, 0, 0.1), 0 4px 6px -2px rgba(0, 0, 0, 0.05)",
+        },
+        ".cm-tooltip-autocomplete ul": {
+          fontFamily: "ui-monospace, monospace",
+          fontSize: "0.75rem",
+        },
+        ".cm-tooltip-autocomplete ul li": {
+          padding: "0.25rem 0.5rem",
+        },
+        ".cm-tooltip-autocomplete ul li[aria-selected]": {
+          backgroundColor: "hsl(var(--accent))",
+          color: "hsl(var(--accent-foreground))",
+        },
+        ".cm-completionLabel": {
+          fontFamily: "ui-monospace, monospace",
+        },
+        ".cm-completionDetail": {
+          marginLeft: "0.5rem",
+          fontStyle: "normal",
+          color: "hsl(var(--muted-foreground))",
+        },
+      }),
+    ];
+
+    // Always add language extension for features (indentation, bracket matching, etc.)
+    if (languageSupport) {
+      exts.push(languageSupport.extension);
+
+      // Create custom Enter key handler that applies our indentation
+      // This is needed because the language parsers may get confused by templates
+      // or may not provide the indentation we want
+      const customEnterKeymap = keymap.of([
+        {
+          key: "Enter",
+          run: (view) => {
+            const state = view.state;
+
+            // If autocomplete is active, let it handle Enter for selection
+            if (completionStatus(state) === "active") {
+              return false;
+            }
+
+            const pos = state.selection.main.head;
+            const textBefore = state.sliceDoc(0, pos);
+            const textAfter = state.sliceDoc(pos);
+            const unit = getIndentUnit(state);
+            const indent = languageSupport.calculateIndentation(
+              textBefore + "\n",
+              unit,
+            );
+            const indentStr = indentString(state, indent);
+
+            // Check if we're between matching brackets/tags
+            // This handles cases like: {|}, [|], <tag>|</tag>
+            if (isBetweenBrackets(textBefore, textAfter)) {
+              // Split: add newline with indent, then newline with previous indent for closing
+              const prevIndent = Math.max(0, indent - unit);
+              const prevIndentStr = indentString(state, prevIndent);
+
+              view.dispatch({
+                changes: {
+                  from: pos,
+                  to: pos,
+                  insert: "\n" + indentStr + "\n" + prevIndentStr,
+                },
+                selection: { anchor: pos + 1 + indentStr.length },
+                scrollIntoView: true,
+                userEvent: "input",
+              });
+            } else {
+              // Normal: insert newline with calculated indentation
+              view.dispatch({
+                changes: { from: pos, to: pos, insert: "\n" + indentStr },
+                selection: { anchor: pos + 1 + indentStr.length },
+                scrollIntoView: true,
+                userEvent: "input",
+              });
+            }
+            return true;
+          },
+        },
+      ]);
+
+      // Always add indentation support and custom highlighter for consistent behavior
+      // Prec.highest ensures our colors take precedence over language parser output
+      exts.push(
+        indentUnit.of("  "), // Configure 2-space indentation
+        Prec.highest(customEnterKeymap), // Override default Enter behavior
+        Prec.highest(createTemplateHighlighter(languageSupport)), // Consistent syntax colors
+      );
+
+      // Add template autocomplete if properties provided
+      if (hasTemplates) {
+        exts.push(
+          createTemplateAutocomplete(templateProperties, languageSupport),
+        );
+      }
+    }
+
+    return exts;
+  }, [language, templateProperties, minHeight]);
+
+  return (
+    <div
+      id={id}
+      className="w-full rounded-md border border-input bg-background font-mono text-sm focus-within:ring-2 focus-within:ring-ring focus-within:border-transparent transition-all box-border"
+    >
+      <CodeMirror
+        value={value}
+        onChange={onChange}
+        extensions={extensions}
+        editable={!readOnly}
+        placeholder={placeholder}
+        basicSetup={{
+          lineNumbers: true,
+          foldGutter: false,
+          highlightActiveLine: false,
+          highlightSelectionMatches: false,
+          autocompletion: false, // We use our own
+        }}
+        theme="none"
+      />
+    </div>
+  );
+};

package/src/components/CodeEditor/index.ts

@@ -0,0 +1,10 @@
+export {
+  CodeEditor,
+  type CodeEditorProps,
+  type CodeEditorLanguage,
+  type TemplateProperty,
+} from "./CodeEditor";
+
+// Re-export language support for testing and extensibility
+export { isValidJsonTemplatePosition } from "./languageSupport";
+export type { LanguageSupport, DecorationRange } from "./languageSupport";

package/src/components/CodeEditor/languageSupport/enterBehavior.test.ts

@@ -0,0 +1,173 @@
+import { describe, expect, it } from "bun:test";
+
+import { isBetweenBrackets } from "./enterBehavior";
+
+describe("isBetweenBrackets", () => {
+  // ============================================================================
+  // Curly braces
+  // ============================================================================
+  describe("curly braces", () => {
+    it("returns true for empty object {|}", () => {
+      expect(isBetweenBrackets("{", "}")).toBe(true);
+    });
+
+    it("returns false when content before closing brace", () => {
+      // Last char is "1", not "{"
+      expect(isBetweenBrackets('{"a": 1', "}")).toBe(false);
+    });
+
+    it("returns false when only opening brace before", () => {
+      expect(isBetweenBrackets("{", "x")).toBe(false);
+    });
+
+    it("returns false when only closing brace after", () => {
+      expect(isBetweenBrackets("x", "}")).toBe(false);
+    });
+
+    it("returns false for nested opening {|{", () => {
+      expect(isBetweenBrackets("{", "{")).toBe(false);
+    });
+  });
+
+  // ============================================================================
+  // Square brackets
+  // ============================================================================
+  describe("square brackets", () => {
+    it("returns true for empty array [|]", () => {
+      expect(isBetweenBrackets("[", "]")).toBe(true);
+    });
+
+    it("returns false when content before closing bracket", () => {
+      // Last char is "2", not "["
+      expect(isBetweenBrackets("[1, 2", "]")).toBe(false);
+    });
+
+    it("returns false when only opening bracket before", () => {
+      expect(isBetweenBrackets("[", "x")).toBe(false);
+    });
+
+    it("returns false when only closing bracket after", () => {
+      expect(isBetweenBrackets("x", "]")).toBe(false);
+    });
+
+    it("returns false for nested opening [|[", () => {
+      expect(isBetweenBrackets("[", "[")).toBe(false);
+    });
+  });
+
+  // ============================================================================
+  // XML/HTML tags
+  // ============================================================================
+  describe("XML/HTML tags", () => {
+    it("returns true for empty tag <div>|</div>", () => {
+      expect(isBetweenBrackets("<div>", "</div>")).toBe(true);
+    });
+
+    it('returns true for tag with attributes <div class="x">|</div>', () => {
+      expect(isBetweenBrackets('<div class="x">', "</div>")).toBe(true);
+    });
+
+    it("returns true for nested tags <outer><inner>|</inner>", () => {
+      expect(isBetweenBrackets("<outer><inner>", "</inner>")).toBe(true);
+    });
+
+    it("returns true for self-named closing tag <a>|</a>", () => {
+      expect(isBetweenBrackets("<a>", "</a>")).toBe(true);
+    });
+
+    it("returns false for adjacent opening tags <a>|<b>", () => {
+      expect(isBetweenBrackets("<a>", "<b>")).toBe(false);
+    });
+
+    it("returns false for text after tag <div>|text", () => {
+      expect(isBetweenBrackets("<div>", "text")).toBe(false);
+    });
+
+    it("returns false for self-closing tag <br/>|", () => {
+      expect(isBetweenBrackets("<br/>", "")).toBe(false);
+    });
+
+    it("returns false for comment after tag <div>|<!--", () => {
+      expect(isBetweenBrackets("<div>", "<!--")).toBe(false);
+    });
+  });
+
+  // ============================================================================
+  // Edge cases
+  // ============================================================================
+  describe("edge cases", () => {
+    it("returns false for empty strings", () => {
+      expect(isBetweenBrackets("", "")).toBe(false);
+    });
+
+    it("returns false for whitespace only", () => {
+      expect(isBetweenBrackets("  ", "  ")).toBe(false);
+    });
+
+    it("returns false for newlines", () => {
+      expect(isBetweenBrackets("{\n", "\n}")).toBe(false);
+    });
+
+    it("returns true for template braces {{|}}", () => {
+      // { followed by } - pattern matches even though it's part of template
+      expect(isBetweenBrackets("{{", "}}")).toBe(true);
+    });
+
+    it("handles single character inputs", () => {
+      expect(isBetweenBrackets("a", "b")).toBe(false);
+    });
+  });
+
+  // ============================================================================
+  // Real-world scenarios
+  // ============================================================================
+  describe("real-world scenarios", () => {
+    it("JSON: empty object literal", () => {
+      expect(isBetweenBrackets('{"name": ', "}")).toBe(false); // Not at the boundary
+    });
+
+    it("JSON: cursor right after opening brace", () => {
+      expect(isBetweenBrackets("{", '  "name": "value"}')).toBe(false); // Has content
+    });
+
+    it("XML: HTML doctype before tag", () => {
+      expect(isBetweenBrackets("<!DOCTYPE html><html>", "</html>")).toBe(true);
+    });
+
+    it("XML: after text content in tag", () => {
+      expect(isBetweenBrackets("<p>Hello", "</p>")).toBe(false); // "o" != ">"
+    });
+
+    it("YAML: no brackets involved", () => {
+      expect(isBetweenBrackets("key:", "")).toBe(false);
+    });
+  });
+
+  // ============================================================================
+  // Regression tests
+  // ============================================================================
+  describe("regression: autocomplete interaction", () => {
+    /**
+     * REGRESSION TEST (documented, requires DOM testing)
+     *
+     * Issue: When autocomplete popup is showing, pressing Enter should select
+     * the completion item, NOT insert a newline.
+     *
+     * Fix: In CodeEditor.tsx, the custom Enter keymap checks `completionStatus(state)`
+     * and returns `false` when autocomplete is "active", allowing the autocomplete
+     * extension to handle the Enter key.
+     *
+     * This cannot be unit tested without a full CodeMirror DOM integration.
+     * Manual verification steps:
+     * 1. Type "{{" in the editor with template properties configured
+     * 2. Autocomplete popup should appear
+     * 3. Press Enter to select a template
+     * 4. Template should be inserted (NOT a newline)
+     */
+    it("documents the autocomplete Enter key requirement", () => {
+      // This test exists purely as documentation
+      // The actual fix is in CodeEditor.tsx: completionStatus(state) check
+      expect(true).toBe(true);
+    });
+  });
+});