update deployment docs for minio bucket creation

[sallyom]

updating deployment to include minio bucket creation

[github-actions]

Claude Code Review

Summary

This PR updates deployment documentation and automation to include MinIO bucket creation. However, it includes 560 lines of seemingly unrelated README.md changes.

Files Changed:

• README.md (+560 lines): Massive documentation dump
• components/manifests/deploy.sh (+22 lines): MinIO automation
• docs/deployment/minio-quickstart.md (+29/-15): Updated docs

Issues by Severity

Blocker: README.md contains 560-line unrelated dump

The PR adds GitLab integration docs, prerequisites, quick start guide, development workflows, and more to README.md. This violates CLAUDE.md standards about avoiding top-level doc proliferation.

Action: Remove or relocate this content. Only MinIO-specific changes belong in this PR.

Critical: Hardcoded paths in deploy.sh

Uses relative ../../scripts/setup-minio.sh which fails if run from different directories.

Fix: Use proper SCRIPT_DIR resolution with dirname "${BASH_SOURCE[0]}".

Major Issues

1. Unclear if MinIO is required - error handling allows continuation but unclear if sessions will work
2. Documentation promises automatic setup but code shows conditional behavior

Positive Highlights

✅ Good automation improvement
✅ Helpful error messages
✅ Clear deployment feedback

Recommendations

Priority 1 (Must Fix):

1. Remove 560-line README.md dump
2. Fix hardcoded paths

Priority 2 (Should Fix):
3. Clarify MinIO requirement
4. Match docs to code behavior

Overall: Core MinIO automation is solid, but README.md changes must be addressed before merge.

---

🔍 View AI decision process (logs available for 90 days)

📋 View memory system files loaded (click to expand)

What Amber Loaded for Code Review

Amber automatically loaded these repository standards from the memory system:

1. CLAUDE.md - Master project instructions, development standards
2. backend-development.md - Go backend, K8s integration patterns
3. frontend-development.md - NextJS, Shadcn UI, React Query patterns
4. security-standards.md - Auth, RBAC, token handling
5. k8s-client-usage.md - User token vs service account patterns
6. error-handling.md - Consistent error patterns
7. react-query-usage.md - Data fetching patterns

Impact: This review used your repository's specific code quality standards, security patterns, and best practices from the memory system (PRs #359, #360) - not just generic code review guidelines.