codex-review-mcp 2.1.0 → 2.2.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/mcp-server.js +9 -0
- package/dist/review/buildPrompt.js +2 -0
- package/package.json +1 -1
package/dist/mcp-server.js
CHANGED
|
@@ -107,5 +107,14 @@ server.registerTool('perform_code_review', {
|
|
|
107
107
|
};
|
|
108
108
|
}
|
|
109
109
|
});
|
|
110
|
+
// Global error handlers to prevent server crashes
|
|
111
|
+
process.on('unhandledRejection', (reason, promise) => {
|
|
112
|
+
console.error('[codex-review-mcp] Unhandled Rejection at:', promise, 'reason:', reason);
|
|
113
|
+
// Don't exit - let the server continue running
|
|
114
|
+
});
|
|
115
|
+
process.on('uncaughtException', (error) => {
|
|
116
|
+
console.error('[codex-review-mcp] Uncaught Exception:', error);
|
|
117
|
+
// Don't exit - let the server continue running
|
|
118
|
+
});
|
|
110
119
|
const transport = new StdioServerTransport();
|
|
111
120
|
await server.connect(transport);
|
|
@@ -33,7 +33,9 @@ export function buildPrompt({ diffText, context, focus, version, isStaticReview
|
|
|
33
33
|
'- Do NOT suggest changes that deviate from the project\'s established patterns or introduce external patterns',
|
|
34
34
|
'- When project guidelines conflict with general best practices, prioritize the project guidelines',
|
|
35
35
|
'- For accessibility audits: apply axe-core best practices and WCAG 2.1 Level AA guidelines',
|
|
36
|
+
'- When reviewing accessibility audit results or configs: VERIFY the audit tool is configured for WCAG 2.1 Level AA only. FLAG if audits are misconfigured (using WCAG 2.2, Level AAA, or only Level A tags) as these will report violations that don\'t match the Axe browser extension defaults or project compliance requirements. Correct axe-core tags should be: ["wcag2a", "wcag2aa", "wcag21a", "wcag21aa"].',
|
|
36
37
|
'- CRITICAL Z-INDEX WARNING: Be extremely cautious with z-index changes on modals, overlays, or components. Analyze the entire z-index hierarchy before suggesting changes. Flag any z-index modifications as HIGH RISK and verify they won\'t break stacking context upstream or downstream. Always check for existing z-index patterns in the codebase first.',
|
|
38
|
+
'- MATERIAL-UI/MUI SPACING: Use theme.spacing() for all margin, padding, and gap values instead of hardcoded px/rem/em values to avoid magic numbers (e.g., gap: theme.spacing(3) for 24px). Flag any hardcoded spacing values.',
|
|
37
39
|
focusLine,
|
|
38
40
|
'',
|
|
39
41
|
'BEFORE making ANY suggestion, ask yourself:',
|