Compare commits
9 Commits
task-maste
...
task-maste
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
10b64ec6f5 | ||
|
|
1a1879483b | ||
|
|
d691cbb7ae | ||
|
|
1b7c9637a5 | ||
|
|
9ff5f158d5 | ||
|
|
b2ff06e8c5 | ||
|
|
6438f6c7c8 | ||
|
|
6bbd777552 | ||
|
|
100482722f |
12
.changeset/pre.json
Normal file
12
.changeset/pre.json
Normal file
@@ -0,0 +1,12 @@
|
|||||||
|
{
|
||||||
|
"mode": "pre",
|
||||||
|
"tag": "rc",
|
||||||
|
"initialVersions": {
|
||||||
|
"task-master-ai": "0.27.0",
|
||||||
|
"docs": "0.0.3",
|
||||||
|
"extension": "0.25.1"
|
||||||
|
},
|
||||||
|
"changesets": [
|
||||||
|
"fluffy-clocks-cross"
|
||||||
|
]
|
||||||
|
}
|
||||||
5
.changeset/soft-dingos-grow.md
Normal file
5
.changeset/soft-dingos-grow.md
Normal file
@@ -0,0 +1,5 @@
|
|||||||
|
---
|
||||||
|
"task-master-ai": patch
|
||||||
|
---
|
||||||
|
|
||||||
|
One last testing final final
|
||||||
@@ -1,5 +1,11 @@
|
|||||||
# task-master-ai
|
# task-master-ai
|
||||||
|
|
||||||
|
## 0.27.1-rc.0
|
||||||
|
|
||||||
|
### Patch Changes
|
||||||
|
|
||||||
|
- [#1232](https://github.com/eyaltoledano/claude-task-master/pull/1232) [`f487736`](https://github.com/eyaltoledano/claude-task-master/commit/f487736670ef8c484059f676293777eabb249c9e) Thanks [@Crunchyman-ralph](https://github.com/Crunchyman-ralph)! - Fix module not found for new 0.27.0 release
|
||||||
|
|
||||||
## 0.27.0
|
## 0.27.0
|
||||||
|
|
||||||
### Minor Changes
|
### Minor Changes
|
||||||
|
|||||||
@@ -18,7 +18,8 @@ export * as ui from './utils/ui.js';
|
|||||||
export {
|
export {
|
||||||
checkForUpdate,
|
checkForUpdate,
|
||||||
performAutoUpdate,
|
performAutoUpdate,
|
||||||
displayUpgradeNotification
|
displayUpgradeNotification,
|
||||||
|
compareVersions
|
||||||
} from './utils/auto-update.js';
|
} from './utils/auto-update.js';
|
||||||
|
|
||||||
// Re-export commonly used types from tm-core
|
// Re-export commonly used types from tm-core
|
||||||
|
|||||||
@@ -7,7 +7,6 @@ import https from 'https';
|
|||||||
import chalk from 'chalk';
|
import chalk from 'chalk';
|
||||||
import ora from 'ora';
|
import ora from 'ora';
|
||||||
import boxen from 'boxen';
|
import boxen from 'boxen';
|
||||||
import packageJson from '../../../../package.json' with { type: 'json' };
|
|
||||||
|
|
||||||
export interface UpdateInfo {
|
export interface UpdateInfo {
|
||||||
currentVersion: string;
|
currentVersion: string;
|
||||||
@@ -16,15 +15,18 @@ export interface UpdateInfo {
|
|||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Get current version from package.json
|
* Get current version from build-time injected environment variable
|
||||||
*/
|
*/
|
||||||
function getCurrentVersion(): string {
|
function getCurrentVersion(): string {
|
||||||
try {
|
// Version is injected at build time via TM_PUBLIC_VERSION
|
||||||
return packageJson.version;
|
const version = process.env.TM_PUBLIC_VERSION;
|
||||||
} catch (error) {
|
if (version && version !== 'unknown') {
|
||||||
console.warn('Could not read package.json for version info');
|
return version;
|
||||||
return '0.0.0';
|
|
||||||
}
|
}
|
||||||
|
|
||||||
|
// Fallback for development or if injection failed
|
||||||
|
console.warn('Could not read version from TM_PUBLIC_VERSION, using fallback');
|
||||||
|
return '0.0.0';
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
@@ -33,7 +35,7 @@ function getCurrentVersion(): string {
|
|||||||
* @param v2 - Second version
|
* @param v2 - Second version
|
||||||
* @returns -1 if v1 < v2, 0 if v1 = v2, 1 if v1 > v2
|
* @returns -1 if v1 < v2, 0 if v1 = v2, 1 if v1 > v2
|
||||||
*/
|
*/
|
||||||
function compareVersions(v1: string, v2: string): number {
|
export function compareVersions(v1: string, v2: string): number {
|
||||||
const toParts = (v: string) => {
|
const toParts = (v: string) => {
|
||||||
const [core, pre = ''] = v.split('-', 2);
|
const [core, pre = ''] = v.split('-', 2);
|
||||||
const nums = core.split('.').map((n) => Number.parseInt(n, 10) || 0);
|
const nums = core.split('.').map((n) => Number.parseInt(n, 10) || 0);
|
||||||
|
|||||||
@@ -1,5 +1,12 @@
|
|||||||
# Change Log
|
# Change Log
|
||||||
|
|
||||||
|
## 0.25.2-rc.0
|
||||||
|
|
||||||
|
### Patch Changes
|
||||||
|
|
||||||
|
- Updated dependencies [[`f487736`](https://github.com/eyaltoledano/claude-task-master/commit/f487736670ef8c484059f676293777eabb249c9e)]:
|
||||||
|
- task-master-ai@0.27.1-rc.0
|
||||||
|
|
||||||
## 0.25.0
|
## 0.25.0
|
||||||
|
|
||||||
### Minor Changes
|
### Minor Changes
|
||||||
|
|||||||
@@ -3,7 +3,7 @@
|
|||||||
"private": true,
|
"private": true,
|
||||||
"displayName": "TaskMaster",
|
"displayName": "TaskMaster",
|
||||||
"description": "A visual Kanban board interface for TaskMaster projects in VS Code",
|
"description": "A visual Kanban board interface for TaskMaster projects in VS Code",
|
||||||
"version": "0.25.1",
|
"version": "0.25.2-rc.0",
|
||||||
"publisher": "Hamster",
|
"publisher": "Hamster",
|
||||||
"icon": "assets/icon.png",
|
"icon": "assets/icon.png",
|
||||||
"engines": {
|
"engines": {
|
||||||
@@ -240,7 +240,7 @@
|
|||||||
"check-types": "tsc --noEmit"
|
"check-types": "tsc --noEmit"
|
||||||
},
|
},
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"task-master-ai": "*"
|
"task-master-ai": "0.27.1-rc.0"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@dnd-kit/core": "^6.3.1",
|
"@dnd-kit/core": "^6.3.1",
|
||||||
|
|||||||
8
package-lock.json
generated
8
package-lock.json
generated
@@ -1,12 +1,12 @@
|
|||||||
{
|
{
|
||||||
"name": "task-master-ai",
|
"name": "task-master-ai",
|
||||||
"version": "0.27.0",
|
"version": "0.27.1-rc.0",
|
||||||
"lockfileVersion": 3,
|
"lockfileVersion": 3,
|
||||||
"requires": true,
|
"requires": true,
|
||||||
"packages": {
|
"packages": {
|
||||||
"": {
|
"": {
|
||||||
"name": "task-master-ai",
|
"name": "task-master-ai",
|
||||||
"version": "0.27.0",
|
"version": "0.27.1-rc.0",
|
||||||
"license": "MIT WITH Commons-Clause",
|
"license": "MIT WITH Commons-Clause",
|
||||||
"workspaces": [
|
"workspaces": [
|
||||||
"apps/*",
|
"apps/*",
|
||||||
@@ -357,9 +357,9 @@
|
|||||||
}
|
}
|
||||||
},
|
},
|
||||||
"apps/extension": {
|
"apps/extension": {
|
||||||
"version": "0.25.1",
|
"version": "0.25.2-rc.0",
|
||||||
"dependencies": {
|
"dependencies": {
|
||||||
"task-master-ai": "*"
|
"task-master-ai": "0.27.1-rc.0"
|
||||||
},
|
},
|
||||||
"devDependencies": {
|
"devDependencies": {
|
||||||
"@dnd-kit/core": "^6.3.1",
|
"@dnd-kit/core": "^6.3.1",
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{
|
{
|
||||||
"name": "task-master-ai",
|
"name": "task-master-ai",
|
||||||
"version": "0.27.0",
|
"version": "0.27.1-rc.0",
|
||||||
"description": "A task management system for ambitious AI-driven development that doesn't overwhelm and confuse Cursor.",
|
"description": "A task management system for ambitious AI-driven development that doesn't overwhelm and confuse Cursor.",
|
||||||
"main": "index.js",
|
"main": "index.js",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
|
|||||||
@@ -5079,25 +5079,7 @@ function setupCLI() {
|
|||||||
const programInstance = new Command()
|
const programInstance = new Command()
|
||||||
.name('dev')
|
.name('dev')
|
||||||
.description('AI-driven development task management')
|
.description('AI-driven development task management')
|
||||||
.version(() => {
|
.version(process.env.TM_PUBLIC_VERSION || 'unknown')
|
||||||
// Read version directly from package.json ONLY
|
|
||||||
try {
|
|
||||||
const packageJsonPath = path.join(process.cwd(), 'package.json');
|
|
||||||
if (fs.existsSync(packageJsonPath)) {
|
|
||||||
const packageJson = JSON.parse(
|
|
||||||
fs.readFileSync(packageJsonPath, 'utf8')
|
|
||||||
);
|
|
||||||
return packageJson.version;
|
|
||||||
}
|
|
||||||
} catch (error) {
|
|
||||||
// Silently fall back to 'unknown'
|
|
||||||
log(
|
|
||||||
'warn',
|
|
||||||
'Could not read package.json for version info in .version()'
|
|
||||||
);
|
|
||||||
}
|
|
||||||
return 'unknown'; // Default fallback if package.json fails
|
|
||||||
})
|
|
||||||
.helpOption('-h, --help', 'Display help')
|
.helpOption('-h, --help', 'Display help')
|
||||||
.addHelpCommand(false); // Disable default help command
|
.addHelpCommand(false); // Disable default help command
|
||||||
|
|
||||||
|
|||||||
@@ -100,17 +100,13 @@ describe('Roo Files Inclusion in Package', () => {
|
|||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
test('source Roo files exist in public/assets directory', () => {
|
test('source Roo files exist in assets directory', () => {
|
||||||
// Verify that the source files for Roo integration exist
|
// Verify that the source files for Roo integration exist
|
||||||
expect(
|
expect(
|
||||||
fs.existsSync(
|
fs.existsSync(path.join(process.cwd(), 'assets', 'roocode', '.roo'))
|
||||||
path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo')
|
|
||||||
)
|
|
||||||
).toBe(true);
|
).toBe(true);
|
||||||
expect(
|
expect(
|
||||||
fs.existsSync(
|
fs.existsSync(path.join(process.cwd(), 'assets', 'roocode', '.roomodes'))
|
||||||
path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes')
|
|
||||||
)
|
|
||||||
).toBe(true);
|
).toBe(true);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -16,9 +16,9 @@ describe('Rules Files Inclusion in Package', () => {
|
|||||||
expect(packageJson.files).toContain('dist/**');
|
expect(packageJson.files).toContain('dist/**');
|
||||||
});
|
});
|
||||||
|
|
||||||
test('source rules files exist in public/assets/rules directory', () => {
|
test('source rules files exist in assets/rules directory', () => {
|
||||||
// Verify that the actual rules files exist
|
// Verify that the actual rules files exist
|
||||||
const rulesDir = path.join(process.cwd(), 'public', 'assets', 'rules');
|
const rulesDir = path.join(process.cwd(), 'assets', 'rules');
|
||||||
expect(fs.existsSync(rulesDir)).toBe(true);
|
expect(fs.existsSync(rulesDir)).toBe(true);
|
||||||
|
|
||||||
// Check for the 4 files that currently exist
|
// Check for the 4 files that currently exist
|
||||||
@@ -86,17 +86,13 @@ describe('Rules Files Inclusion in Package', () => {
|
|||||||
expect(rooJsContent.includes('${mode}-rules')).toBe(true);
|
expect(rooJsContent.includes('${mode}-rules')).toBe(true);
|
||||||
});
|
});
|
||||||
|
|
||||||
test('source Roo files exist in public/assets directory', () => {
|
test('source Roo files exist in assets directory', () => {
|
||||||
// Verify that the source files for Roo integration exist
|
// Verify that the source files for Roo integration exist
|
||||||
expect(
|
expect(
|
||||||
fs.existsSync(
|
fs.existsSync(path.join(process.cwd(), 'assets', 'roocode', '.roo'))
|
||||||
path.join(process.cwd(), 'public', 'assets', 'roocode', '.roo')
|
|
||||||
)
|
|
||||||
).toBe(true);
|
).toBe(true);
|
||||||
expect(
|
expect(
|
||||||
fs.existsSync(
|
fs.existsSync(path.join(process.cwd(), 'assets', 'roocode', '.roomodes'))
|
||||||
path.join(process.cwd(), 'public', 'assets', 'roocode', '.roomodes')
|
|
||||||
)
|
|
||||||
).toBe(true);
|
).toBe(true);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|||||||
@@ -96,11 +96,10 @@ import {
|
|||||||
RULES_SETUP_ACTION,
|
RULES_SETUP_ACTION,
|
||||||
RULES_ACTIONS
|
RULES_ACTIONS
|
||||||
} from '../../src/constants/rules-actions.js';
|
} from '../../src/constants/rules-actions.js';
|
||||||
|
import { compareVersions } from '@tm/cli';
|
||||||
|
|
||||||
describe('Commands Module - CLI Setup and Integration', () => {
|
describe('Commands Module - CLI Setup and Integration', () => {
|
||||||
const mockExistsSync = jest.spyOn(fs, 'existsSync');
|
const mockExistsSync = jest.spyOn(fs, 'existsSync');
|
||||||
const mockReadFileSync = jest.spyOn(fs, 'readFileSync');
|
|
||||||
const mockJoin = jest.spyOn(path, 'join');
|
|
||||||
|
|
||||||
beforeEach(() => {
|
beforeEach(() => {
|
||||||
jest.clearAllMocks();
|
jest.clearAllMocks();
|
||||||
@@ -118,52 +117,39 @@ describe('Commands Module - CLI Setup and Integration', () => {
|
|||||||
expect(program.name()).toBe('dev');
|
expect(program.name()).toBe('dev');
|
||||||
});
|
});
|
||||||
|
|
||||||
test('should read version from package.json when available', () => {
|
test('should return version that matches package.json when TM_PUBLIC_VERSION is set', () => {
|
||||||
mockExistsSync.mockReturnValue(true);
|
// Read actual version from package.json
|
||||||
mockReadFileSync.mockReturnValue('{"version": "1.0.0"}');
|
const packageJson = JSON.parse(fs.readFileSync('package.json', 'utf8'));
|
||||||
mockJoin.mockReturnValue('package.json');
|
const expectedVersion = packageJson.version;
|
||||||
|
|
||||||
|
// Set environment variable to match package.json
|
||||||
|
const originalEnv = process.env.TM_PUBLIC_VERSION;
|
||||||
|
process.env.TM_PUBLIC_VERSION = expectedVersion;
|
||||||
|
|
||||||
const program = setupCLI();
|
const program = setupCLI();
|
||||||
const version = program._version();
|
const version = program.version();
|
||||||
expect(mockReadFileSync).toHaveBeenCalledWith('package.json', 'utf8');
|
expect(version).toBe(expectedVersion);
|
||||||
expect(version).toBe('1.0.0');
|
|
||||||
|
// Restore original environment
|
||||||
|
if (originalEnv !== undefined) {
|
||||||
|
process.env.TM_PUBLIC_VERSION = originalEnv;
|
||||||
|
} else {
|
||||||
|
delete process.env.TM_PUBLIC_VERSION;
|
||||||
|
}
|
||||||
});
|
});
|
||||||
|
|
||||||
test('should use default version when package.json is not available', () => {
|
test('should use default version when TM_PUBLIC_VERSION is not available', () => {
|
||||||
mockExistsSync.mockReturnValue(false);
|
const originalEnv = process.env.TM_PUBLIC_VERSION;
|
||||||
|
delete process.env.TM_PUBLIC_VERSION;
|
||||||
|
|
||||||
const program = setupCLI();
|
const program = setupCLI();
|
||||||
const version = program._version();
|
const version = program.version();
|
||||||
expect(mockReadFileSync).not.toHaveBeenCalled();
|
|
||||||
expect(version).toBe('unknown');
|
|
||||||
});
|
|
||||||
|
|
||||||
test('should use default version when package.json reading throws an error', () => {
|
|
||||||
mockExistsSync.mockReturnValue(true);
|
|
||||||
mockReadFileSync.mockImplementation(() => {
|
|
||||||
throw new Error('Read error');
|
|
||||||
});
|
|
||||||
|
|
||||||
// Mock console methods to prevent chalk formatting conflicts
|
|
||||||
const consoleErrorSpy = jest
|
|
||||||
.spyOn(console, 'error')
|
|
||||||
.mockImplementation(() => {});
|
|
||||||
const consoleLogSpy = jest
|
|
||||||
.spyOn(console, 'log')
|
|
||||||
.mockImplementation(() => {});
|
|
||||||
const consoleWarnSpy = jest
|
|
||||||
.spyOn(console, 'warn')
|
|
||||||
.mockImplementation(() => {});
|
|
||||||
|
|
||||||
const program = setupCLI();
|
|
||||||
const version = program._version();
|
|
||||||
expect(mockReadFileSync).toHaveBeenCalled();
|
|
||||||
expect(version).toBe('unknown');
|
expect(version).toBe('unknown');
|
||||||
|
|
||||||
// Restore console methods
|
// Restore original environment
|
||||||
consoleErrorSpy.mockRestore();
|
if (originalEnv !== undefined) {
|
||||||
consoleLogSpy.mockRestore();
|
process.env.TM_PUBLIC_VERSION = originalEnv;
|
||||||
consoleWarnSpy.mockRestore();
|
}
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
||||||
@@ -277,31 +263,6 @@ describe('Commands Module - CLI Setup and Integration', () => {
|
|||||||
|
|
||||||
// Test utility functions that commands rely on
|
// Test utility functions that commands rely on
|
||||||
describe('Version comparison utility', () => {
|
describe('Version comparison utility', () => {
|
||||||
let compareVersions;
|
|
||||||
|
|
||||||
beforeAll(async () => {
|
|
||||||
// Import from @tm/cli instead of commands.js
|
|
||||||
const { compareVersions: cv } = await import(
|
|
||||||
'../../apps/cli/src/utils/auto-update.js'
|
|
||||||
);
|
|
||||||
|
|
||||||
// Create a local compareVersions function for testing
|
|
||||||
compareVersions = (v1, v2) => {
|
|
||||||
const v1Parts = v1.split('.').map((p) => parseInt(p, 10));
|
|
||||||
const v2Parts = v2.split('.').map((p) => parseInt(p, 10));
|
|
||||||
|
|
||||||
for (let i = 0; i < Math.max(v1Parts.length, v2Parts.length); i++) {
|
|
||||||
const v1Part = v1Parts[i] || 0;
|
|
||||||
const v2Part = v2Parts[i] || 0;
|
|
||||||
|
|
||||||
if (v1Part < v2Part) return -1;
|
|
||||||
if (v1Part > v2Part) return 1;
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
};
|
|
||||||
});
|
|
||||||
|
|
||||||
test('compareVersions correctly compares semantic versions', () => {
|
test('compareVersions correctly compares semantic versions', () => {
|
||||||
expect(compareVersions('1.0.0', '1.0.0')).toBe(0);
|
expect(compareVersions('1.0.0', '1.0.0')).toBe(0);
|
||||||
expect(compareVersions('1.0.0', '1.0.1')).toBe(-1);
|
expect(compareVersions('1.0.0', '1.0.1')).toBe(-1);
|
||||||
|
|||||||
@@ -7,6 +7,18 @@ dotenvLoad();
|
|||||||
// Get all TM_PUBLIC_* env variables for build-time injection
|
// Get all TM_PUBLIC_* env variables for build-time injection
|
||||||
const getBuildTimeEnvs = () => {
|
const getBuildTimeEnvs = () => {
|
||||||
const envs: Record<string, string> = {};
|
const envs: Record<string, string> = {};
|
||||||
|
|
||||||
|
// Inject package.json version at build time
|
||||||
|
try {
|
||||||
|
const packageJson = JSON.parse(
|
||||||
|
require('fs').readFileSync('package.json', 'utf8')
|
||||||
|
);
|
||||||
|
envs['TM_PUBLIC_VERSION'] = packageJson.version || 'unknown';
|
||||||
|
} catch (error) {
|
||||||
|
console.warn('Could not read package.json version during build:', error);
|
||||||
|
envs['TM_PUBLIC_VERSION'] = 'unknown';
|
||||||
|
}
|
||||||
|
|
||||||
for (const [key, value] of Object.entries(process.env)) {
|
for (const [key, value] of Object.entries(process.env)) {
|
||||||
if (key.startsWith('TM_PUBLIC_')) {
|
if (key.startsWith('TM_PUBLIC_')) {
|
||||||
// Return the actual value, not JSON.stringify'd
|
// Return the actual value, not JSON.stringify'd
|
||||||
@@ -23,7 +35,7 @@ export default defineConfig(
|
|||||||
'mcp-server': 'mcp-server/server.js'
|
'mcp-server': 'mcp-server/server.js'
|
||||||
},
|
},
|
||||||
outDir: 'dist',
|
outDir: 'dist',
|
||||||
copy: ['public'],
|
copy: ['assets'],
|
||||||
ignoreWatch: ['node_modules', 'dist', 'tests', 'apps/extension'],
|
ignoreWatch: ['node_modules', 'dist', 'tests', 'apps/extension'],
|
||||||
// Bundle only our workspace packages, keep npm dependencies external
|
// Bundle only our workspace packages, keep npm dependencies external
|
||||||
noExternal: [/^@tm\//],
|
noExternal: [/^@tm\//],
|
||||||
|
|||||||
Reference in New Issue
Block a user