Implement support for Azure DevOps Server and GitHub Server (onpremise)#71
Implement support for Azure DevOps Server and GitHub Server (onpremise)#71Copilot wants to merge 16 commits into
Conversation
…Server Co-authored-by: norschel <12895005+norschel@users.noreply.github.com>
Co-authored-by: norschel <12895005+norschel@users.noreply.github.com>
Co-authored-by: norschel <12895005+norschel@users.noreply.github.com>
…ing or encoding Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ing or encoding Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ing or encoding Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ession for hostnames Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ssion for hostnames Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ing or encoding Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ession for hostnames Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ing or encoding Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
…ssion for hostnames Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
|
@copilot verify the implementation again with a newer and better modal |
Co-authored-by: norschel <12895005+norschel@users.noreply.github.com>
Re-verified the implementation and found two bugs that have been fixed in commit
|
|
@copilot resolve the merge conflicts in this pull request. check again the code and make improvements if needed |
# Conflicts: # package-lock.json # package.json Co-authored-by: norschel <12895005+norschel@users.noreply.github.com>
Merge conflicts resolved in
|
Adds support for self-hosted Azure DevOps Server and GitHub Server (onpremise) by detecting the organization/project and owner/repo from custom remote hostnames.
Changes Made
azd.ts:getAzureDevOpsHostname()returns the raw hostname; escaping is applied withescapeStringRegexponly at the regex construction sites (fixing the previous double-escaping and the always-false SSH hostname comparison).azd.ts: The SSH remote URL pattern is now matched before the HTTPS pattern. Because the SSH host (ssh.{hostname}) contains the HTTPS hostname as a substring, matching HTTPS first incorrectly captured thev3segment as the organization forgit@ssh.{hostname}:v3/org/projectURLs.gitHub.ts: UsesescapeStringRegexpfor consistent, complete escaping of all regex special characters in hostnames (replacing manual.replace(...)escaping).docs/regex-test.js: Demo script updated to use a single correct regex-escape helper and SSH-first matching.escape-string-regexpis pinned to^4.0.0(CommonJS) so thetscbuild succeeds — the previously used v5 is ESM-only and broke compilation (TS1479);esbuildwas moved todevDependencies.Merge conflict resolution
origin/masterinto the branch, keeping master's newer dependency versions inpackage.json/package-lock.json.azDevOpsPullrequestFunctions.ts, which the merge had mangled by injecting a stray localgetAzureDevOpsConnectiondefinition into the middle ofStateMultipleAzDPrsInStream.Testing
npm run compile(tsc) passesnpm run lintpassesdocs/regex-test.jsdemo confirms HTTPS and SSH remote URLs resolve the correct org/project and owner/repo for both default and custom hostnames💬 Share your feedback on Copilot coding agent for the chance to win a $200 gift card! Click here to start the survey.