Improve local CI validation

[andystaples]

Summary

• Add Nox sessions that mirror CI linting, Pyright, unit-test matrices, Azure Managed emulator tests, and Functions E2E tests; invoke them from GitHub Actions.
• Automatically provision isolated DTS and Azurite-backed validation environments, avoiding shared-emulator conflicts.
• Document nox -R focused validation with file paths and test selectors; forward selectors to lint and type-check sessions.
• Surface Functions host-log paths for local E2E diagnosis.

Validation

• nox -s lint
• nox -s typecheck_core typecheck_functions
• nox -s core_tests-3.10 core_tests-3.11 core_tests-3.13 core_tests-3.14
• nox -s functions_unit-3.13 functions_unit-3.14
• nox -s azuremanaged_tests-3.10 (84 passed, 1 skipped)
• Focused nox -R lint, Pyright, and pytest selector forwarding

[Copilot]

Pull request overview

This PR introduces a Nox-based local validation workflow intended to mirror the repository’s GitHub Actions checks, and updates CI workflows/documentation to use those Nox sessions for consistent lint/typecheck/test execution.

Changes:

• Added/expanded Nox sessions for linting, strict Pyright checks, core/azuremanaged/functions test matrices, and a ci aggregate session.
• Updated GitHub Actions workflows to invoke Nox sessions instead of duplicating install/test logic.
• Documented the new Nox-driven validation workflow (including nox -R usage) in docs/development.md.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
noxfile.py	Adds CI-mirroring Nox sessions and emulator provisioning helpers; introduces a ci meta-session.
docs/development.md	Documents running CI-equivalent validation locally via Nox, including -R reuse guidance and examples.
.github/workflows/typecheck.yml	Switches Pyright CI jobs to run via nox -s typecheck_core/typecheck_functions.
.github/workflows/durabletask.yml	Replaces lint/test steps with nox -s lint and nox -s core_tests-<py>.
.github/workflows/durabletask-azuremanaged.yml	Replaces lint/docker test orchestration with nox -s lint and nox -s azuremanaged_tests-<py>.
.github/workflows/durabletask-azurefunctions.yml	Replaces lint/unit/e2e orchestration pieces with Nox-driven sessions.

Comments suppressed due to low confidence (1)

noxfile.py:318

• azuremanaged_tests ignores session.posargs, so nox -R -s azuremanaged_tests-3.10 -- <paths/selectors> (as documented) can't forward focused pytest selection to the underlying pytest invocation.

        session.run("pytest", "tests/durabletask-azuremanaged", "-m", "dts", "--verbose")

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (1)

noxfile.py:298

• This pytest invocation does not forward session.posargs, so commands like nox -R -s core_tests-3.10 -- tests/... -k ... (as documented) are ignored. Append *session.posargs to let file paths / selectors flow through.

        targets = session.posargs or ("tests/durabletask",)

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (2)

noxfile.py:121

• subprocess.Popen is not subscriptable at runtime, so using subprocess.Popen[str] in a return annotation will raise TypeError when importing noxfile.py (breaking all Nox sessions unless annotations are postponed). Use the non-parameterized type here (or add from __future__ import annotations and keep the generic).

    ports: Sequence[int],

noxfile.py:163

• Same runtime issue as above: subprocess.Popen[str] will raise TypeError on import because Popen isn't subscriptable without postponed evaluation. Use subprocess.Popen (unparameterized) or postpone annotations.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

noxfile.py:376

• The E2E session starts Azurite without specifying --location, which means Azurite will persist its metadata/DB files into the current working directory. That can leave untracked __azurite_db__* artifacts in the repo after local runs.

Use a session temp directory (or another dedicated folder) via --location to keep the repo clean.

        arguments=(
            "--silent",
            "--skipApiVersionCheck",
            "--blobPort",
            "10000",
            "--queuePort",
            "10001",
            "--tablePort",
            "10002",
        ),

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

noxfile.py:142

• _start_azurite always resolves the azurite executable. For blob-only sessions like core_tests, that can still bind (and potentially conflict on) the default queue/table ports even though the session only needs blob storage. Prefer azurite-blob when only blob is required (it supports --location as used elsewhere in the repo) and fall back to azurite otherwise.

    executable = shutil.which("azurite")
    if executable is None:
        session.error(
            "Azurite is required for this session. Install it with "
            "`npm install -g azurite` and run the session again.")

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

noxfile.py:136

• This error message assumes the listeners are Azurite, but _is_port_open only checks whether some process is accepting connections. Consider wording this in terms of ports being in use so the message stays accurate even if another service is bound to the port(s).

        session.error(
            "Azurite has only some required ports open "
            f"({', '.join(str(port) for port in open_ports)}). Stop the "
            "existing instance or start one that exposes all required ports "
            f"({', '.join(str(port) for port in ports)}).")

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comments suppressed due to low confidence (1)

docs/development.md:28

• The docs suggest running nox -s core_tests / nox -s azuremanaged_tests for full-matrix validation, but those session names don’t exist because these sessions are parameterized by Python version (e.g., core_tests-3.10, azuremanaged_tests-3.10). Update the examples to use the version-suffixed session names (and optionally mention repeating for other versions).

Run the versioned test sessions directly when a change needs complete matrix
coverage, for example `nox -s core_tests` or `nox -s azuremanaged_tests`.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.