docs: Add mandatory pre-completion checklist to AGENTS.md

Added a "Required Pre-Completion Checklist" section that instructs agents to:
- Always run format, check, lint, test, and build before finishing
- Fix any failures before marking work complete
- Never skip these mandatory quality gates

This addresses the requirement to ensure all code changes are properly validated.

Co-authored-by: stnguyen90 <1477010+stnguyen90@users.noreply.github.com>

Author: Copilot

.github/workflows/dockerize-profiles.yml

@@ -1,49 +1,49 @@
 name: Dockerize Profiles
 
 on:
-  push:
-    branches: [feat-profiles]
-  pull_request:
-    types: [opened, synchronize, reopened]
-    branches: [feat-profiles]
-  workflow_dispatch:
+    push:
+        branches: [feat-profiles]
+    pull_request:
+        types: [opened, synchronize, reopened]
+        branches: [feat-profiles]
+    workflow_dispatch:
 
 jobs:
-  dockerize-profiles:
-    runs-on: ubuntu-latest
-    
-    steps:
-      - name: Checkout the repo
-        uses: actions/checkout@v2
-      - name: Set up QEMU
-        uses: docker/setup-qemu-action@v2
-      - name: Set up Docker Buildx
-        uses: docker/setup-buildx-action@v2
-      - name: Log in to Docker Hub
-        uses: docker/login-action@v3
-        with:
-            username: ${{ vars.DOCKERHUB_USERNAME }}
-            password: ${{ secrets.DOCKERHUB_TOKEN }}
+    dockerize-profiles:
+        runs-on: ubuntu-latest
 
-      - name: Extract metadata (tags, labels) for Docker
-        id: meta
-        uses: docker/metadata-action@v5
-        with:
-            images: appwrite/console-profiles
-            tags: |
-                type=ref,event=branch,prefix=branch-
-                type=ref,event=pr
-                type=sha,prefix=sha-
-                type=raw,value=gh-${{ github.run_id}}
-            flavor: |
-              latest=false
+        steps:
+            - name: Checkout the repo
+              uses: actions/checkout@v2
+            - name: Set up QEMU
+              uses: docker/setup-qemu-action@v2
+            - name: Set up Docker Buildx
+              uses: docker/setup-buildx-action@v2
+            - name: Log in to Docker Hub
+              uses: docker/login-action@v3
+              with:
+                  username: ${{ vars.DOCKERHUB_USERNAME }}
+                  password: ${{ secrets.DOCKERHUB_TOKEN }}
 
-      - name: Build and push Docker image
-        id: push
-        uses: docker/build-push-action@v6
-        with:
-            context: .
-            push: true
-            platforms: linux/amd64,linux/arm64
-            tags: ${{ steps.meta.outputs.tags }}
-            labels: ${{ steps.meta.outputs.labels }}
+            - name: Extract metadata (tags, labels) for Docker
+              id: meta
+              uses: docker/metadata-action@v5
+              with:
+                  images: appwrite/console-profiles
+                  tags: |
+                      type=ref,event=branch,prefix=branch-
+                      type=ref,event=pr
+                      type=sha,prefix=sha-
+                      type=raw,value=gh-${{ github.run_id}}
+                  flavor: |
+                      latest=false
+
+            - name: Build and push Docker image
+              id: push
+              uses: docker/build-push-action@v6
+              with:
+                  context: .
+                  push: true
+                  platforms: linux/amd64,linux/arm64
+                  tags: ${{ steps.meta.outputs.tags }}
+                  labels: ${{ steps.meta.outputs.labels }}

AGENTS.md

@@ -101,4 +101,23 @@ src/
 5. Before commit: `pnpm run check && pnpm run format && pnpm run lint && pnpm run test && pnpm run build`
 6. **Take screenshots**: For any UI changes, capture screenshots and include them in the PR description or comments before finalizing
 
+## Required Pre-Completion Checklist
+
+**CRITICAL**: Before finishing any work or marking a task complete, agents MUST run the following commands in order and ensure all pass:
+
+1. **`pnpm run format`** - Auto-fix all formatting issues
+2. **`pnpm run check`** - Verify TypeScript/Svelte types (must show 0 errors, 0 warnings)
+3. **`pnpm run lint`** - Check code style (ignore pre-existing issues in files you didn't modify)
+4. **`pnpm run test`** - Run all unit tests (all tests must pass)
+5. **`pnpm run build`** - Ensure production build succeeds
+
+If any command fails:
+
+- **Format/Lint**: Run `pnpm run format` to auto-fix, then re-check
+- **Type errors**: Fix all TypeScript errors in files you modified
+- **Test failures**: Fix failing tests or ensure failures are unrelated to your changes
+- **Build failures**: Debug and resolve build issues before proceeding
+
+**Never skip these checks** - they are mandatory quality gates before any work is considered complete.
+
 **Trust these instructions** - only search if incomplete/incorrect. See CONTRIBUTING.md for PR conventions. Use `--frozen-lockfile` always. Docker builds: multi-stage, final image is nginx serving static files from `/console` path.