Remove dead policies code (#7783)

# Description of Changes
In #7681, we redefined what Policies were and merged the Policies page
into the Pipelines page. This left over a lot of now dead/redundant
Policies code which should be removed because the Pipelines code already
covers it all. This PR does not remove _all_ of the Policies code
because some of it is still being used in the UI now, but as in-editor
pipelines and as pipeline templates. That code needs to be renamed to
get it to the correct naming, but this PR is big enough as it is to just
delete all the dead code, so that'll come in another PR.

Author: jbrunton96

frontend/editor/public/locales/en-US/translation.toml

@@ -6996,45 +6996,6 @@ dismiss = "Dismiss overlay"
 enforcingTitle = "Enforcing policy..."
 viewAnyway = "View file (policy still enforcing)"
 
-[policyOption]
-autoRedactPhi = "Auto-redact PHI"
-blockExport = "Block export"
-ccpa = "CCPA"
-contracts = "Contracts"
-documents = "Documents"
-fedramp = "FedRAMP"
-financialReports = "Financial reports"
-flagForReview = "Flag for review"
-gdpr = "GDPR"
-hipaa = "HIPAA"
-hold = "Hold"
-hrRecords = "HR records"
-indefinite = "Indefinite"
-insurance = "Insurance"
-invoices = "Invoices"
-iso27001 = "ISO 27001"
-legalFilings = "Legal filings"
-medicalPhi = "Medical / PHI"
-never = "Never"
-ninetyDays = "90 days"
-oneYear = "1 year"
-p60 = "60%"
-p70 = "70%"
-p80 = "80%"
-p90 = "90%"
-p95 = "95%"
-pciDss = "PCI DSS"
-quarantineDocument = "Quarantine document"
-routeToBucket = "Route to bucket"
-s3Bucket = "S3 bucket"
-sevenYears = "7 years"
-sharePoint = "SharePoint"
-soc2 = "SOC 2"
-taxDocuments = "Tax documents"
-thirtyDays = "30 days"
-threeYears = "3 years"
-webhook = "Webhook"
-
 [portal]
 comingSoon = "Coming soon"
 
@@ -8663,7 +8624,6 @@ defaultName = "{{category}} Pipeline"
 
 [portal.policies.card]
 comingSoon = "Upgrade to Enterprise"
-notSetUp = "Not set up"
 requiresAiEngine = "Requires AI engine"
 
 [portal.policies.categories.classification]
@@ -8791,7 +8751,6 @@ onEveryUpload = "On every upload"
 outputAsNewFile = "as a new file"
 outputAsNewVersion = "as a new version"
 recentActivity = "Recent activity"
-retry = "Retry"
 showLess = "Show less"
 showMore = "Show more"
 sources = "Sources"
@@ -8802,7 +8761,6 @@ delete = "Delete"
 editSettings = "Edit settings"
 pause = "Pause"
 resume = "Resume"
-runNow = "Run now"
 
 [portal.policies.detail.clearHistory]
 body = "This pipeline will forget every file it has already processed and reprocess everything currently in its sources on the next run. The files themselves are not changed. This cannot be undone."
@@ -9027,29 +8985,6 @@ docsEnforced = "Docs processed"
 active = "Active"
 paused = "Paused"
 
-[portal.policies.summary.active]
-description = "Enforcing on upload/export"
-label = "Active policies"
-
-[portal.policies.summary.categories]
-description = "Available to configure"
-label = "Categories"
-
-[portal.policies.summary.docsEnforced]
-description = "Across active policies"
-label = "Docs enforced"
-
-[portal.policies.summary.paused]
-description = "Configured but not firing"
-label = "Paused"
-
-[portal.policies.table]
-appliesTo = "Applies to"
-docs = "Docs enforced"
-enforces = "Enforces"
-policy = "Policy"
-status = "Status"
-
 [portal.policies.wizard.actions]
 cancel = "Cancel"
 customise = "Customise"
@@ -9103,9 +9038,6 @@ setUp = "Set up {{category}} pipeline"
 [portal.policies.wizard.workflow]
 description = "Choose what this pipeline does to every document it processes."
 
-[portal.policySummary.action]
-setUp = "Set up"
-
 [portal.processorFlow]
 footnote = "Counts are over the last 24 hours. Flow speed is illustrative."
 liveBadge = "Live"

frontend/editor/src/core/i18n/translationAudit.ts

@@ -109,12 +109,6 @@ export const I18N_PROJECTS: TranslationProject[] = [
       // "portal.policies.operations" - the shape heuristic treats that interpolation as one
       // segment, so this whole catalogue-driven family is matched here instead.
       /^portal\.policies\.operations\./,
-      // Policy field labels + option display copy are looked up with keys
-      // derived from catalogue data (t(`policies.field.${key}`),
-      // t(`policyOption.${id}`)) in the PolicyFieldRows and setup wizards —
-      // invisible to the static scan. The raw catalogue value is the fallback.
-      /^policies\.field\./,
-      /^policyOption\./,
       // A failure's disabled reason arrives from the server as a key and is rendered with
       // t(thatKey), so nothing in source names it, but the copy still has to exist.
       /^portal\.failures\.disabled\./,

frontend/editor/src/core/ui/Modal.tsx

@@ -1,7 +1,8 @@
-import { useEffect, useId, type ReactNode } from "react";
+import { useEffect, useId, useRef, type ReactNode } from "react";
 import { createPortal } from "react-dom";
 import { FocusTrap } from "@mantine/core";
 import { Button } from "@app/ui/Button";
+import { useIsOverflowing } from "@app/hooks/useIsOverflowing";
 import "@app/ui/Modal.css";
 
 export type ModalWidth = "sm" | "md" | "lg" | "xl";
@@ -43,6 +44,11 @@ export function Modal({
   children,
 }: ModalProps) {
   const titleId = useId();
+  const bodyRef = useRef<HTMLDivElement>(null);
+  // A body that overflows must be reachable by keyboard to scroll; only its non-focusable
+  // content (header close + footer live outside it). tabindex lands only when it scrolls, so
+  // fitting modals gain no stray tab stop (axe scrollable-region-focusable).
+  const bodyScrolls = useIsOverflowing(bodyRef);
 
   useEffect(() => {
     if (!open || disableEscapeClose) return;
@@ -152,7 +158,13 @@ export function Modal({
               />
             </header>
           )}
-          <div className="sui-modal__body">{children}</div>
+          <div
+            ref={bodyRef}
+            className="sui-modal__body"
+            tabIndex={bodyScrolls ? 0 : undefined}
+          >
+            {children}
+          </div>
           {footer && <footer className="sui-modal__footer">{footer}</footer>}
         </div>
       </FocusTrap>

frontend/editor/src/portal/api/pipelines.ts

@@ -3,6 +3,11 @@ import {
   type SupportingFileBindings,
   type ToolApiStep,
 } from "@app/hooks/tools/shared/toolAutomation";
+import type {
+  PolicyRunView,
+  PolicyRunStatus,
+  RunOutputFile,
+} from "@app/policies/types";
 
 /**
  * Pipelines service layer: the backend contract.
@@ -132,37 +137,10 @@ export interface TriggerInfo {
   supportedSourceTypes: string[];
 }
 
-export type PolicyRunStatus =
-  | "PENDING"
-  | "RUNNING"
-  | "WAITING_FOR_INPUT"
-  | "COMPLETED"
-  | "FAILED"
-  | "CANCELLED";
-
-/** One file a run produced, downloadable via /api/v1/general/files/{fileId}. */
-export interface RunOutputFile {
-  fileId: string;
-  fileName: string | null;
-}
-
-/** A run's current state. Mirrors the backend `PolicyRunView`. */
-export interface PolicyRunView {
-  runId: string;
-  policyId: string | null;
-  status: PolicyRunStatus;
-  currentStep: number;
-  stepCount: number;
-  /** Human-readable failure message; set when status is FAILED. */
-  error: string | null;
-  errorCode: string | null;
-  /**
-   * Files the run produced, present once it completes. Whole-run, not per step: the backend keeps
-   * one flat list, so nothing here can be attributed to an individual step.
-   */
-  outputs?: RunOutputFile[] | null;
-  createdAt: number;
-}
+// One run view for the whole app: the builder test-run poll and the catalogue runs list read the
+// same backend PolicyRunView, so the type is defined once in the codec (imported above) and
+// re-exported here for callers that reach it through the pipelines API.
+export type { PolicyRunView, PolicyRunStatus, RunOutputFile };
 
 /** GET /api/v1/policies/overview: KPI strip + one row per policy for the admin. */
 export async function fetchPipelines(): Promise<PipelinesOverviewResponse> {

frontend/editor/src/portal/api/policies.ts

@@ -29,15 +29,7 @@ import type {
   WirePolicy,
 } from "@app/policies/types";
 
-export type {
-  PolicyActivityItem,
-  PolicyDecodedState,
-  PolicyRunView,
-  PolicyStats,
-  WireOutputOptions,
-  WireOutputSpec,
-  WirePolicy,
-} from "@app/policies/types";
+export type { PolicyRunView, WirePolicy } from "@app/policies/types";
 
 // Re-export the wire step type under the legacy name components depend on.
 export type { WirePipelineStep as PipelineStep } from "@app/policies/types";
@@ -46,9 +38,7 @@ export type { WirePipelineStep as PipelineStep } from "@app/policies/types";
 /*  Catalogue model — portal-specific                                        */
 /* ──────────────────────────────────────────────────────────────────────── */
 
-export type PolicyStatus = "active" | "paused";
-
-export type PolicyRowStatus = "active" | "paused" | "setup";
+type PolicyStatus = "active" | "paused";
 
 export type PolicyFieldType = "toggle" | "select" | "chips" | "text";
 
@@ -130,15 +120,7 @@ export interface DecoratedPolicy {
   activity: import("@app/policies/types").PolicyActivityItem[];
 }
 
-export interface PoliciesSummary {
-  active: number;
-  paused: number;
-  categories: number;
-  docsEnforced: number;
-}
-
 export interface PoliciesResponse {
-  summary: PoliciesSummary;
   catalogue: CatalogueEntry[];
 }
 
@@ -156,7 +138,7 @@ export interface CatalogueEntry {
  * i18n keys keyed by endpoint; labels stored steps in the detail view. Mostly
  * {@link ToolEndpoint}s, plus the AI classify endpoint, which isn't part of the generated union.
  */
-export const ENDPOINT_LABELS: Partial<
+const ENDPOINT_LABELS: Partial<
   Record<ToolEndpoint | "/api/v1/ai/tools/classify-and-label", string>
 > = {
   "/api/v1/security/auto-redact": "portal.policies.endpoints.autoRedact",
@@ -423,17 +405,6 @@ export const POLICY_CONFIG: Record<string, PolicyConfigDef> = {
   },
 };
 
-export const POLICY_DOC_TYPES: string[] = [
-  "contracts",
-  "invoices",
-  "taxDocuments",
-  "hrRecords",
-  "insurance",
-  "medicalPhi",
-  "legalFilings",
-  "financialReports",
-];
-
 // ── Client-side catalogue assembly ───────────────────────────────────────────
 
 function decoratePolicy(
@@ -517,25 +488,7 @@ export function assemblePolicies(
     return { category, config: POLICY_CONFIG[category.id], policy };
   });
 
-  const active = wirePolicies.filter((p) => p.enabled).length;
-  const paused = wirePolicies.filter((p) => !p.enabled).length;
-  const enabledPolicyIds = new Set(
-    wirePolicies.filter((p) => p.enabled).map((p) => p.id),
-  );
-  const docsEnforced = runs.filter(
-    (r) =>
-      r.status === "COMPLETED" &&
-      r.policyId != null &&
-      enabledPolicyIds.has(r.policyId),
-  ).length;
-  const summary: PoliciesSummary = {
-    active,
-    paused,
-    categories: POLICY_CATEGORIES.length,
-    docsEnforced,
-  };
-
-  return { summary, catalogue };
+  return { catalogue };
 }
 
 /**
@@ -613,13 +566,6 @@ export function parseSimplePolicy(
   };
 }
 
-/** GET /api/v1/policies/{id} — one stored policy's raw record. */
-export async function fetchPolicy(id: string): Promise<WirePolicy> {
-  return apiClient.local.json<WirePolicy>(
-    `/api/v1/policies/${encodeURIComponent(id)}`,
-  );
-}
-
 /**
  * POST /api/v1/policies — create (blank id) or update (matched id). The
  * backend stamps owner + teamId server-side and returns the stored record.