Feature/dap 93 clean up (#172)

New Code Structure

```
domain/                        # unchanged — pure types & logic
lib/                           # unchanged
components/                    # generic UI: ui/, error-boundary, + move loading-page here

features/
  degree-audit-app/            # the SPA mounted by entrypoints/degree-audit
    providers/                 #   audit-provider, audit-mutations, section-groups,
                               #   preferences-provider, course-modal-provider
    audit-view/                #   degree-audit-page, requirement-breakdown, graph,
                               #   audit-card, gpa-credit-cards
    planner-view/              #   degree-planner-page, semester-card, semester-dropdowns
    course-search/             #   course-add-modal, CourseSearchPanel, course-card
    shell/                     #   navbar, sidebar, degree-completion-donut (shared by both views)
  catalog/                     # DATA ONLY: catalog-db, mappers, department-map,
                               #   seed-catalog, scraping/  (no React, no audit imports)
  audit-scraping/              # unchanged
  popup/                       # unchanged
  misc/                      # unchanged (used to be banner)
```
- Added catalog scripts for refreshing, validating, and generating department mappings.
- Added and expanded tests for scraping, audit providers, storage, mutations, and catalog refreshes.
- Improved audit scraping behavior and handling of audit data without cached results.
- Updated planner drag-and-drop handling.
- Updated build, linting, formatting, and CI configuration.
- Removed obsolete backend modules, old documentation, unused assets, and outdated example files.

Overall this PR was created to ensure a more maintainable codebase.

Author: spotta85

.github/workflows/ci.yml

@@ -4,13 +4,25 @@ name: CI
 on: [push, pull_request]
 
 jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout repository
+        uses: actions/checkout@v7
+      - name: Install bun
+        uses: oven-sh/setup-bun@v2
+      - name: Install dependencies
+        run: bun install
+      - name: Run tests
+        run: bun test
+
   check:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v7
       - name: Install bun
-        uses: oven-sh/setup-bun@v1
+        uses: oven-sh/setup-bun@v2
       - name: Install dependencies
         run: bun install
       - name: Check types
@@ -19,9 +31,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v7
       - name: Install bun
-        uses: oven-sh/setup-bun@v1
+        uses: oven-sh/setup-bun@v2
       - name: Install dependencies
         run: bun install
       - name: Lint
@@ -31,9 +43,9 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v7
       - name: Install bun
-        uses: oven-sh/setup-bun@v1
+        uses: oven-sh/setup-bun@v2
       - name: Install dependencies
         run: bun install
       - name: Format
@@ -43,17 +55,17 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout repository
-        uses: actions/checkout@v3
+        uses: actions/checkout@v7
       - name: Install bun
-        uses: oven-sh/setup-bun@v1
+        uses: oven-sh/setup-bun@v2
       - name: Build extension
         run: |
           bun install
           bun run build
           cd .output/chrome-mv3
           zip -r ../extension.zip .
       - name: Upload artifact
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v7
         with:
           name: extension
           path: .output/extension.zip

.gitignore

@@ -14,6 +14,9 @@ stats-*.json
 .wxt
 web-ext.config.ts
 claude.md
+
+# Bun is the package manager; keep npm's lockfile out
+package-lock.json
 # Editor directories and files
 .vscode/*
 !.vscode/extensions.json
@@ -27,3 +30,5 @@ claude.md
 .claude
 *.code-workspace
 .claude
+
+docs/

.husky/pre-commit

@@ -0,0 +1,8 @@
+node_modules
+.output
+.wxt
+bun.lock
+package-lock.json
+
+# Test fixtures are raw HTML captures; do not reformat
+tests/fixtures/**/*.html

.prettierignore

@@ -0,0 +1,73 @@
+import { Component, type ErrorInfo, type ReactNode } from "react";
+
+interface ErrorBoundaryProps {
+  children: ReactNode;
+  fallback?: ReactNode;
+}
+
+interface ErrorBoundaryState {
+  hasError: boolean;
+  error: Error | null;
+}
+
+/**
+ * Single root-level error boundary. Catches render errors (e.g. getCourseById
+ * throwing on a missing course id) so a bad reference doesn't white-screen the
+ * whole degree-audit page.
+ *
+ * Place one instance at the page root; don't scatter throughout the tree.
+ */
+export default class ErrorBoundary extends Component<
+  ErrorBoundaryProps,
+  ErrorBoundaryState
+> {
+  state: ErrorBoundaryState = { hasError: false, error: null };
+
+  static getDerivedStateFromError(error: Error): ErrorBoundaryState {
+    return { hasError: true, error };
+  }
+
+  componentDidCatch(error: Error, info: ErrorInfo) {
+    console.error(
+      "[Degree Audit Plus] Render error caught by ErrorBoundary:",
+      error,
+      info,
+    );
+  }
+
+  handleReset = () => {
+    this.setState({ hasError: false, error: null });
+  };
+
+  render() {
+    if (this.state.hasError) {
+      if (this.props.fallback) return this.props.fallback;
+
+      return (
+        <div className="flex min-h-screen w-full flex-col items-center justify-center gap-4 bg-background px-6 text-center text-text">
+          <h1 className="text-2xl font-bold text-dap-primary">
+            Something went wrong
+          </h1>
+          <p className="max-w-md text-sm text-dap-gray-light">
+            An unexpected error occurred while rendering your degree audit.
+            Re-run your audit from the popup to refresh the data, or click the
+            button below to try again.
+          </p>
+          {this.state.error && (
+            <pre className="max-w-lg rounded bg-gray-100 px-4 py-2 text-left text-xs text-red-600 dark:bg-gray-800">
+              {this.state.error.message}
+            </pre>
+          )}
+          <button
+            onClick={this.handleReset}
+            className="rounded-md bg-dap-primary px-4 py-2 text-sm font-semibold text-white hover:opacity-90"
+          >
+            Try again
+          </button>
+        </div>
+      );
+    }
+
+    return this.props.children;
+  }
+}

bun.lock

@@ -1,4 +1,4 @@
-import { VStack } from "@/entrypoints/components/common/helperdivs";
+import { VStack } from "@/components/ui/stack";
 import { SpinnerIcon } from "@phosphor-icons/react";
 
 const LoadingPage = () => {

components/error-boundary.tsx

@@ -0,0 +1,76 @@
+import { forwardRef } from "react";
+import { cn } from "~/lib/utils";
+
+type ButtonProps = {
+  color?: "orange" | "white" | "black";
+  fill?: "solid" | "outline" | "none";
+  size?: "icon" | "small" | "med" | "large";
+} & React.ButtonHTMLAttributes<HTMLButtonElement>;
+
+const styleMap: Record<
+  NonNullable<ButtonProps["color"]>,
+  Record<NonNullable<ButtonProps["fill"]>, string>
+> = {
+  orange: {
+    solid: "bg-dap-primary text-white border-transparent",
+    outline: "bg-transparent text-dap-primary border-dap-primary",
+    none: "bg-transparent text-dap-primary border-transparent",
+  },
+  white: {
+    solid: "bg-background text-text border-transparent",
+    outline: "bg-transparent text-white border-white",
+    none: "bg-transparent text-white border-transparent",
+  },
+  black: {
+    solid: "bg-text text-background border-transparent",
+    outline: "bg-transparent text-text border-text",
+    none: "bg-transparent text-text border-transparent",
+  },
+};
+
+const sizeMap: Record<NonNullable<ButtonProps["size"]>, string> = {
+  icon: "w-8 h-8 p-0",
+  small: "px-2 py-1 text-sm",
+  med: "px-4 py-2 text-base",
+  large: "px-6 py-3 text-lg",
+};
+const Button = forwardRef<HTMLButtonElement, ButtonProps>((props, ref) => {
+  const {
+    color = "orange",
+    fill = "solid",
+    size = null,
+    className,
+    ...rest
+  } = props;
+
+  return (
+    <button
+      ref={ref}
+      className={cn(
+        "px-4 py-2 rounded-lg whitespace-nowrap w-fit flex flex-row items-center justify-center gap-2 border hover:opacity-80 cursor-pointer transition-all duration-300",
+        styleMap[color][fill],
+        size && sizeMap[size],
+        className,
+      )}
+      {...rest}
+    >
+      {rest.children}
+    </button>
+  );
+});
+Button.displayName = "Button";
+
+export const IconButton = forwardRef<
+  HTMLButtonElement,
+  Omit<ButtonProps, "children"> & { icon: React.ReactNode; label?: string }
+>((props, ref) => {
+  return (
+    <Button ref={ref} {...props}>
+      {props.icon}
+      {props.label && <span className="text-sm">{props.label}</span>}
+    </Button>
+  );
+});
+IconButton.displayName = "IconButton";
+
+export default Button;

…degree-audit/components/loading-page.tsx components/loading-page.tsxentrypoints/degree-audit/components/loading-page.tsx renamed to components/loading-page.tsx entrypoints/degree-audit/components/loading-page.tsx renamed to components/loading-page.tsx

@@ -9,7 +9,7 @@ import React, {
 } from "react";
 import { cn } from "~/lib/utils";
 import Container from "./container";
-import { HStack, VStack } from "./helperdivs";
+import { HStack, VStack } from "./stack";
 
 // Context to share dropdown state with sub-components
 type DropdownContextType = {
@@ -130,8 +130,4 @@ const Dropdown = forwardRef<
 
 Dropdown.displayName = "Dropdown";
 
-// Attach sub-components to main component
-(Dropdown as any).Header = DropdownHeader;
-(Dropdown as any).Content = DropdownContent;
-
 export default Dropdown;