cleanup

Author: ibolton336

shared/src/types/index.ts

@@ -94,6 +94,6 @@ export interface LocalChange {
 export interface ResolutionMessage {
   type: string;
   solution: GetSolutionResult;
-  violation: any;
-  incident: any;
+  violation: Violation;
+  incident: Incident;
 }

webview-ui/src/components/IncidentList.tsx

@@ -20,11 +20,11 @@ interface IncidentListProps {
   onSelectIncident: (incident: Incident) => void;
 }
 
-export function IncidentList({
+const IncidentList: React.FC<IncidentListProps> = ({
   incidents,
   selectedIncident,
   onSelectIncident,
-}: IncidentListProps) {
+}: IncidentListProps) => {
   const getSeverityColor = (severity: string = "low") => {
     switch (severity.toLowerCase()) {
       case "high":
@@ -93,4 +93,5 @@ export function IncidentList({
       ))}
     </Stack>
   );
-}
+};
+export default IncidentList;

webview-ui/src/components/ResolutionsPage.tsx

@@ -1,4 +1,3 @@
-import "./ResolutionsPage.css";
 import React, { useEffect, useState } from "react";
 import {
   Page,
@@ -18,10 +17,10 @@ import {
 import { vscode } from "../utils/vscode";
 import { Change, ResolutionMessage } from "@shared/types";
 import { CheckCircleIcon, WarningTriangleIcon } from "@patternfly/react-icons";
-import { IncidentList } from "./IncidentList";
-import { CodePreview } from "./CodePreview";
-import { FileChanges } from "./FileChanges";
 import spacing from "@patternfly/react-styles/css/utilities/Spacing/spacing";
+import IncidentList from "./IncidentList";
+import { FileChanges } from "./FileChanges";
+import { CodePreview } from "./CodePreview";
 
 const ResolutionPage: React.FC = () => {
   const [resolution, setResolution] = useState<ResolutionMessage | null>(null);
@@ -141,15 +140,11 @@ const ResolutionPage: React.FC = () => {
               <CardTitle>
                 {resolution?.violation.description || "No Violation Data"}
                 <Badge className={spacing.mSm}>
-                  {/* {mockViolation.category} */}
                   {resolution?.violation.category}
                 </Badge>
               </CardTitle>
               <CardBody>
                 <IncidentList
-                  // incidents={mockViolation.incidents}
-                  // selectedIncident={selectedIncident}
-                  // onSelectIncident={setSelectedIncident}
                   incidents={resolution?.violation.incidents}
                   selectedIncident={resolution?.incident}
                   onSelectIncident={(incident) => {}}