Skip to content

Commit c8ecbd2

Browse files
jonphippsclaude
andcommitted
fix: remove references to obsolete build:preset command
- Remove build:preset steps from deploy-dev-legacy.yml workflow - Update build-with-env.js script to remove preset package references - Simplify clean options to only handle theme package The preset package is no longer built separately, resolving CI failures. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <[email protected]>
1 parent 294a72e commit c8ecbd2

File tree

2 files changed

+9
-54
lines changed

2 files changed

+9
-54
lines changed

.github/workflows/deploy-dev-legacy.yml

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -109,9 +109,6 @@ jobs:
109109
- name: Build theme package
110110
run: pnpm build:theme
111111

112-
- name: Build preset package
113-
run: pnpm build:preset
114-
115112
- name: Build portal
116113
run: pnpm exec docusaurus build portal
117114
env:
@@ -186,10 +183,6 @@ jobs:
186183
if: steps.should-build.outputs.build == 'true'
187184
run: pnpm build:theme
188185

189-
- name: Build preset package
190-
if: steps.should-build.outputs.build == 'true'
191-
run: pnpm build:preset
192-
193186
- name: Build ${{ matrix.standard }}
194187
if: steps.should-build.outputs.build == 'true'
195188
run: pnpm exec docusaurus build standards/${{ matrix.standard }}

scripts/build-with-env.js

Lines changed: 9 additions & 47 deletions
Original file line numberDiff line numberDiff line change
@@ -39,14 +39,13 @@ const validSites = discoverSites();
3939
program
4040
.option('--env <environment>', 'Environment to build for')
4141
.option('--site <site>', 'Site to build')
42-
.option('--clean-packages', 'Clean and rebuild theme and preset packages before building')
42+
.option('--clean-packages', 'Clean and rebuild theme package before building')
4343
.option('--clean-theme', 'Clean and rebuild the theme package before building')
44-
.option('--clean-preset', 'Clean and rebuild the preset package before building')
4544
.parse(process.argv);
4645

4746
async function main() {
4847
const options = program.opts();
49-
let { env, site, cleanPackages, cleanTheme, cleanPreset } = options;
48+
let { env, site, cleanPackages, cleanTheme } = options;
5049

5150
// If no environment provided, ask user to select
5251
if (!env) {
@@ -77,32 +76,24 @@ async function main() {
7776
}
7877

7978
// If clean options not specified via CLI, ask user
80-
if (cleanPackages === undefined && cleanTheme === undefined && cleanPreset === undefined) {
79+
if (cleanPackages === undefined && cleanTheme === undefined) {
8180
const cleanAnswer = await inquirer.prompt([
8281
{
8382
type: 'list',
8483
name: 'cleanOption',
8584
message: 'Clean packages before building?',
8685
choices: [
8786
{ name: 'No cleaning', value: 'none' },
88-
{ name: 'Clean both theme and preset packages', value: 'packages' },
89-
{ name: 'Clean theme package only', value: 'theme' },
90-
{ name: 'Clean preset package only', value: 'preset' }
87+
{ name: 'Clean theme package', value: 'theme' }
9188
],
9289
default: 'none'
9390
}
9491
]);
9592

9693
switch (cleanAnswer.cleanOption) {
97-
case 'packages':
98-
cleanPackages = true;
99-
break;
10094
case 'theme':
10195
cleanTheme = true;
10296
break;
103-
case 'preset':
104-
cleanPreset = true;
105-
break;
10697
default:
10798
// none - leave all as undefined/false
10899
break;
@@ -112,7 +103,6 @@ async function main() {
112103
// Default clean options to false if still undefined
113104
cleanPackages = cleanPackages || false;
114105
cleanTheme = cleanTheme || false;
115-
cleanPreset = cleanPreset || false;
116106

117107
// Validate environment
118108
if (!validEnvironments.includes(env)) {
@@ -127,45 +117,17 @@ async function main() {
127117
}
128118

129119
// Clean and rebuild packages if requested
130-
if (cleanPackages) {
131-
console.log('\nCleaning and rebuilding theme and preset packages...');
120+
if (cleanPackages || cleanTheme) {
121+
console.log('\nCleaning and rebuilding theme package...');
132122
try {
133-
execSync('pnpm clear:packages', { stdio: 'inherit' });
123+
execSync('pnpm clear:theme', { stdio: 'inherit' });
134124
execSync('pnpm build:theme', { stdio: 'inherit' });
135-
execSync('pnpm build:preset', { stdio: 'inherit' });
136-
console.log('Theme and preset packages rebuilt successfully.');
125+
console.log('Theme package rebuilt successfully.');
137126

138127
} catch (error) {
139-
console.error('Failed to rebuild packages.');
128+
console.error('Failed to rebuild theme package.');
140129
process.exit(1);
141130
}
142-
} else {
143-
// Clean and rebuild individual packages if requested
144-
if (cleanTheme) {
145-
console.log('\nCleaning and rebuilding theme package...');
146-
try {
147-
execSync('pnpm clear:theme', { stdio: 'inherit' });
148-
execSync('pnpm build:theme', { stdio: 'inherit' });
149-
console.log('Theme package rebuilt successfully.');
150-
151-
} catch (error) {
152-
console.error('Failed to rebuild theme package.');
153-
process.exit(1);
154-
}
155-
}
156-
157-
if (cleanPreset) {
158-
console.log('\nCleaning and rebuilding preset package...');
159-
try {
160-
execSync('pnpm clear:preset', { stdio: 'inherit' });
161-
execSync('pnpm build:preset', { stdio: 'inherit' });
162-
console.log('Preset package rebuilt successfully.');
163-
164-
} catch (error) {
165-
console.error('Failed to rebuild preset package.');
166-
process.exit(1);
167-
}
168-
}
169131
}
170132

171133
// Build command

0 commit comments

Comments
 (0)