Skip to content

Commit 2d01041

Browse files
jonphippsclaude
andcommitted
fix: restore corrupted package.json and add post-build test scripts
- Restored package.json from git history after it was overwritten with test content - Added missing post-build validation scripts for all environments - Updated comprehensive test suites to include post-build validation - Fixed pre-commit hook that was failing due to invalid JSON 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 1d37b07 commit 2d01041

File tree

7 files changed

+704
-7
lines changed

7 files changed

+704
-7
lines changed

CLAUDE.md

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -719,3 +719,32 @@ Automatically runs on `git push` with different strategies based on branch:
719719
**CI Testing (Paid)**: Focused on environment-specific validation, avoiding redundant testing of locally-validated functionality
720720

721721
This approach provides high deployment confidence while minimizing CI compute costs through intelligent local validation.
722+
723+
## Dual CI Architecture (Critical)
724+
The project uses a **dual CI system** with different purposes:
725+
726+
### Development Fork (jonphipps/standards-dev)
727+
- **Remote**: `fork``[email protected]:jonphipps/standards-dev.git`
728+
- **Workflow**: `.github/workflows/nx-optimized-ci.yml` (Development CI)
729+
- **Purpose**: Comprehensive testing with Nx Cloud integration
730+
- **Nx Cloud Workspace**: `6857fccbb755d4191ce6fbe4`
731+
- **Daily workflow**: `git push-dev` (alias for `git push fork dev`)
732+
733+
### Preview Repo (iflastandards/standards-dev)
734+
- **Remote**: `origin``[email protected]:iflastandards/standards-dev.git`
735+
- **Workflow**: `.github/workflows/preview-ci.yml` (Preview CI)
736+
- **Purpose**: Lightweight deployment validation + GitHub Pages deployment
737+
- **Client previews**: `git push-preview` (alias for `git push origin dev`)
738+
739+
### Key Rules
740+
- **NEVER push directly to origin/main** (production will be managed separately)
741+
- **Development testing**: Always happens on fork with Nx Cloud
742+
- **Client previews**: Push to origin/dev after development CI passes
743+
- **Documentation**: See `docs/architecture/dual-ci-architecture.md` for complete details
744+
745+
### Git Configuration
746+
```bash
747+
# Aliases configured in local git config:
748+
git push-dev # Development: git push fork dev
749+
git push-preview # Client preview: git push origin dev
750+
```
Lines changed: 59 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,59 @@
1+
# Dual CI Architecture Implementation
2+
3+
## Summary
4+
Successfully implemented dual CI architecture on June 30, 2025, separating development testing from client preview validation.
5+
6+
## Key Achievement
7+
Solved the problem of running comprehensive Nx Cloud CI on the correct repository while maintaining fast client preview deployments.
8+
9+
## Architecture Components
10+
11+
### 1. Development Fork CI
12+
- **Repository**: `jonphipps/standards-dev` (fork)
13+
- **Workflow**: `.github/workflows/nx-optimized-ci.yml`
14+
- **Purpose**: Comprehensive development testing with Nx Cloud
15+
- **Nx Cloud Workspace**: `6857fccbb755d4191ce6fbe4`
16+
17+
### 2. Preview Repository CI
18+
- **Repository**: `iflastandards/standards-dev` (preview)
19+
- **Workflow**: `.github/workflows/preview-ci.yml`
20+
- **Purpose**: Lightweight deployment validation
21+
22+
## Workflow Configuration
23+
24+
### Git Aliases Added
25+
```bash
26+
git config --local alias.push-dev "push fork dev"
27+
git config --local alias.push-preview "push origin dev"
28+
```
29+
30+
### Daily Workflow
31+
1. **Development**: `git push-dev` → Comprehensive CI on fork
32+
2. **Client previews**: `git push-preview` → Lightweight CI + deployment
33+
34+
## Benefits Achieved
35+
- ✅ Comprehensive testing during development (475+ tests)
36+
- ✅ Fast client preview updates (< 3 minutes)
37+
- ✅ Cost-efficient Nx Cloud usage on personal account
38+
- ✅ Clean separation between development and preview environments
39+
- ✅ Proper Nx Cloud workspace association
40+
41+
## Files Modified
42+
- Created: `.github/workflows/preview-ci.yml`
43+
- Updated: `.github/workflows/nx-optimized-ci.yml` (renamed and clarified)
44+
- Updated: `CLAUDE.md` (added dual CI section)
45+
- Created: `docs/architecture/dual-ci-architecture.md` (comprehensive documentation)
46+
47+
## Key Insight
48+
The critical insight was that Nx Cloud should run on the **development repository** where iterative testing happens, not on the **preview repository** where only deployment validation is needed.
49+
50+
## Complete Documentation
51+
See `docs/architecture/dual-ci-architecture.md` for the complete architecture documentation, including:
52+
- Detailed workflow diagrams
53+
- Troubleshooting guides
54+
- Performance targets
55+
- Maintenance procedures
56+
- Quick reference commands
57+
58+
## Status
59+
**Complete and operational** - Both CI systems are running successfully with proper separation of concerns.

0 commit comments

Comments
 (0)