Fix/platform polish and ci cd - #83
Conversation
|
Too many files changed for review (152 files, 100 file limit). Bypass the limit by tagging |
|
| GitGuardian id | GitGuardian status | Secret | Commit | Filename | |
|---|---|---|---|---|---|
| 27968088 | Triggered | Generic Password | c3e5c18 | docker-compose.yml | View secret |
🛠 Guidelines to remediate hardcoded secrets
- Understand the implications of revoking this secret by investigating where it is used in your code.
- Replace and store your secret safely. Learn here the best practices.
- Revoke and rotate this secret.
- If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.
To avoid such incidents in the future consider
- following these best practices for managing and storing secrets including API keys and other credentials
- install secret detection on pre-commit to catch secret before it leaves your machine and ease remediation.
🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (6)
📒 Files selected for processing (89)
📝 WalkthroughWalkthroughThis pull request adds a federated Community Hub, refactors Docker image publishing and runtime images, updates deployment configuration, refreshes shared branding, expands issue detail interactions, and redesigns many application pages. ChangesContainer and platform updates
Community Hub
Main application refresh
Estimated code review effort: 5 (Critical) | ~120 minutes Possibly related PRs
Suggested labels: ✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…d remediate secrets
ae4a3ae to
c3e5c18
Compare
There was a problem hiding this comment.
Your trial has ended. Reactivate Greptile to resume code reviews.
Summary of Changes Included in this Branch
1. Docker Build Speed & Image Size Optimization
• .dockerignore Added: Prevents copying node_modules (2.5GB),
data/ (436MB), community/ (410MB), and docs-site/ (290MB) into
the Docker context. Context size dropped from >3.7 GB to ~15 MB.
• Multi-Stage Dockerfile: Split into deps, prod-deps (with
test/doc pruning), builder, and a lean production runner
(oven/bun:1-slim). Production image size is now 251 MB.
• Dockerfile.worker & Dockerfile.runner: Optimized layer
caching, reduced package footprints, and cleared npm/apk caches.
2. CI/CD Pipeline Acceleration
• Parallel Matrix Builds (docker-publish.yml): Replaced the 4-
hour sequential build with parallel matrix execution across
independent GitHub Actions runners for App, Worker, and Runner.
• Fast Continuous Delivery: Fast single-arch (linux/amd64)
build on main push (< 4 minutes with GHA cache) and full multi-
arch (linux/amd64,linux/arm64) on release tags (v*).
3. Site-Wide UI & Design System Cohesion
• Authentic Branding: Added unified vector logo components
(Logo.astro and Logo.tsx) matching the landing page and profile
aesthetics.
• Whole-Site Polish: Harmonized styles across pulls, issues,
explore, stars, repositories, notifications, actions, dashboard,
merge-queue, stacks, orgs, insights, inbox, login, and register.
• Issue Conversation & Comment Timeline: Built complete
timeline view, original issue card, markdown comment thread,
write/preview composer, and clean sidebar controls in
IssueDetail.tsx.
• Password Reset Pages: Overhauled password-reset.astro and
/reset-password/[token]
file:///home/swadhin/owngit/OpenCodeHub/src/pages/reset-
password/%5Btoken%5D.astro with full dark mode parity, input
icons, and loading feedback.
4. Stability & Local Dev Error Resilience
• Database Safety: Wrapped database queries in
UserProfileView.astro with try/catch and fallback guards
against placeholder-user to prevent ECONNREFUSED crashes during
astro dev.
• Docker Compose Ports: Exposed PostgreSQL (5432) and Redis
(6379) in docker-compose.yml for local development workflows.
Summary by CodeRabbit
New Features
Style
Chores