mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-03-16 21:53:07 +00:00
fix: address PR #745 review comments
- .gitignore: add missing trailing newline - Dockerfile: remove misleading || echo fallback in Playwright install - index.ts: truncate long paths from beginning instead of end in warning box - verify-claude-auth.ts: use effectiveAuthMethod to prevent undefined authType - agent-context-parser.ts: handle claude-opus alias as Opus 4.6 - thinking-level-selector.tsx: improve model prop documentation Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
2
.gitignore
vendored
2
.gitignore
vendored
@@ -96,4 +96,4 @@ data/credentials.json
|
|||||||
data/
|
data/
|
||||||
.codex/
|
.codex/
|
||||||
.mcp.json
|
.mcp.json
|
||||||
.planning
|
.planning
|
||||||
|
|||||||
@@ -154,7 +154,7 @@ COPY --from=server-builder /app/node_modules ./node_modules
|
|||||||
USER automaker
|
USER automaker
|
||||||
RUN ./node_modules/.bin/playwright install chromium && \
|
RUN ./node_modules/.bin/playwright install chromium && \
|
||||||
echo "=== Playwright Chromium installed ===" && \
|
echo "=== Playwright Chromium installed ===" && \
|
||||||
ls -la /home/automaker/.cache/ms-playwright/ || echo "Playwright browsers installed"
|
ls -la /home/automaker/.cache/ms-playwright/
|
||||||
USER root
|
USER root
|
||||||
|
|
||||||
# Create data and projects directories
|
# Create data and projects directories
|
||||||
|
|||||||
@@ -211,7 +211,13 @@ const BOX_CONTENT_WIDTH = 67;
|
|||||||
pathsCheckedInfo = `
|
pathsCheckedInfo = `
|
||||||
║ ║
|
║ ║
|
||||||
║ ${'Paths checked:'.padEnd(BOX_CONTENT_WIDTH)}║
|
║ ${'Paths checked:'.padEnd(BOX_CONTENT_WIDTH)}║
|
||||||
${pathsChecked.map((p) => `║ ${p.substring(0, BOX_CONTENT_WIDTH - 4).padEnd(BOX_CONTENT_WIDTH - 4)} ║`).join('\n')}`;
|
${pathsChecked
|
||||||
|
.map((p) => {
|
||||||
|
const maxLen = BOX_CONTENT_WIDTH - 4;
|
||||||
|
const display = p.length > maxLen ? '...' + p.slice(-(maxLen - 3)) : p;
|
||||||
|
return `║ ${display.padEnd(maxLen)} ║`;
|
||||||
|
})
|
||||||
|
.join('\n')}`;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -322,11 +322,12 @@ export function createVerifyClaudeAuthHandler() {
|
|||||||
});
|
});
|
||||||
|
|
||||||
// Determine specific auth type for success messages
|
// Determine specific auth type for success messages
|
||||||
|
const effectiveAuthMethod = authMethod ?? 'api_key';
|
||||||
let authType: 'oauth' | 'api_key' | 'cli' | undefined;
|
let authType: 'oauth' | 'api_key' | 'cli' | undefined;
|
||||||
if (authenticated) {
|
if (authenticated) {
|
||||||
if (authMethod === 'api_key') {
|
if (effectiveAuthMethod === 'api_key') {
|
||||||
authType = 'api_key';
|
authType = 'api_key';
|
||||||
} else if (authMethod === 'cli') {
|
} else if (effectiveAuthMethod === 'cli') {
|
||||||
// Check if CLI auth is via OAuth (Claude Code subscription) or generic CLI
|
// Check if CLI auth is via OAuth (Claude Code subscription) or generic CLI
|
||||||
try {
|
try {
|
||||||
const indicators = await getClaudeAuthIndicators();
|
const indicators = await getClaudeAuthIndicators();
|
||||||
|
|||||||
@@ -9,7 +9,8 @@ interface ThinkingLevelSelectorProps {
|
|||||||
selectedLevel: ThinkingLevel;
|
selectedLevel: ThinkingLevel;
|
||||||
onLevelSelect: (level: ThinkingLevel) => void;
|
onLevelSelect: (level: ThinkingLevel) => void;
|
||||||
testIdPrefix?: string;
|
testIdPrefix?: string;
|
||||||
/** Optional model ID to filter available thinking levels (e.g., Opus 4.6 only shows None/Adaptive) */
|
/** Model ID is required for correct thinking level filtering.
|
||||||
|
* Without it, adaptive thinking won't be available for Opus 4.6. */
|
||||||
model?: string;
|
model?: string;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -34,7 +34,7 @@ export const DEFAULT_MODEL = 'claude-opus-4-6';
|
|||||||
*/
|
*/
|
||||||
export function formatModelName(model: string): string {
|
export function formatModelName(model: string): string {
|
||||||
// Claude models
|
// Claude models
|
||||||
if (model.includes('opus-4-6')) return 'Opus 4.6';
|
if (model.includes('opus-4-6') || model === 'claude-opus') return 'Opus 4.6';
|
||||||
if (model.includes('opus')) return 'Opus 4.5';
|
if (model.includes('opus')) return 'Opus 4.5';
|
||||||
if (model.includes('sonnet')) return 'Sonnet 4.5';
|
if (model.includes('sonnet')) return 'Sonnet 4.5';
|
||||||
if (model.includes('haiku')) return 'Haiku 4.5';
|
if (model.includes('haiku')) return 'Haiku 4.5';
|
||||||
|
|||||||
Reference in New Issue
Block a user