@checkstack/gitops-frontend 0.2.0 → 0.2.1

package/CHANGELOG.md

@@ -1,5 +1,30 @@
 # @checkstack/gitops-frontend
 
+## 0.2.1
+
+### Patch Changes
+
+- 86bab6a: ### GitOps: Fix authentication token handling
+
+  - Made `authToken` optional in `ReconcileProviderParams` and `ScraperOptions` to support unauthenticated access to public repositories
+  - GitHub and GitLab scrapers now conditionally set authentication headers only when a token is provided
+  - Sync worker now decrypts the encrypted `authToken` from the database before passing it to scrapers, fixing authentication failures caused by sending encrypted values in HTTP headers
+
+  ### SLO: Fix premature Nines Club achievement unlock
+
+  - The "Nines Club" achievement now requires both ≥99.99% availability **and** a 365-day compliance streak, preventing immediate unlock on newly created SLOs with 100% default availability
+
+  ### SLO: Align frontend achievement descriptions with backend criteria
+
+  - Fixed mismatched descriptions for Iron Uptime (7-day, not 30), Diamond Uptime (30-day, not 90), Clean Sheet (rolling window, not quarter), Full Coverage (3+ SLOs, not all systems in group), and Nines Club (99.99%)
+
+  ### SLO: Enrich milestones with system names
+
+  - The `getRecentMilestones` endpoint now resolves human-readable system names via the Catalog API instead of returning raw system IDs
+
+- Updated dependencies [86bab6a]
+  - @checkstack/gitops-common@0.1.1
+
 ## 0.2.0
 
 ### Minor Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@checkstack/gitops-frontend",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "type": "module",
   "main": "src/index.tsx",
   "checkstack": {
@@ -14,8 +14,8 @@
   "dependencies": {
     "@checkstack/common": "0.6.5",
     "@checkstack/frontend-api": "0.3.9",
-    "@checkstack/gitops-common": "0.0.1",
-    "@checkstack/ui": "1.3.5",
+    "@checkstack/gitops-common": "0.1.0",
+    "@checkstack/ui": "1.3.6",
     "lucide-react": "^0.344.0",
     "react": "^18.2.0",
     "react-router-dom": "^6.22.0"

package/src/components/ProviderEditor.tsx

@@ -24,7 +24,7 @@ interface ProviderEditorProps {
     target: string;
     pathPattern: string;
     baseUrl?: string;
-    authToken?: string;
+    authToken?: string | null;
     syncInterval?: number;
     deletionPolicy?: "orphan" | "auto";
   }) => void;
@@ -50,6 +50,7 @@ export const ProviderEditor: React.FC<ProviderEditorProps> = ({
   const [pathPattern, setPathPattern] = useState(initialData?.pathPattern ?? ".checkstack/**/*.yaml");
   const [baseUrl, setBaseUrl] = useState(initialData?.baseUrl ?? "");
   const [authToken, setAuthToken] = useState("");
+  const [clearToken, setClearToken] = useState(false);
   const [syncInterval, setSyncInterval] = useState(String(initialData?.syncInterval ?? 300));
   const [deletionPolicy, setDeletionPolicy] = useState<"orphan" | "auto">(
     initialData?.deletionPolicy ?? "orphan",
@@ -62,6 +63,7 @@ export const ProviderEditor: React.FC<ProviderEditorProps> = ({
       setPathPattern(initialData?.pathPattern ?? ".checkstack/**/*.yaml");
       setBaseUrl(initialData?.baseUrl ?? "");
       setAuthToken("");
+      setClearToken(false);
       setSyncInterval(String(initialData?.syncInterval ?? 300));
       setDeletionPolicy(initialData?.deletionPolicy ?? "orphan");
     }
@@ -71,12 +73,24 @@ export const ProviderEditor: React.FC<ProviderEditorProps> = ({
     e.preventDefault();
     if (!target.trim() || !pathPattern.trim()) return;
 
+    // Determine authToken value:
+    // - clearToken=true → null (explicitly remove)
+    // - non-empty string → new token
+    // - empty string → undefined (keep current)
+    let authTokenValue: string | null | undefined;
+    if (clearToken) {
+      // eslint-disable-next-line unicorn/no-null
+      authTokenValue = null;
+    } else if (authToken.trim()) {
+      authTokenValue = authToken.trim();
+    }
+
     onSave({
       type,
       target: target.trim(),
       pathPattern: pathPattern.trim(),
       baseUrl: baseUrl.trim() || undefined,
-      authToken: authToken.trim() || undefined,
+      authToken: authTokenValue,
       syncInterval: Number(syncInterval) || 300,
       deletionPolicy,
     });
@@ -160,13 +174,45 @@ export const ProviderEditor: React.FC<ProviderEditorProps> = ({
               <Label htmlFor="provider-auth-token">
                 Auth Token {initialData ? "(leave empty to keep current)" : "(optional)"}
               </Label>
-              <Input
-                id="provider-auth-token"
-                type="password"
-                placeholder="ghp_xxxx..."
-                value={authToken}
-                onChange={(e) => setAuthToken(e.target.value)}
-              />
+              {clearToken ? (
+                <div className="flex items-center gap-2 p-2 rounded-md border border-border bg-muted/50">
+                  <span className="text-sm text-muted-foreground flex-1">
+                    Token will be removed on save
+                  </span>
+                  <Button
+                    type="button"
+                    variant="outline"
+                    size="sm"
+                    onClick={() => setClearToken(false)}
+                  >
+                    Undo
+                  </Button>
+                </div>
+              ) : (
+                <div className="flex gap-2">
+                  <Input
+                    id="provider-auth-token"
+                    type="password"
+                    placeholder="ghp_xxxx..."
+                    value={authToken}
+                    onChange={(e) => setAuthToken(e.target.value)}
+                    className="flex-1"
+                  />
+                  {initialData && (
+                    <Button
+                      type="button"
+                      variant="outline"
+                      size="sm"
+                      onClick={() => {
+                        setAuthToken("");
+                        setClearToken(true);
+                      }}
+                    >
+                      Remove
+                    </Button>
+                  )}
+                </div>
+              )}
             </div>
 
             <div className="grid grid-cols-2 gap-4">

package/src/components/ProviderList.tsx

@@ -119,7 +119,7 @@ export const ProviderList = () => {
     target: string;
     pathPattern: string;
     baseUrl?: string;
-    authToken?: string;
+    authToken?: string | null;
     syncInterval?: number;
     deletionPolicy?: "orphan" | "auto";
   }) => {
@@ -136,7 +136,11 @@ export const ProviderList = () => {
         },
       });
     } else {
-      createMutation.mutate(data);
+      // On create, authToken is always string|undefined (Remove button not shown)
+      createMutation.mutate({
+        ...data,
+        authToken: data.authToken ?? undefined,
+      });
     }
   };