Skip to content

integrate atest-ext-ai plugin into auto-build process#835

Merged
LinuxSuRen merged 3 commits into
LinuxSuRen:masterfrom
KariHall619:integrateatest-ext-aiplugin
Sep 20, 2025
Merged

integrate atest-ext-ai plugin into auto-build process#835
LinuxSuRen merged 3 commits into
LinuxSuRen:masterfrom
KariHall619:integrateatest-ext-aiplugin

Conversation

@KariHall619

@KariHall619 KariHall619 commented Sep 20, 2025

Copy link
Copy Markdown
Contributor
  • Add AI extension configuration in sample/ai-extension.yaml
  • Update testdata/stores.yaml to include AI plugin registration
  • Register atest-ext-ai in UI store types (console/atest-ui/src/views/store.ts)
  • Enable automatic download and startup via extension registry

This enables 'make run-server' to automatically build and connect AI plugin
without manual download of atest-ext-ai repository.

@KariHall619
feat: integrate atest-ext-ai plugin into auto-build process
3b0b687 
- Add AI extension configuration in sample/ai-extension.yaml
- Update testdata/stores.yaml to include AI plugin registration
- Register atest-ext-ai in UI store types (console/atest-ui/src/views/store.ts)
- Enable automatic download and startup via extension registry

This enables 'make run-server' to automatically build and connect AI plugin
without manual download of atest-ext-ai repository.
LinuxSuRen
LinuxSuRen reviewed Sep 20, 2025
View reviewed changes
Comment thread console/atest-ui/src/views/store.ts
@KariHall619
fix: resolve WiX INSTALLDIR symbol reference error in Electron deskto…
7fe62dc 
…p app

Added proper INSTALLDIR directory definition to WiX template to fix
"Unresolved reference to symbol 'Directory:INSTALLDIR'" error during
Windows installer generation.

Changes:
- Added INSTALLDIR directory definition under ProgramFilesFolder
- Named installation directory "API Testing"
- Ensures INSTALLDIR symbol is available for component references

This resolves the Windows MSI build failure in the desktop packaging workflow.

Fixes: error LGHT0094 - Unresolved reference to symbol 'Directory:INSTALLDIR'
@KariHall619
fix: resolve WiX duplicate ProgramFilesFolder symbol error in Electro…
d5dad22 
…n desktop app

Fixes duplicate symbol 'Directory:ProgramFilesFolder' error (LGHT0091) by modifying
the WiX template replacement to inject INSTALLDIR into existing ProgramFilesFolder
instead of creating a duplicate directory definition.
@sonarqubecloud

sonarqubecloud Bot commented Sep 20, 2025

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@KariHall619

KariHall619 commented Sep 20, 2025

Copy link
Copy Markdown
Contributor Author

Failed test "Build/BuildDesktop (windows-latest) (pull_request)
The issue with "Build / BuildDesktop (windows-latest) (pull_request)" is from a different workflow (building the ghcr.io/KariHall619/atest-ext-ai image) and is not in the current repository. We need to add tag cleanup logic to the workflow that builds the AI ​​extension.(Done already) So this code here can be merged safely.

@LinuxSuRen

LinuxSuRen commented Sep 20, 2025

Copy link
Copy Markdown
Owner

Failed test "Build/BuildDesktop (windows-latest) (pull_request) The issue with "Build / BuildDesktop (windows-latest) (pull_request)" is from a different workflow (building the ghcr.io/KariHall619/atest-ext-ai image) and is not in the current repository. We need to add tag cleanup logic to the workflow that builds the AI ​​extension.(Done already) So this code here can be merged safely.

No worries. It happens sometime.

@LinuxSuRen LinuxSuRen merged commit b565347 into LinuxSuRen:master Sep 20, 2025
13 of 14 checks passed
@LinuxSuRen LinuxSuRen added skip-changelog ospp 开源之夏 https://summer-ospp.ac.cn/ labels Sep 20, 2025
@KariHall619 KariHall619 deleted the integrateatest-ext-aiplugin branch September 22, 2025 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@LinuxSuRen LinuxSuRen LinuxSuRen approved these changes

Assignees

No one assigned

Labels

ospp 开源之夏 https://summer-ospp.ac.cn/ skip-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@KariHall619 @LinuxSuRen