architecture-reviewer

Review multi-tenant Node.js/Express/MongoDB code for isolation and financial correctness.

Updated Aug 27, 2026
One-click install
npx skills add https://github.com/paulinett1508-dev/SuperCartolaManagerv5-production --skill architecture-reviewer-paulinett1508-dev
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: architecture-reviewer
Source: https://github.com/paulinett1508-dev/SuperCartolaManagerv5-production/tree/main/.claude/skills/architecture-reviewer
Command: npx skills add https://github.com/paulinett1508-dev/SuperCartolaManagerv5-production --skill architecture-reviewer-paulinett1508-dev

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This Skill helps you catch high-risk architectural issues early—especially in multi-tenant Node.js/Express/MongoDB systems—before they turn into data leaks, financial inconsistencies, or brittle features.

Core Features & Use Cases

  • Multi-tenant isolation review: Verifies every MongoDB query and endpoint is properly scoped by liga_id to prevent cross-liga data leakage.
  • Module system integrity checks: Assesses whether dynamic modules are truly isolated, correctly activated per league, and not confused across fundamentally different module types.
  • Financial correctness guardrails: Reviews idempotency, audit trail coverage, session validation, transaction safety, and point rounding rules to reduce irreversible financial bugs.
  • Cache and API consistency analysis: Ensures caches have TTL, are invalidated on writes, are keyed per league, and endpoints follow consistent apiResponse/error handling patterns.
  • Frontend architecture validation: Confirms the Vanilla JS ES-module SPA patterns (including initialization strategy) and prevents accidental inclusion of incompatible frontend patterns.

Quick Start

Ask the architecture-reviewer skill to review a proposed change for multi-tenant isolation, module activation logic, financial integrity, cache invalidation, API consistency, and frontend initialization risks.

Frequently Asked Questions about architecture-reviewer

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I prevent cross-tenant data leakage in a multi-tenant MongoDB application?

To prevent cross-tenant data leakage, you must verify that every MongoDB query and endpoint is properly scoped by a tenant identifier like liga_id. Architectural review checks isolation boundaries across queries, middleware, and route definitions to ensure no unauthorized cross-tenant access occurs.

How does architectural review help ensure financial correctness in a Node.js application?

Financial correctness review assesses idempotency, audit trail coverage, session validation, and transaction safety in Node.js applications. It evaluates point rounding rules and database transaction handling to reduce the risk of irreversible financial bugs and data inconsistencies.

What is the best way to implement cache invalidation for a multi-tenant API?

The best way to implement cache invalidation for a multi-tenant API is to use TTL-based caches keyed per league or tenant, ensuring they are invalidated on writes. Architectural review validates that caching layers maintain TTLs and proper invalidation triggers to prevent stale data.

Can I use Vanilla JavaScript ES-modules for frontend SPA initialization in a multi-tenant system?

Yes, you can use Vanilla JavaScript ES-modules for frontend SPA initialization, and architectural validation confirms this pattern. It checks frontend initialization strategies to ensure compatible ES-module patterns are used and prevents accidental inclusion of incompatible frameworks.

How do I review API consistency and standardized error handling across Express endpoints?

To review API consistency, you check whether endpoints follow standardized apiResponse and error handling patterns across the Express application. Architectural review analyzes route definitions to ensure uniform response structures, error codes, and middleware behavior throughout the API.

When do I need to check module activation logic in a multi-tenant architecture?

You need to check module activation logic when dynamic modules are introduced to a multi-tenant architecture, ensuring they are correctly activated per league. Architectural review assesses whether dynamic modules are truly isolated and not confused across fundamentally different module types.