fix: Connect dashboard to real PR analysis data#11
Merged
Conversation
Testing workflow
add semgrep and action canges
Pipeline/fix workflow
Initial implementation of peer review feature that validates PRs against linked Jira tickets: - Add PeerReviewIntegration class with ticket extraction and analysis - Add JiraMcpClient for MCP-based Jira communication - Add JiraSubAgent for AI-powered requirement validation - Add types for Jira tickets, acceptance criteria, and validation results - Extend analyze command with --peer-review flag - Update config loader to support peerReview configuration section The feature extracts ticket references from PR titles, branch names, and commit messages, then uses AI to validate that code changes satisfy the ticket's acceptance criteria. Work in progress - build verification pending. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Fix LangChain import path (langchain/output_parsers -> @langchain/core/output_parsers) - Fix TypeScript type casting issues in jira-mcp-client.ts - Refactor peer review to use ProviderFactory for multi-provider support - Peer review now uses the same provider/model as the main PR analysis 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add Zhipu AI provider with Anthropic-compatible API support - Fix peerReview config being stripped by Zod validation (add schema) - Make criteriaId optional in AC validation to prevent parsing errors - Update README with comprehensive peer review documentation - Update README with Zhipu AI provider details - Add verbose debug output for peer review troubleshooting 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Resolved conflicts: - README.md: Combined peer review and static analysis documentation - src/cli/commands/analyze.command.ts: Merged imports and output logic 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Added visual dashboard with ROI calculator, quality trends, and risk analysis - Implemented SQLite persistence to store and aggregate PR analysis results - Added npm run cli dashboard command for standalone local usage - Configured Probot server to serve dashboard at root URL - Updated build pipeline to copy static assets to distribution folder - Added documentation for Dashboard features and deployment
- Combines peer-review Jira integration with Zhipu AI provider - Adds analytics dashboard from dime/ui_dashboard branch - Demo branch for 3pm presentation Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Add saveAnalysis() call in analyze command to persist results to database - Fix database path to use module-relative location instead of cwd - Calculate complexity from fileAnalyses instead of missing overallComplexity - Add helper functions for repo info, author, and PR number extraction The dashboard now displays real PR analysis data from any analyzed repository. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Changed action.yml to point to dist/action.js (ncc-bundled) instead of dist/index.js - The bundled action.js includes all dependencies (express, etc.) - Fixes "Cannot find package 'express'" error in GitHub Actions Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
process.cwd()saveAnalysis()call in CLI analyze command to persist analysis resultsfileAnalysesChanges
src/db/index.ts- Database path now resolves relative to module locationsrc/cli/commands/analyze.command.ts- Added helper functions and saveAnalysis callsrc/index.ts- Fixed complexity calculation in webhook handlerTest plan
npm run buildto verify TypeScript compiles/dashboard/api/stats🤖 Generated with Claude Code