Code refactor with rules and coding standards#214
Open
vishnuv688 wants to merge 88 commits into
Open
Conversation
…er; fix service vite externals
…d getRequestType helpers
…omStack stack-frame helpers
…nd routes use them
…nd routes use them
…G_SOURCES satisfies it
…dapters and backend use both
…nightwatch/demo:selenium root scripts
…ers; move TestStats/SuiteStats to shared
…omplete; route clientDisconnected to parent worker; suite-level feature rerun uses .feature file path; skip pkill in reuse mode; don't finalize root suite per scenario
…key dedupe, rerun sibling preservation, child-rerun isolation, state derivation)
…/step-defs module
…rce + AST helpers) into utils/source-mapping module; collapse utils.ts to a thin barrel
…o subfolder modules (utils/*) don't leak constants.ts as a non-emitted external
…sforms into mark-running module
…unner; replace 17 bare-string references across backend, nightwatch, service, selenium adapters
…; replace bare-string scope refs across backend/core/app/nightwatch/selenium
…tection rules, and RetryTracker state machine
… type cast on browser surface; cast as readonly string[] for INTERNAL_COMMANDS_TO_IGNORE.includes
…te-merge, DataManager, markers
… all max-lines-per-function warnings
…cles to dedicated modules
… dedicated modules
…g index.ts under file-size cap
…coverage; bump CI thresholds to 77/67/79/77
…src; allow any in tests via eslint override
…ice/app src; allow any in tests via eslint override
…nCapturer + helpers/utils; test(selenium): cover SessionCapturer
…/controller): extract pure context-update helpers + tests
…lup branches; test(service/launcher): cover detectInvocationConfigPath + detectInvocationSpecs
…http server; test(coverage): ratchet thresholds to lock in the gains
…): dedupe SPEC_FILE_RE / TEST_FILE_PATTERN + FEATURE_FILE_RE
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.