Skip to content

Commit b6a91c5

Browse files
committed
Migrate generic settings writes to shared behavior controller
1 parent ed58e35 commit b6a91c5

1 file changed

Lines changed: 12 additions & 7 deletions

File tree

.github/workflows/settings-audit.yml

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,9 @@ on:
77
- 'src/settings-catalog.json'
88
- 'scripts/settings_audit.py'
99
- 'scripts/phase2_catalog_migration.py'
10+
- 'scripts/phase2_behavior_migration.py'
1011
- 'tests/test_settings_audit.py'
12+
- 'tests/test_settings_behavior_migration.py'
1113
- '.github/workflows/settings-audit.yml'
1214
- 'docs/settings-refactor-status.md'
1315
workflow_dispatch:
@@ -31,21 +33,24 @@ jobs:
3133
- name: Complete Phase 2 catalog migration
3234
if: github.event_name == 'pull_request' && github.event.pull_request.head.ref == 'agent/settings-refactor-phase1-safety'
3335
run: python3 scripts/phase2_catalog_migration.py
34-
- name: Commit Phase 2 catalog migration
36+
- name: Install shared settings behavior controller
37+
if: github.event_name == 'pull_request' && github.event.pull_request.head.ref == 'agent/settings-refactor-phase1-safety'
38+
run: python3 scripts/phase2_behavior_migration.py
39+
- name: Commit Phase 2 migrations
3540
if: github.event_name == 'pull_request' && github.event.pull_request.head.ref == 'agent/settings-refactor-phase1-safety'
3641
shell: bash
3742
run: |
38-
if git diff --quiet -- src/settings-catalog.json; then
39-
echo 'Catalog is already migrated.'
43+
if git diff --quiet -- src/settings-catalog.json src/settings.qml; then
44+
echo 'Phase 2 migrations are already applied.'
4045
else
4146
git config user.name 'github-actions[bot]'
4247
git config user.email '41898282+github-actions[bot]@users.noreply.github.com'
43-
git add src/settings-catalog.json
44-
git commit -m 'Complete settings catalog coverage for phase 2'
48+
git add src/settings-catalog.json src/settings.qml
49+
git commit -m 'Apply settings refactor phase 2 migrations'
4550
git push origin HEAD:${{ github.event.pull_request.head.ref }}
4651
fi
47-
- name: Test audit parser
48-
run: python3 -m unittest tests/test_settings_audit.py
52+
- name: Test settings refactor infrastructure
53+
run: python3 -m unittest tests/test_settings_audit.py tests/test_settings_behavior_migration.py
4954
- name: Resolve and fetch ABI base
5055
id: base
5156
shell: bash

0 commit comments

Comments
 (0)