Student/calliejones init

.github/workflows/deploy-staging.yml

@@ -3,6 +3,18 @@ name: Deploy Staging Branch
 on:
   push:
     branches: [ staging ]
+    paths-ignore:
+      # Ignore changes that don't affect the site
+      - 'src/assignments/**'
+      - 'scripts/**'
+      - 'tests/**'
+      - 'private/**'
+      - '**.md'
+      - '.github/**'
+      - 'data/**'
+      - 'pyproject.toml'
+      - 'pytest.ini'
+      - 'uv.lock'
 
 jobs:
   deploy:

.github/workflows/deploy.yml

@@ -3,6 +3,18 @@ name: Deploy to Production
 on:
   push:
     branches: [ main ]
+    paths-ignore:
+      # Ignore changes that don't affect the site
+      - 'src/assignments/**'
+      - 'scripts/**'
+      - 'tests/**'
+      - 'private/**'
+      - '**.md'
+      - '.github/**'
+      - 'data/**'
+      - 'pyproject.toml'
+      - 'pytest.ini'
+      - 'uv.lock'
 
 jobs:
   deploy:

.github/workflows/shadow-tester.yml

@@ -0,0 +1,139 @@
+name: Shadow Tester
+
+on:
+  repository_dispatch:
+    types: [run-shadow-tests]
+
+permissions:
+  contents: read
+  pull-requests: write  # Needed to comment on PRs
+
+jobs:
+  shadow-test:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout Private Repo
+        uses: actions/checkout@v4
+        with:
+          repository: crheckman/private-vla-foundations
+          token: ${{ secrets.PRIVATE_REPO_TOKEN }}  # PAT with access to private repo
+
+      - name: Set up Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+
+      - name: Install Python Dependencies
+        run: |
+          pip install pytest torch numpy
+
+      - name: Fetch Student Code from Public PR
+        env:
+          PR_NUMBER: ${{ github.event.client_payload.pr_number }}
+          HEAD_BRANCH: ${{ github.event.client_payload.head_branch }}
+          HEAD_SHA: ${{ github.event.client_payload.head_sha }}
+          REPO_URL: ${{ github.event.client_payload.repo_url }}
+        run: |
+          echo "Fetching student code from PR #${PR_NUMBER}"
+
+          # Clone the public repo
+          git clone https://github.com/arpg/vla-foundations.git /tmp/public-repo
+          cd /tmp/public-repo
+
+          # Fetch the PR branch
+          git fetch origin pull/${PR_NUMBER}/head:pr-${PR_NUMBER}
+          git checkout pr-${PR_NUMBER}
+
+          # Copy student code to our testing directory
+          # Copy src/assignments to the current repo
+          if [ -d "src/assignments" ]; then
+            cp -r src/assignments/* $GITHUB_WORKSPACE/src/assignments/ || true
+          fi
+
+          echo "Student code fetched successfully"
+
+      - name: Run Internal Rigorous Tests
+        id: tests
+        continue-on-error: true
+        run: |
+          # Run pytest with internal tests
+          pytest tests/internal/ -v --tb=short --maxfail=5 > test_output.txt 2>&1
+          TEST_EXIT_CODE=$?
+
+          # Capture output
+          cat test_output.txt
+
+          # Save exit code for later
+          echo "exit_code=${TEST_EXIT_CODE}" >> $GITHUB_OUTPUT
+
+          # Exit with the actual test result
+          exit $TEST_EXIT_CODE
+
+      - name: Prepare Test Summary
+        if: always()
+        id: summary
+        run: |
+          if [ -f test_output.txt ]; then
+            # Extract summary from pytest output
+            SUMMARY=$(tail -20 test_output.txt | grep -E "(PASSED|FAILED|ERROR)" || echo "Test execution completed")
+
+            # Escape newlines for GitHub output
+            SUMMARY="${SUMMARY//$'\n'/'%0A'}"
+            echo "summary=${SUMMARY}" >> $GITHUB_OUTPUT
+          else
+            echo "summary=No test output available" >> $GITHUB_OUTPUT
+          fi
+
+      - name: Comment on Public PR - Pass
+        if: steps.tests.outcome == 'success'
+        uses: peter-evans/create-or-update-comment@v3
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}  # Default token works for same repo
+          repository: arpg/vla-foundations
+          issue-number: ${{ github.event.client_payload.pr_number }}
+          body: |
+            ## ✅ Shadow CI: Internal Tests Passed
+
+            Your submission passed all internal rigorous tests!
+
+            <details>
+            <summary>Test Summary</summary>
+
+            ```
+            ${{ steps.summary.outputs.summary }}
+            ```
+
+            </details>
+
+            ---
+            *These are hidden internal tests run by the instructor. Your code meets the required standards.*
+
+      - name: Comment on Public PR - Fail
+        if: steps.tests.outcome == 'failure'
+        uses: peter-evans/create-or-update-comment@v3
+        with:
+          token: ${{ secrets.GITHUB_TOKEN }}  # Default token works for same repo
+          repository: arpg/vla-foundations
+          issue-number: ${{ github.event.client_payload.pr_number }}
+          body: |
+            ## ❌ Shadow CI: Internal Tests Failed
+
+            Your submission did not pass all internal tests. Please review the feedback and make necessary corrections.
+
+            <details>
+            <summary>Test Summary</summary>
+
+            ```
+            ${{ steps.summary.outputs.summary }}
+            ```
+
+            </details>
+
+            ### Next Steps:
+            1. Review the test failures above
+            2. Make corrections to your code
+            3. Push updates to your PR branch
+            4. Tests will automatically re-run
+
+            ---
+            *These are hidden internal tests run by the instructor. Contact @crheckman if you need clarification on the failures.*

.github/workflows/vla-audit.yml

@@ -42,7 +42,15 @@ jobs:
 
             ### Common Issues:
 
-            **1. Semantic Line Breaks**
+            **1. Required Frontmatter Fields**
+            - Every audit MDX file must include these fields:
+              - `title`: Paper title
+              - `author`: Paper author(s)
+              - `topic`: Research topic/category
+              - `paper`: Link to paper or citation
+            - All fields must have non-empty values (no placeholders like "TBD" or "TODO")
+
+            **2. Semantic Line Breaks**
             - Each sentence should be on its own line
             - This makes PR commenting and reviewing much easier
             - Example:
@@ -53,7 +61,7 @@ jobs:
               + This makes PR review much easier.
               ```
 
-            **2. Clean Git History**
+            **3. Clean Git History**
             - No "Merge branch 'main'" commits allowed
             - Use `git rebase main` instead of `git merge main`
             - Keep your commit history linear and clean
@@ -144,22 +152,3 @@ jobs:
 
             ---
             *This preview will be removed when the PR is closed.*
-
-  trigger-shadow-tests:
-    runs-on: ubuntu-latest
-    if: github.event_name == 'pull_request' && github.base_ref == 'staging'
-    needs: audit
-    steps:
-      - name: Trigger Shadow CI in Private Repo
-        uses: peter-evans/repository-dispatch@v2
-        with:
-          token: ${{ secrets.PRIVATE_DISPATCH_TOKEN }}
-          repository: crheckman/private-vla-foundations
-          event-type: run-shadow-tests
-          client-payload: |
-            {
-              "pr_number": "${{ github.event.pull_request.number }}",
-              "head_branch": "${{ github.event.pull_request.head.ref }}",
-              "head_sha": "${{ github.event.pull_request.head.sha }}",
-              "repo_url": "${{ github.event.pull_request.head.repo.clone_url }}"
-            }

.gitignore

@@ -53,15 +53,13 @@ __pycache__/
 
 # project-specific
 /arxiv-digest/
-# Private repo files - do not commit to public
-private/
-tests/
-scripts/_sanitize_todos.py
-scripts/manage_solutions.py
-scripts/sanitize.sh
-scripts/setup_private_repo.sh
-scripts/add_github_secret.sh
-pytest.ini
-PRIVATE_REPO_SETUP.md
-SETUP_WITH_GH_CLI.md
-QUICK_REFERENCE.md
+
+# Private solution infrastructure (NEVER commit to public branches)
+*.backup.py
+
+# Claude Code skill outputs (generated reports)
+.claude/releases/*.md
+.claude/sync-reports/*.md
+!.claude/releases/.gitkeep
+!.claude/sync-reports/.gitkeep
+!tests/internal/reports/.gitkeep

README.md

@@ -83,103 +83,6 @@ git push --force-with-lease
 
 ---
 
-## Repository Structure
-
-```
-vla-foundations/
-├── app/                           # Next.js App Router (web framework)
-│   ├── page.tsx                   # Landing page
-│   ├── textbook/[slug]/           # Dynamic chapter pages
-│   ├── course/                    # Course overview page
-│   │   └── assignments/[slug]/    # Dynamic assignment pages
-│   └── contributors/[slug]/       # Dynamic contributor profile pages
-│
-├── content/                       # All MDX content (rendered as web pages)
-│   ├── textbook/                  # 8-chapter VLA textbook
-│   │   ├── foundations/           # Chapter 0: Core concepts
-│   │   ├── architectures/         # Chapter 1: Model designs
-│   │   ├── data/                  # Chapter 2: Dataset construction
-│   │   ├── training/              # Chapter 3: Optimization methods
-│   │   ├── evaluation/            # Chapter 4: Metrics and benchmarks
-│   │   ├── deployment/            # Chapter 5: Production systems
-│   │   ├── applications/          # Chapter 6: Real-world use cases
-│   │   └── future/                # Chapter 7: Open problems
-│   │
-│   ├── course/                    # Course materials
-│   │   ├── Syllabus.mdx           # Course syllabus
-│   │   ├── assignments/           # Assignment specifications
-│   │   └── submissions/           # Student submission reports
-│   │
-│   └── contributors/              # Contributor profiles
-│       └── [github-handle].mdx    # One profile per contributor
-│
-└── src/                           # Executable source code
-    └── assignments/               # Assignment code templates
-        └── scratch-1/             # Example: Transformer implementation
-            ├── README.md          # Minimal README
-            ├── backbone.py        # Implementation template with TODOs
-            └── generate_data.py   # Dataset generator script
-```
-
----
-
-## The 8-Chapter Textbook
-
-0. **Foundations** - Core concepts and problem formulation
-1. **Architectures** - Model designs and network topologies
-2. **Data** - Dataset construction and curation strategies
-3. **Training** - Optimization and fine-tuning methods
-4. **Evaluation** - Metrics and benchmarking protocols
-5. **Deployment** - Production systems and scaling
-6. **Applications** - Real-world use cases and case studies
-7. **Future Directions** - Open problems and research frontiers
-
----
-
-## Development Workflow
-
-### Initial Setup
-
-```bash
-# Clone the repository
-git clone https://github.com/arpg/vla-foundations.git
-cd vla-foundations
-
-# Install dependencies
-pnpm install
-
-# Run development server
-pnpm dev
-```
-
-Navigate to `http://localhost:3000` to see the site.
-
-### Local Build
-
-```bash
-# Build the static site
-pnpm build
-
-# Preview the production build
-pnpm start
-```
-
----
-
-## Technologies
-
-### Core
-- **Next.js 16**: Static site generation
-- **TypeScript**: Type safety
-- **Tailwind CSS**: Styling
-- **MDX**: Markdown with JSX
-
-### Content Processing
-- **remark-math** + **rehype-katex**: LaTeX rendering
-- **remark-gfm**: GitHub-flavored Markdown
-
----
-
 ## Resources
 
 ### Documentation

app/textbook/audits/[...slug]/page.tsx

@@ -89,14 +89,22 @@ export default async function AuditPage({ params }: PageProps) {
   // Get chapters for sidebar
   const chapters = getAllChapters();
 
+  // Determine if we're in review mode based on environment
+  const isReviewMode = isStaging && process.env.STAGING_PR_NUMBER !== undefined;
+  const prNumber = process.env.STAGING_PR_NUMBER;
+
   return (
-    <AuditLayout chapters={chapters}>
+    <AuditLayout
+      chapters={chapters}
+      isReviewMode={isReviewMode}
+      prNumber={prNumber}
+    >
       <Link href="/textbook/audits" className="text-sm text-blue-600 hover:text-blue-800 mb-8 inline-block">
         ← Back to Audits
       </Link>
 
-      {/* Staging Banner */}
-      {isStaging && (
+      {/* Staging Banner (shown when in staging but not in review mode) */}
+      {isStaging && !isReviewMode && (
         <div className="bg-yellow-50 border-l-4 border-yellow-400 text-yellow-800 px-6 py-4 mb-8 rounded">
           <p className="font-bold text-lg">⚠️ DRAFT AUDIT - UNDER REVIEW</p>
           <p className="mt-2 text-sm">