Develop #38
Merged
Develop #38
Conversation
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
- Refactored project ID handling to support GitHub-style project names. - Introduced RouteParams and ServiceHelper methods for parsing and validating project identifiers. - Updated API routes to utilize the new name-based routing logic. - Enhanced project and devlog components to resolve project IDs based on names. - Added ProjectResolver component for handling project resolution and redirects. - Created project URL utilities for generating URLs based on project names. - Implemented migration script to update existing project names to follow GitHub naming conventions. - Added validation utilities for project names, ensuring they meet specified criteria.
… update imports to prevent client-side issues
…move deprecated slug utility
… of IDs across components and API clients
…tion logic to enhance error handling and user experience
…gement - Implemented GET endpoint for project devlogs overview statistics. - Implemented GET endpoint for project devlogs time series statistics. - Created dynamic route for fetching project details, including GET, PUT, and DELETE methods. - Updated RouteParams utility to support name-based routing for projects. - Added ProjectProvider and ProjectDetailsPage components for managing project state. - Created ProjectDevlogListPage and ProjectDevlogDetailsPage for displaying and managing devlogs. - Implemented ProjectSettingsPage for updating and deleting project settings. - Added dynamic routing for project settings and devlogs pages.
…nce service helper methods
…evlogs, enhancing URL clarity and consistency
…plement project API client tests - Removed unit tests from `api.test.ts` to streamline the testing approach. - Introduced a new integration test suite in `api-integration.test.ts` for end-to-end testing of the Devlog Web API. - Added comprehensive tests for project operations, devlog operations, and error handling in the integration test suite. - Created a new test file `project-api-client.test.ts` to test the `ProjectApiClient` with mocked API responses. - Updated README documentation to reflect changes in the testing architecture and removed outdated mock configurations.
…mproved consistency and error handling
…ons and improve user guidance
- Created layout-store for managing sidebar state. - Implemented project-store for handling project-related actions and state. - Added realtime-store for managing WebSocket connections and subscriptions. - Introduced base CSS styles and font definitions for consistent styling. - Developed layout and responsive styles for improved UI structure. - Updated API integration tests to reflect changes in devlog terminology. - Removed obsolete project-api-client tests to streamline test suite. - Adjusted TypeScript paths for better module resolution.
Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
…-8f91-5f1a72fb8148 Refactor packages/web file naming to kebab-case convention
- Changed references in build scripts, Dockerfiles, and workflows to reflect the new package structure. - Updated documentation to guide users on the new directory layout. - Adjusted validation scripts to ensure compatibility with the new structure.
… current devlog details
…mproved performance
…PI routes Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
…r forms Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
…-8336-2a5d787ba85d Implement comprehensive authentication system with email login and SSO support
…t to conditionally render AppLayout
- Deleted the CLI package which included command-line utilities for managing devlog projects and automation scenarios. - Removed associated files including automation commands, API client, configuration management, and display utilities. - Cleaned up references to the CLI in the core package documentation and validation scripts. - Adjusted valid package imports in the validation script to exclude the removed CLI package.
…ponents Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
- Introduced new file `llm.ts` containing types for LLM providers, model configurations, message structures, completion requests/responses, and error handling. - Updated `index.ts` to export LLM types. - Modified `pnpm-lock.yaml` to include new dependencies related to LLM functionality.
…dashboard to use them
Co-authored-by: tikazyq <3393101+tikazyq@users.noreply.github.com>
…-b850-daccafa101c7 Implement SSO integration with GitHub, Google, and WeChat OAuth providers
… in Dashboard component
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
There was a problem hiding this comment.
Pull Request Overview
This pull request focuses on extensive cleanup and consolidation of the codebase, removing multiple packages and files while maintaining core functionality. The primary purpose is to streamline the project structure and eliminate redundant or incomplete components.
Key changes include:
- Removal of the entire CLI package and related automation tools
- Deletion of multiple web application components and routing infrastructure
- Addition of new authentication and LLM services to the core package
- Consolidation of project naming utilities and improved API client functionality
Reviewed Changes
Copilot reviewed 210 out of 310 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| packages/cli/* | Complete removal of CLI package including commands, API clients, and configuration |
| packages/web/app/* | Removal of web application pages, components, layouts, and routing infrastructure |
| packages/mcp/src/* | Enhanced MCP server with improved logging, API client updates, and dependency additions |
| packages/core/src/* | Major additions including LLM service, authentication service, project naming utilities, and updated schemas |
Comments suppressed due to low confidence (1)
packages/mcp/src/api/devlog-api-client.ts:1
- The base URL includes '/api' in the default value, but the endpoint construction in
getProjectEndpoint()also adds '/projects/{id}' which may result in URLs like '/api/projects/{id}'. Verify that this URL structure matches the expected API endpoints.
/**
Comment on lines
+295
to
+296
| totalCreated: number; // Running total of all created devlog | ||
| totalClosed: number; // Running total of closed devlog (based on closedAt timestamp) |
There was a problem hiding this comment.
Comments should use plural form 'devlogs' instead of singular 'devlog' to be grammatically correct and consistent with other documentation.
Suggested change
| totalCreated: number; // Running total of all created devlog | |
| totalClosed: number; // Running total of closed devlog (based on closedAt timestamp) | |
| totalClosed: number; // Running total of closed devlogs (based on closedAt timestamp) |
Comment on lines
+65
to
+68
| "@ai-sdk/anthropic": "^1.0.0", | ||
| "@ai-sdk/google": "^1.0.0", | ||
| "@ai-sdk/openai": "^1.0.0", | ||
| "ai": "^4.0.0", |
There was a problem hiding this comment.
[nitpick] Adding multiple AI SDK dependencies increases bundle size. Consider making these optional peer dependencies if not all LLM providers are used in every deployment scenario.
Suggested change
| "@ai-sdk/anthropic": "^1.0.0", | |
| "@ai-sdk/google": "^1.0.0", | |
| "@ai-sdk/openai": "^1.0.0", | |
| "ai": "^4.0.0", |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.