rollback releaseSelections renaming

Author: cryptodev-2s

src/ui/App.tsx

@@ -13,7 +13,7 @@ const setsAreEqual = (a: Set<string>, b: Set<string>) => {
 };
 
 type SubmitButtonProps = {
-  releaseSelections: Record<string, string>;
+  selections: Record<string, string>;
   packageDependencyErrors: Record<
     string,
     { missingDependentNames: string[]; missingDependencies: string[] }
@@ -22,16 +22,14 @@ type SubmitButtonProps = {
 };
 
 function SubmitButton({
-  releaseSelections,
+  selections,
   packageDependencyErrors,
   onSubmit,
 }: SubmitButtonProps) {
   const isDisabled =
-    Object.keys(releaseSelections).length === 0 ||
+    Object.keys(selections).length === 0 ||
     Object.keys(packageDependencyErrors).length > 0 ||
-    Object.values(releaseSelections).every(
-      (value) => value === 'intentionally-skip',
-    );
+    Object.values(selections).every((value) => value === 'intentionally-skip');
 
   return (
     <button
@@ -50,9 +48,7 @@ function SubmitButton({
 
 function App() {
   const [packages, setPackages] = useState<Package[]>([]);
-  const [releaseSelections, setReleaseSelections] = useState<
-    Record<string, string>
-  >({});
+  const [selections, setSelections] = useState<Record<string, string>>({});
   const [isSubmitting, setIsSubmitting] = useState(false);
   const [error, setError] = useState<string | null>(null);
   const [changelogs, setChangelogs] = useState<Record<string, string>>({});
@@ -80,7 +76,7 @@ function App() {
   const previousPackages = useRef<Set<string>>(new Set());
 
   useEffect(() => {
-    const majorBumps = Object.entries(releaseSelections)
+    const majorBumps = Object.entries(selections)
       .filter(([_, type]) => type === 'major')
       .map(([pkgName]) => pkgName);
 
@@ -94,9 +90,9 @@ function App() {
       .then((data: Package[]) => {
         const newPackageNames = new Set(data.map((pkg) => pkg.name));
 
-        // Only clean up releaseSelections if the package list actually changed
+        // Only clean up selections if the package list actually changed
         if (!setsAreEqual(previousPackages.current, newPackageNames)) {
-          setReleaseSelections((prev) =>
+          setSelections((prev) =>
             Object.fromEntries(
               Object.entries(prev).filter(([pkgName]) =>
                 newPackageNames.has(pkgName),
@@ -115,7 +111,7 @@ function App() {
         setError(err.message);
         console.error('Error fetching packages:', err);
       });
-  }, [releaseSelections]);
+  }, [selections]);
 
   const checkDependencies = async (selectionData: Record<string, string>) => {
     if (Object.keys(selectionData).length === 0) return;
@@ -148,11 +144,11 @@ function App() {
 
   useEffect(() => {
     const timeoutId = setTimeout(() => {
-      void checkDependencies(releaseSelections);
+      void checkDependencies(selections);
     }, 500);
 
     return () => clearTimeout(timeoutId);
-  }, [releaseSelections]);
+  }, [selections]);
 
   const handleCustomVersionChange = (packageName: string, version: string) => {
     try {
@@ -182,7 +178,7 @@ function App() {
         return rest;
       });
 
-      setReleaseSelections((prev) => ({
+      setSelections((prev) => ({
         ...prev,
         [packageName]: version,
       }));
@@ -199,14 +195,14 @@ function App() {
     value: ReleaseType | '',
   ): void => {
     if (value === '') {
-      const { [packageName]: _, ...rest } = releaseSelections;
-      setReleaseSelections(rest);
+      const { [packageName]: _, ...rest } = selections;
+      setSelections(rest);
 
       const { [packageName]: __, ...remainingErrors } = packageDependencyErrors;
       setPackageDependencyErrors(remainingErrors);
     } else {
-      setReleaseSelections({
-        ...releaseSelections,
+      setSelections({
+        ...selections,
         [packageName]: value,
       });
     }
@@ -218,7 +214,7 @@ function App() {
       const response = await fetch('/api/release', {
         method: 'POST',
         headers: { 'Content-Type': 'application/json' },
-        body: JSON.stringify(releaseSelections),
+        body: JSON.stringify(selections),
       });
 
       const data: {
@@ -254,8 +250,8 @@ function App() {
       const errorMessage =
         err instanceof Error ? err.message : 'An error occurred';
       setError(errorMessage);
-      console.error('Error submitting releaseSelections:', err);
-      alert('Failed to submit releaseSelections. Please try again.');
+      console.error('Error submitting selections:', err);
+      alert('Failed to submit selections. Please try again.');
     } finally {
       setIsSubmitting(false);
     }
@@ -280,11 +276,11 @@ function App() {
   };
 
   const handleBulkAction = (action: ReleaseType) => {
-    const newReleaseSelections = { ...releaseSelections };
+    const newReleaseSelections = { ...selections };
     selectedPackages.forEach((packageName) => {
       newReleaseSelections[packageName] = action;
     });
-    setReleaseSelections(newReleaseSelections);
+    setSelections(newReleaseSelections);
     setSelectedPackages(new Set());
     setShowCheckboxes(true);
   };
@@ -393,7 +389,7 @@ function App() {
           <PackageItem
             key={pkg.name}
             pkg={pkg}
-            releaseSelections={releaseSelections}
+            selections={selections}
             versionErrors={versionErrors}
             packageDependencyErrors={packageDependencyErrors}
             loadingChangelogs={loadingChangelogs}
@@ -403,7 +399,7 @@ function App() {
             onSelectionChange={handleSelectionChange}
             onCustomVersionChange={handleCustomVersionChange}
             onFetchChangelog={fetchChangelog}
-            setReleaseSelections={setReleaseSelections}
+            setSelections={setSelections}
             setChangelogs={setChangelogs}
             onToggleSelect={() => togglePackageSelection(pkg.name)}
           />
@@ -412,7 +408,7 @@ function App() {
 
       {packages.length > 0 && (
         <SubmitButton
-          releaseSelections={releaseSelections}
+          selections={selections}
           packageDependencyErrors={packageDependencyErrors}
           onSubmit={handleSubmit}
         />

src/ui/PackageItem.tsx

@@ -6,7 +6,7 @@ import { Package, ReleaseType } from './types.js';
 
 type PackageItemProps = {
   pkg: Package;
-  releaseSelections: Record<string, string>;
+  selections: Record<string, string>;
   versionErrors: Record<string, string>;
   packageDependencyErrors: Record<
     string,
@@ -22,16 +22,14 @@ type PackageItemProps = {
   onSelectionChange: (packageName: string, value: ReleaseType | '') => void;
   onCustomVersionChange: (packageName: string, version: string) => void;
   onFetchChangelog: (packageName: string) => Promise<void>;
-  setReleaseSelections: React.Dispatch<
-    React.SetStateAction<Record<string, string>>
-  >;
+  setSelections: React.Dispatch<React.SetStateAction<Record<string, string>>>;
   setChangelogs: React.Dispatch<React.SetStateAction<Record<string, string>>>;
   onToggleSelect: () => void;
 };
 
 export function PackageItem({
   pkg,
-  releaseSelections,
+  selections,
   versionErrors,
   packageDependencyErrors,
   loadingChangelogs,
@@ -41,7 +39,7 @@ export function PackageItem({
   onSelectionChange,
   onCustomVersionChange,
   onFetchChangelog,
-  setReleaseSelections,
+  setSelections,
   setChangelogs,
   onToggleSelect,
 }: PackageItemProps) {
@@ -50,8 +48,7 @@ export function PackageItem({
       key={pkg.name}
       id={`package-${pkg.name}`}
       className={`border p-4 rounded-lg ${
-        releaseSelections[pkg.name] &&
-        releaseSelections[pkg.name] !== 'intentionally-skip'
+        selections[pkg.name] && selections[pkg.name] !== 'intentionally-skip'
           ? 'border-2'
           : 'border-gray-200'
       } ${
@@ -79,19 +76,17 @@ export function PackageItem({
           <div className="flex items-center justify-between">
             <div>
               <p className="text-gray-600">Current version: {pkg.version}</p>
-              {releaseSelections[pkg.name] &&
-                releaseSelections[pkg.name] !== 'intentionally-skip' &&
-                releaseSelections[pkg.name] !== 'custom' &&
+              {selections[pkg.name] &&
+                selections[pkg.name] !== 'intentionally-skip' &&
+                selections[pkg.name] !== 'custom' &&
                 !versionErrors[pkg.name] && (
                   <p className="text-yellow-700">
                     New version:{' '}
-                    {!['patch', 'minor', 'major'].includes(
-                      releaseSelections[pkg.name],
-                    )
-                      ? releaseSelections[pkg.name]
+                    {!['patch', 'minor', 'major'].includes(selections[pkg.name])
+                      ? selections[pkg.name]
                       : new SemVer(pkg.version)
                           .inc(
-                            releaseSelections[pkg.name] as Exclude<
+                            selections[pkg.name] as Exclude<
                               ReleaseType,
                               'intentionally-skip' | 'custom' | string
                             >,
@@ -107,7 +102,7 @@ export function PackageItem({
             </div>
             <VersionSelector
               packageName={pkg.name}
-              selection={releaseSelections[pkg.name]}
+              selection={selections[pkg.name]}
               onSelectionChange={onSelectionChange}
               onCustomVersionChange={onCustomVersionChange}
               onFetchChangelog={onFetchChangelog}
@@ -125,7 +120,7 @@ export function PackageItem({
               <DependencyErrorSection
                 title="Missing Dependencies"
                 items={packageDependencyErrors[pkg.name].missingDependencies}
-                setSelections={setReleaseSelections}
+                setSelections={setSelections}
                 description={`The following packages are dependencies or peer dependencies of ${pkg.name}. Because they may have introduced new changes that ${pkg.name} is now using, you need to verify whether to include them in the release.
 
 To do this, look at the change history for each package and compare it with the change history for ${pkg.name}. If ${pkg.name} uses any new changes from a package, then you need to include it by bumping its version. If you have confirmed that the changes to a package do not affect ${pkg.name}, you may omit it from the release by choosing "Skip" instead.`}
@@ -139,7 +134,7 @@ To do this, look at the change history for each package and compare it with the
                   items={
                     packageDependencyErrors[pkg.name].missingDependentNames
                   }
-                  setSelections={setReleaseSelections}
+                  setSelections={setSelections}
                   description={`Because ${pkg.name} is being released with a new major version, to prevent peer dependency warnings in consuming projects, all of the following packages which list ${pkg.name} as a peer dependency need to be included in the release. Please choose new versions for these packages. If for some reason you feel it is safe to omit a package you may choose "Skip".`}
                 />
               </div>