mirror of https://github.com/garrytan/gstack.git
refactor: preamble, co-author trailer, and resolver suppression use host configs
- preamble.ts: hostConfigDir derived from config.globalRoot instead of hardcoded Record - utility.ts: generateCoAuthorTrailer reads from config.coAuthorTrailer instead of host switch statement - gen-skill-docs.ts: suppressedResolvers from config skip resolver execution at placeholder replacement time (belt+suspenders with existing ctx.host checks in individual resolvers) Golden-file comparison: all 3 hosts produce IDENTICAL output to baselines. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
parent
9612b1c82e
commit
d82d2c5650
|
|
@ -38,7 +38,7 @@ const claude: HostConfig = {
|
||||||
linkingStrategy: 'real-dir-symlink',
|
linkingStrategy: 'real-dir-symlink',
|
||||||
},
|
},
|
||||||
|
|
||||||
coAuthorTrailer: 'Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>',
|
coAuthorTrailer: 'Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>',
|
||||||
learningsMode: 'full',
|
learningsMode: 'full',
|
||||||
};
|
};
|
||||||
|
|
||||||
|
|
|
||||||
|
|
@ -32,12 +32,11 @@ const codex: HostConfig = {
|
||||||
],
|
],
|
||||||
|
|
||||||
suppressedResolvers: [
|
suppressedResolvers: [
|
||||||
'DESIGN_OUTSIDE_VOICES',
|
'DESIGN_OUTSIDE_VOICES', // design.ts:485 — Codex can't invoke itself
|
||||||
'ADVERSARIAL_STEP',
|
'ADVERSARIAL_STEP', // review.ts:408 — Codex can't invoke itself
|
||||||
'SPEC_REVIEW_LOOP',
|
'CODEX_SECOND_OPINION', // review.ts:257 — Codex can't invoke itself
|
||||||
'PLAN_VERIFICATION_EXEC',
|
'CODEX_PLAN_REVIEW', // review.ts:541 — Codex can't invoke itself
|
||||||
'CODEX_SECOND_OPINION',
|
'REVIEW_ARMY', // review-army.ts:180 — Codex shouldn't orchestrate
|
||||||
'REVIEW_ARMY',
|
|
||||||
],
|
],
|
||||||
|
|
||||||
runtimeRoot: {
|
runtimeRoot: {
|
||||||
|
|
|
||||||
|
|
@ -437,10 +437,14 @@ function processTemplate(tmplPath: string, host: Host = 'claude'): { outputPath:
|
||||||
const ctx: TemplateContext = { skillName, tmplPath, benefitsFrom, host, paths: HOST_PATHS[host], preambleTier };
|
const ctx: TemplateContext = { skillName, tmplPath, benefitsFrom, host, paths: HOST_PATHS[host], preambleTier };
|
||||||
|
|
||||||
// Replace placeholders (supports parameterized: {{NAME:arg1:arg2}})
|
// Replace placeholders (supports parameterized: {{NAME:arg1:arg2}})
|
||||||
|
// Config-driven: suppressedResolvers return empty string for this host
|
||||||
|
const currentHostConfig = getHostConfig(host);
|
||||||
|
const suppressed = new Set(currentHostConfig.suppressedResolvers || []);
|
||||||
let content = tmplContent.replace(/\{\{(\w+(?::[^}]+)?)\}\}/g, (match, fullKey) => {
|
let content = tmplContent.replace(/\{\{(\w+(?::[^}]+)?)\}\}/g, (match, fullKey) => {
|
||||||
const parts = fullKey.split(':');
|
const parts = fullKey.split(':');
|
||||||
const resolverName = parts[0];
|
const resolverName = parts[0];
|
||||||
const args = parts.slice(1);
|
const args = parts.slice(1);
|
||||||
|
if (suppressed.has(resolverName)) return '';
|
||||||
const resolver = RESOLVERS[resolverName];
|
const resolver = RESOLVERS[resolverName];
|
||||||
if (!resolver) throw new Error(`Unknown placeholder {{${resolverName}}} in ${relTmplPath}`);
|
if (!resolver) throw new Error(`Unknown placeholder {{${resolverName}}} in ${relTmplPath}`);
|
||||||
return args.length > 0 ? resolver(ctx, args) : resolver(ctx);
|
return args.length > 0 ? resolver(ctx, args) : resolver(ctx);
|
||||||
|
|
|
||||||
|
|
@ -1,4 +1,5 @@
|
||||||
import type { TemplateContext } from './types';
|
import type { TemplateContext } from './types';
|
||||||
|
import { getHostConfig } from '../../hosts/index';
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* Preamble architecture — why every skill needs this
|
* Preamble architecture — why every skill needs this
|
||||||
|
|
@ -13,10 +14,10 @@ import type { TemplateContext } from './types';
|
||||||
*/
|
*/
|
||||||
|
|
||||||
function generatePreambleBash(ctx: TemplateContext): string {
|
function generatePreambleBash(ctx: TemplateContext): string {
|
||||||
const hostConfigDir: Record<string, string> = { codex: '.codex', factory: '.factory' };
|
const hostConfig = getHostConfig(ctx.host);
|
||||||
const runtimeRoot = (ctx.host !== 'claude')
|
const runtimeRoot = hostConfig.usesEnvVars
|
||||||
? `_ROOT=$(git rev-parse --show-toplevel 2>/dev/null)
|
? `_ROOT=$(git rev-parse --show-toplevel 2>/dev/null)
|
||||||
GSTACK_ROOT="$HOME/${hostConfigDir[ctx.host]}/skills/gstack"
|
GSTACK_ROOT="$HOME/${hostConfig.globalRoot}"
|
||||||
[ -n "$_ROOT" ] && [ -d "$_ROOT/${ctx.paths.localSkillRoot}" ] && GSTACK_ROOT="$_ROOT/${ctx.paths.localSkillRoot}"
|
[ -n "$_ROOT" ] && [ -d "$_ROOT/${ctx.paths.localSkillRoot}" ] && GSTACK_ROOT="$_ROOT/${ctx.paths.localSkillRoot}"
|
||||||
GSTACK_BIN="$GSTACK_ROOT/bin"
|
GSTACK_BIN="$GSTACK_ROOT/bin"
|
||||||
GSTACK_BROWSE="$GSTACK_ROOT/browse/dist"
|
GSTACK_BROWSE="$GSTACK_ROOT/browse/dist"
|
||||||
|
|
|
||||||
|
|
@ -367,13 +367,9 @@ Minimum 0 per category.
|
||||||
}
|
}
|
||||||
|
|
||||||
export function generateCoAuthorTrailer(ctx: TemplateContext): string {
|
export function generateCoAuthorTrailer(ctx: TemplateContext): string {
|
||||||
if (ctx.host === 'codex') {
|
const { getHostConfig } = require('../../hosts/index');
|
||||||
return 'Co-Authored-By: OpenAI Codex <noreply@openai.com>';
|
const hostConfig = getHostConfig(ctx.host);
|
||||||
}
|
return hostConfig.coAuthorTrailer || 'Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>';
|
||||||
if (ctx.host === 'factory') {
|
|
||||||
return 'Co-Authored-By: Factory Droid <droid@users.noreply.github.com>';
|
|
||||||
}
|
|
||||||
return 'Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>';
|
|
||||||
}
|
}
|
||||||
|
|
||||||
export function generateChangelogWorkflow(_ctx: TemplateContext): string {
|
export function generateChangelogWorkflow(_ctx: TemplateContext): string {
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue