Skip to content

Commit ee1d536

Browse files
xiaoyu2erxiaoyu2er
committed
feat: update check-locale-changes
1 parent bb737d2 commit ee1d536

File tree

5 files changed

+539
-17
lines changed

5 files changed

+539
-17
lines changed

‎.github/actions/check-locale-changes/action.yml‎

Lines changed: 63 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ name: 'Check Locale Changes'
22
description: 'Check which enabled locales have changes and generate deployment matrix'
33
inputs:
44
trigger-type:
5-
description: 'Type of trigger: "manual", "auto", or "docs-pr"'
5+
description: 'Type of trigger: "manual" or "auto"'
66
required: true
77
manual-locales:
88
description: 'Comma-separated list of locales for manual trigger (optional)'
@@ -11,17 +11,73 @@ inputs:
1111
outputs:
1212
matrix-include:
1313
description: 'JSON array of locales to deploy with their config'
14-
value: ${{ steps.check-locales.outputs.matrix-include }}
14+
value: ${{ steps.generate-matrix.outputs.include }}
1515
has-changes:
1616
description: 'Whether any enabled locales have changes'
17-
value: ${{ steps.check-locales.outputs.has-changes }}
17+
value: ${{ steps.generate-matrix.outputs.has-changes }}
1818

1919
runs:
2020
using: 'composite'
2121
steps:
22-
- name: Check locale changes and generate matrix
23-
id: check-locales
22+
- name: Generate dynamic files config (for auto triggers)
23+
if: inputs.trigger-type == 'auto'
24+
id: generate-files-config
2425
shell: bash
2526
run: |
26-
# Use the unified Node.js script to handle everything
27-
node .github/scripts/check-locale-changes.js "${{ inputs.trigger-type }}" "${{ inputs.manual-locales }}" "true"
27+
# Use the dedicated script to generate files config
28+
files_yaml=$(node .github/scripts/generate-files-config.js)
29+
30+
echo "Generated files_yaml:"
31+
echo "$files_yaml"
32+
33+
# Save to output for next step
34+
{
35+
echo "files_yaml<<EOF"
36+
echo "$files_yaml"
37+
echo "EOF"
38+
} >> $GITHUB_OUTPUT
39+
40+
- name: Get changed files (for auto triggers)
41+
if: inputs.trigger-type == 'auto'
42+
id: changes
43+
uses: tj-actions/changed-files@v41
44+
with:
45+
files_yaml: ${{ steps.generate-files-config.outputs.files_yaml }}
46+
47+
- name: Generate deployment matrix
48+
id: generate-matrix
49+
shell: bash
50+
run: |
51+
# Prepare arguments for the matrix generation script
52+
trigger_type="${{ inputs.trigger-type }}"
53+
manual_locales="${{ inputs.manual-locales }}"
54+
55+
if [ "$trigger_type" == "manual" ]; then
56+
# For manual trigger, we don't need changes JSON
57+
output=$(node .github/scripts/generate-locale-matrix.js "$trigger_type" "$manual_locales")
58+
else
59+
# For auto triggers, create a minimal JSON with only the boolean change indicators
60+
changes_json=$(cat << 'EOF'
61+
{
62+
"core_any_changed": "${{ steps.changes.outputs.core_any_changed }}",
63+
"ar_any_changed": "${{ steps.changes.outputs.ar_any_changed }}",
64+
"de_any_changed": "${{ steps.changes.outputs.de_any_changed }}",
65+
"en_any_changed": "${{ steps.changes.outputs.en_any_changed }}",
66+
"es_any_changed": "${{ steps.changes.outputs.es_any_changed }}",
67+
"fr_any_changed": "${{ steps.changes.outputs.fr_any_changed }}",
68+
"ja_any_changed": "${{ steps.changes.outputs.ja_any_changed }}",
69+
"ru_any_changed": "${{ steps.changes.outputs.ru_any_changed }}",
70+
"zh-hans_any_changed": "${{ steps.changes.outputs.zh-hans_any_changed }}",
71+
"zh-hant_any_changed": "${{ steps.changes.outputs.zh-hant_any_changed }}"
72+
}
73+
EOF
74+
)
75+
76+
temp_file=$(mktemp)
77+
echo "$changes_json" > "$temp_file"
78+
output=$(node .github/scripts/generate-locale-matrix.js "$trigger_type" "" "$temp_file")
79+
rm -f "$temp_file"
80+
fi
81+
82+
# Parse the output (the script outputs two lines: include= and has-changes=)
83+
echo "$output" >> $GITHUB_OUTPUT

‎.github/scripts/check-locale-changes.js‎

Lines changed: 5 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -38,9 +38,7 @@ function usage() {
3838
);
3939
console.log('');
4040
console.log('Arguments:');
41-
console.log(
42-
" trigger-type Type of trigger: 'manual', 'auto', or 'docs-pr'",
43-
);
41+
console.log(" trigger-type Type of trigger: 'manual' or 'auto'");
4442
console.log(
4543
' manual-locales Comma-separated list of locales (optional, for manual trigger)',
4644
);
@@ -280,7 +278,7 @@ function processManualTrigger(localeConfig, manualLocales) {
280278
}
281279

282280
/**
283-
* Process automatic/docs-pr trigger
281+
* Process automatic trigger
284282
*/
285283
function processAutoTrigger(localeConfig, changes) {
286284
let matrixInclude = [];
@@ -398,10 +396,8 @@ function main() {
398396
usage();
399397
}
400398

401-
if (!['manual', 'auto', 'docs-pr'].includes(triggerType)) {
402-
console.error(
403-
"Error: Invalid trigger-type. Must be 'manual', 'auto', or 'docs-pr'",
404-
);
399+
if (!['manual', 'auto'].includes(triggerType)) {
400+
console.error("Error: Invalid trigger-type. Must be 'manual' or 'auto'");
405401
usage();
406402
}
407403

@@ -419,7 +415,7 @@ function main() {
419415
if (triggerType === 'manual') {
420416
result = processManualTrigger(localeConfig, manualLocales);
421417
} else {
422-
// For auto and docs-pr triggers, generate files config and detect changes
418+
// For auto triggers, generate files config and detect changes
423419
filesYaml = generateFilesConfig(localeConfig);
424420
log('Generated files configuration for change detection');
425421

‎.github/scripts/generate-files-config.js‎

Lines changed: 113 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,113 @@
1+
#!/usr/bin/env node
2+
3+
/**
4+
* Script to generate dynamic files configuration for change detection
5+
* Usage: node generate-files-config.js
6+
*/
7+
8+
const fs = require('node:fs');
9+
const path = require('node:path');
10+
11+
// Default values
12+
const SCRIPT_DIR = __dirname;
13+
const ROOT_DIR = path.resolve(SCRIPT_DIR, '../..');
14+
const LOCALE_CONFIG_FILE = path.join(ROOT_DIR, '.github/locales-config.json');
15+
16+
/**
17+
* Log messages with timestamp
18+
* @param {string} message
19+
*/
20+
function log(message) {
21+
const timestamp = new Date().toISOString().replace('T', ' ').substring(0, 19);
22+
console.error(`[${timestamp}] ${message}`);
23+
}
24+
25+
/**
26+
* Remove comments from JSON string
27+
* @param {string} jsonString
28+
* @returns {string}
29+
*/
30+
function stripJsonComments(jsonString) {
31+
// Remove single line comments starting with //
32+
return jsonString.replace(/^\s*\/\/.*$/gm, '');
33+
}
34+
35+
/**
36+
* Validate JSON content
37+
* @param {string} jsonString
38+
* @returns {boolean}
39+
*/
40+
function validateJson(jsonString) {
41+
try {
42+
JSON.parse(stripJsonComments(jsonString));
43+
return true;
44+
} catch (error) {
45+
return false;
46+
}
47+
}
48+
49+
/**
50+
* Main function to generate files configuration
51+
*/
52+
function main() {
53+
try {
54+
// Check if locale config file exists
55+
if (!fs.existsSync(LOCALE_CONFIG_FILE)) {
56+
console.error(
57+
`Error: Locale config file not found: ${LOCALE_CONFIG_FILE}`,
58+
);
59+
process.exit(1);
60+
}
61+
62+
// Read locale config
63+
let localeConfigContent;
64+
try {
65+
localeConfigContent = fs.readFileSync(LOCALE_CONFIG_FILE, 'utf8');
66+
} catch (error) {
67+
console.error('Error: Failed to read locale config file');
68+
process.exit(1);
69+
}
70+
71+
// Validate locale config JSON
72+
if (!validateJson(localeConfigContent)) {
73+
console.error('Error: Invalid JSON in locale config file');
74+
process.exit(1);
75+
}
76+
77+
const localeConfig = JSON.parse(stripJsonComments(localeConfigContent));
78+
79+
log('=== Files Config Generator ===');
80+
log(`Locale config file: ${LOCALE_CONFIG_FILE}`);
81+
82+
// Start with core files config
83+
let filesYaml = `core:
84+
- 'apps/docs/**'
85+
- 'packages/**'
86+
- '!apps/docs/content/**'
87+
- '!apps/docs/messages/**'`;
88+
89+
// Add each locale from config dynamically
90+
const locales = Object.keys(localeConfig);
91+
for (const locale of locales) {
92+
filesYaml += `
93+
${locale}:
94+
- 'apps/docs/content/${locale}/**'
95+
- 'apps/docs/messages/${locale}.json'`;
96+
}
97+
98+
log('Generated files_yaml configuration');
99+
100+
// Output the files_yaml configuration
101+
console.log(filesYaml);
102+
} catch (error) {
103+
console.error(`Error: ${error.message}`);
104+
process.exit(1);
105+
}
106+
}
107+
108+
// Run main function if script is executed directly
109+
if (require.main === module) {
110+
main();
111+
}
112+
113+
module.exports = { main };

0 commit comments

Comments
 (0)