Arham-Begani commented on PR #4552: URL: https://github.com/apache/iggy/pull/4552#issuecomment-6382609327
Thanks for the scheduling update. Two final lifecycle corrections are pushed at `672e5ff1`, directly extending `029fdf28`: - Listeners now receive the public client as `this`, with callback identity preserved for removal, duplicate registrations and `once` self-removal. - A throwing `connected` listener no longer enters transport retry handling. The new regressions failed on the published parent and pass with these corrections. [Validation of this exact source](https://github.com/Arham-Begani/Arham-Begani/actions/runs/39939377232) passed 717 unit tests on each of Node 23/24, lint/build, all nine applicable unmodified hooks, 82 TCP tests, 17 TLS tests and 16 BDD scenarios / 114 steps. Windows build, protocol mirror, direct ESLint and 18 focused lifecycle tests also passed. The PR description now records this source and separates these results from upstream CI. Could an authorized maintainer approve the fresh [Pre-merge](https://github.com/apache/iggy/actions/runs/40082406889) and [PR Title](https://github.com/apache/iggy/actions/runs/40082437595) runs when ready? Both report `action_required` on this new head. The earlier green runs apply to `029fdf28`. This update was prepared with AI assistance, as disclosed in the PR. -- 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]
