yuluo-yx opened a new pull request, #2853:
URL: https://github.com/apache/rocketmq-dashboard/pull/2853

   ## What is the purpose of the change
   
   Consolidate the reviewed build, contributor, E2E lifecycle, and Nginx 
edge-contract changes from #2723, #2726, #2727, and #2733 into one 
tooling/deployment PR, following the maintainer guidance on #2724.
   
   The old #2725 review-skill patch is intentionally excluded because its 
`.claude/skills/pr-review/SKILL.md` target no longer exists on the latest 
`rocketmq-studio` branch.
   
   ## Brief changelog
   
   - document reproducible frontend inputs and declare Node.js `>=20.19.0` in 
package metadata;
   - exclude local Vite environment overrides from Docker context;
   - align server contributor guidance and targeted-test commands with the 
current codebase;
   - validate native-alert E2E inputs and clean owned process/cookie/log 
artifacts on exit or signals, with explicit retention support;
   - proxy Springdoc/Swagger routes at Nginx and apply streaming settings only 
to `/api/ai/chat`;
   - add executable shell and static Nginx contract regressions.
   
   ## Verifying this change
   
   - native alert lifecycle regression passed;
   - Nginx edge contract passed;
   - ShellCheck passed for all three changed shell scripts;
   - `bash -n` and `sh -n` passed;
   - Node 24.18.0 `npm run build` passed (8,025 modules);
   - `git diff --check` passed;
   - scope: 9 files, 300 insertions, 37 deletions, 4 reviewed commits.
   
   Fixes #2720
   
   Fixes #2721
   
   Fixes #2722
   
   Fixes #2731
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to