Skip to content

Commit c862aad

Browse files
author
Marvin Zhang
committed
refactor: Refactor DevlogManager usage and remove deprecated versions
- Updated imports in MCPDevlogAdapter to use the new DevlogManager instead of NewDevlogManager. - Removed the test file for DevlogManager as it is no longer needed. - Refactored web server initialization to use the new DevlogManager and adjusted the configuration accordingly. - Updated devlog routes to use findOrCreateDevlog method instead of createDevlog. - Cleaned up code for better readability and maintainability.
1 parent 4ef3b99 commit c862aad

File tree

10 files changed

+363
-1411
lines changed

10 files changed

+363
-1411
lines changed

.github/copilot-instructions.md

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,22 @@
11
# Devlog Project - Copilot Instructions
22

3+
## Early Development Stage Guidelines
4+
5+
**IMPORTANT**: This project is in early development. We prioritize clean, modern architecture over backwards compatibility.
6+
7+
### Legacy Code Management
8+
- **Remove obsolete code**: Don't preserve legacy implementations when refactoring
9+
- **Modernize aggressively**: Prefer current best practices over maintaining old patterns
10+
- **Clean slate approach**: It's better to rewrite than to patch outdated code
11+
- **Breaking changes are acceptable**: We're not bound by API compatibility during this phase
12+
- **Focus on the future**: Design for where we want to be, not where we've been
13+
14+
### Development Philosophy
15+
- **Quality over continuity**: A well-architected solution is more valuable than preserving broken legacy code
16+
- **Rapid iteration**: Make bold changes to improve the codebase structure
17+
- **Technical debt elimination**: Actively remove code that doesn't serve the current vision
18+
- **Modern tooling**: Always use the latest stable versions and best practices
19+
320
## Dogfooding Guidelines
421

522
This project uses **itself** for development tracking. When working on devlog features, ALWAYS:

packages/core/src/__tests__/devlog-manager-v2.test.ts

Lines changed: 0 additions & 197 deletions
This file was deleted.

packages/core/src/devlog-manager-v2.ts

Lines changed: 0 additions & 135 deletions
This file was deleted.

packages/core/src/devlog-manager-v3.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,19 +19,19 @@ import {
1919

2020
import { StorageProvider, StorageConfig, StorageProviderFactory } from "./storage/storage-provider.js";
2121

22-
export interface NewDevlogManagerOptions {
22+
export interface DevlogManagerOptions {
2323
workspaceRoot?: string;
2424
storage?: StorageConfig;
2525
integrations?: EnterpriseIntegration;
2626
}
2727

28-
export class NewDevlogManager {
28+
export class DevlogManager {
2929
private storageProvider!: StorageProvider;
3030
private workspaceRoot: string;
3131
private integrations?: EnterpriseIntegration;
3232
private initialized = false;
3333

34-
constructor(private options: NewDevlogManagerOptions = {}) {
34+
constructor(private options: DevlogManagerOptions = {}) {
3535
this.workspaceRoot = options.workspaceRoot || process.cwd();
3636
this.integrations = options.integrations;
3737
}

0 commit comments

Comments
 (0)