ros/claude-task-master
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: runtime env variables working with new tm_ env variables

Browse Source
This commit is contained in:
Ralph Khreish
2025-10-15 22:47:30 +02:00
parent 9853f39325
commit 8957110b50
4 changed files with 19 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
apps/cli/src/commands/context.command.ts
View File

@@ -501,7 +501,7 @@ export class ContextCommand extends Command {
if (!briefId) { if (!briefId) {
spinner.fail('Could not extract a brief ID from the provided input'); spinner.fail('Could not extract a brief ID from the provided input');
ui.displayError( ui.displayError(
`Provide a valid brief ID or a Hamster brief URL, e.g. https://${process.env.TM_PUBLIC_BASE_DOMAIN}/home/hamster/briefs/<id>` `Provide a valid brief ID or a Hamster brief URL, e.g. https://${process.env.TM_BASE_DOMAIN || process.env.TM_PUBLIC_BASE_DOMAIN}/home/hamster/briefs/<id>`
); );
process.exit(1); process.exit(1);
} }

9
packages/tm-core/src/auth/credential-store.ts
View File

@@ -24,6 +24,8 @@ export class CredentialStore {
private config: AuthConfig; private config: AuthConfig;
// Clock skew tolerance for expiry checks (30 seconds) // Clock skew tolerance for expiry checks (30 seconds)
private readonly CLOCK_SKEW_MS = 30_000; private readonly CLOCK_SKEW_MS = 30_000;
// Track if we've already warned about missing expiration to avoid spam
private hasWarnedAboutMissingExpiration = false;
private constructor(config?: Partial<AuthConfig>) { private constructor(config?: Partial<AuthConfig>) {
this.config = getAuthConfig(config); this.config = getAuthConfig(config);
@@ -84,7 +86,11 @@ export class CredentialStore {
// Validate expiration time for tokens // Validate expiration time for tokens
if (expiresAtMs === undefined) { if (expiresAtMs === undefined) {
// Only log this warning once to avoid spam during auth flows
if (!this.hasWarnedAboutMissingExpiration) {
this.logger.warn('No valid expiration time provided for token'); this.logger.warn('No valid expiration time provided for token');
this.hasWarnedAboutMissingExpiration = true;
}
return null; return null;
} }
@@ -174,6 +180,9 @@ export class CredentialStore {
mode: 0o600 mode: 0o600
}); });
fs.renameSync(tempFile, this.config.configFile); fs.renameSync(tempFile, this.config.configFile);
// Reset the warning flag so it can be shown again for future invalid tokens
this.hasWarnedAboutMissingExpiration = false;
} catch (error) { } catch (error) {
throw new AuthenticationError( throw new AuthenticationError(
`Failed to save auth credentials: ${(error as Error).message}`, `Failed to save auth credentials: ${(error as Error).message}`,

7
packages/tm-core/src/services/export.service.ts
View File

@@ -358,11 +358,12 @@ export class ExportService {
tasks: any[] tasks: any[]
): Promise<void> { ): Promise<void> {
// Check if we should use the API endpoint or direct Supabase // Check if we should use the API endpoint or direct Supabase
const useAPIEndpoint = process.env.TM_PUBLIC_BASE_DOMAIN; const apiEndpoint =
process.env.TM_BASE_DOMAIN || process.env.TM_PUBLIC_BASE_DOMAIN;
if (useAPIEndpoint) { if (apiEndpoint) {
// Use the new bulk import API endpoint // Use the new bulk import API endpoint
const apiUrl = `${process.env.TM_PUBLIC_BASE_DOMAIN}/ai/api/v1/briefs/${briefId}/tasks`; const apiUrl = `${apiEndpoint}/ai/api/v1/briefs/${briefId}/tasks`;
// Transform tasks to flat structure for API // Transform tasks to flat structure for API
const flatTasks = this.transformTasksForBulkImport(tasks); const flatTasks = this.transformTasksForBulkImport(tasks);

5
scripts/dev.js
View File

@@ -9,6 +9,8 @@
*/ */
import dotenv from 'dotenv'; import dotenv from 'dotenv';
// Load .env BEFORE any other imports to ensure env vars are available
dotenv.config(); dotenv.config();
// Add at the very beginning of the file // Add at the very beginning of the file
@@ -16,7 +18,8 @@ if (process.env.DEBUG === '1') {
console.error('DEBUG - dev.js received args:', process.argv.slice(2)); console.error('DEBUG - dev.js received args:', process.argv.slice(2));
} }
import { runCLI } from './modules/commands.js'; // Use dynamic import to ensure dotenv.config() runs before module-level code executes
const { runCLI } = await import('./modules/commands.js');
// Run the CLI with the process arguments // Run the CLI with the process arguments
runCLI(process.argv); runCLI(process.argv);