chore: merge develop to main

[ferr3ira-gabriel]

Summary

Merges develop branch to main, including the new frontend-pr-analysis workflow.

Changes Included

• feat: add frontend-pr-analysis reusable workflow (feat: add frontend-pr-analysis reusable workflow #62)
  • Comprehensive Frontend/Node.js PR analysis for monorepos
  • Change detection, ESLint, TypeScript, npm audit, tests, coverage, build
  • Support for npm, yarn, and pnpm
  • Documentation at docs/frontend-pr-analysis-workflow.md
  • README updated with workflow ci(deps): bump cycjimmy/semantic-release-action from 4 to 5 #13

Summary by CodeRabbit

• New Features

  • Added Frontend PR Analysis workflow for automated frontend monorepo CI/CD, including change detection, per-app linting, type checking, security audits, testing with coverage, and build verification.

• Documentation

  • Added comprehensive workflow documentation and updated README with feature overview.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This pull request introduces a reusable GitHub Actions workflow for comprehensive frontend/Node.js CI on monorepos. The workflow includes change detection, per-app matrix execution, ESLint linting, TypeScript type checking, npm/yarn/pnpm security audits, unit tests with coverage collection, coverage threshold enforcement with PR comments, build verification, and Slack notifications. Documentation files describe workflow stages, configuration options, and usage patterns for both single-app and monorepo repositories. The workflow supports multiple package managers and generates dynamic per-app matrices based on detected changes.

Sequence Diagram

sequenceDiagram
    actor PR as PR Trigger
    participant WF as Frontend PR Analysis<br/>(Workflow)
    participant DC as Detect Changes<br/>(Job)
    participant Matrix as Per-App Matrix<br/>(Dynamic)
    participant Lint as Lint<br/>(Job)
    participant TypeCheck as Type Check<br/>(Job)
    participant Security as Security Audit<br/>(Job)
    participant Tests as Tests + Coverage<br/>(Job)
    participant Coverage as Coverage<br/>(Job)
    participant Build as Build<br/>(Job)
    participant Notify as Notify<br/>(Job)
    participant Slack as Slack<br/>(External)

    PR->>WF: Trigger on PR
    
    WF->>DC: Start: Detect changed files
    DC-->>Matrix: Output: has_changes flag +<br/>app matrix
    
    alt has_changes == true
        Matrix->>Lint: For each app in matrix
        Lint-->>Lint: Run ESLint per app
        
        Matrix->>TypeCheck: For each app in matrix
        TypeCheck-->>TypeCheck: Run TypeScript compiler
        
        Matrix->>Security: For each app in matrix
        Security-->>Security: Run npm/yarn audit
        
        Matrix->>Tests: For each app in matrix
        Tests-->>Tests: Run tests + collect coverage
        Tests->>Coverage: Upload coverage artifacts
        
        Coverage->>Coverage: Download artifacts +<br/>parse metrics
        Coverage->>PR: Post coverage table +<br/>summary comment
        Coverage-->>Coverage: Check threshold<br/>(optional fail)
        
        Matrix->>Build: For each app in matrix
        Build-->>Build: Build each app
    else has_changes == false
        WF->>WF: No Changes: skip processing
    end
    
    Lint-->>Notify: Collect status (passed/failed)
    TypeCheck-->>Notify: Collect status
    Security-->>Notify: Collect status
    Tests-->>Notify: Collect status
    Coverage-->>Notify: Collect status
    Build-->>Notify: Collect status
    
    Notify->>Slack: Post aggregated results
    Slack-->>Notify: Acknowledgment

Loading

Possibly related PRs

• feat: enhanced Go workflows with monorepo support and Slack notifications #23: Introduces/standardizes the shared Slack notification workflow and change-detection patterns used as primitives by this Frontend PR Analysis workflow for monorepo matrix execution and notifications.

Suggested labels

30 min review

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch develop

---

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 576ddb6 and e69186d.

📒 Files selected for processing (3)

• .github/workflows/frontend-pr-analysis.yml
• README.md
• docs/frontend-pr-analysis-workflow.md

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}