Skip to content

feat: add typespec-ts and typespec-java emitters to benchmark - #4907

Draft
timotheeguerin with Copilot wants to merge 4 commits into
mainfrom
copilot/update-benchmark-include-typespec-emitters
Draft

feat: add typespec-ts and typespec-java emitters to benchmark#4907
timotheeguerin with Copilot wants to merge 4 commits into
mainfrom
copilot/update-benchmark-include-typespec-emitters

Conversation

Copilot AI commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

The benchmark only covered autorest, openapi3, python, and http-client-js emitters. This adds typespec-ts and typespec-java so emit performance is tracked for all major Azure SDK emitters.

Changes

  • packages/benchmark/package.json — add @azure-tools/typespec-ts and @azure-tools/typespec-java as workspace dependencies
  • All three benchmark tspconfig.yaml files — add both emitters with minimal options:
    • typespec-ts: package-details.name
    • typespec-java: namespace, generate-samples: false, generate-tests: false (skip sample/test gen to keep focus on core emit perf)

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
2 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

@timotheeguerin

Copy link
Copy Markdown
Member

@copilot pull upstream main and resolve conflicts, fix spellcheck too

Copilot AI and others added 2 commits July 28, 2026 19:40
…ark-include-typespec-emitters

# Conflicts:
#	pnpm-lock.yaml

Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>
Co-authored-by: timotheeguerin <1031227+timotheeguerin@users.noreply.github.com>

Copilot AI commented Jul 28, 2026

Copy link
Copy Markdown
Contributor Author

@copilot pull upstream main and resolve conflicts, fix spellcheck too

Done — merged main (resolved the pnpm-lock.yaml conflict by regenerating the lockfile) and added msinternal to the cspell.yaml words list to fix the spellcheck failures. Commit: fix: resolve merge conflict with main and fix spellcheck (msinternal)

Comment thread pnpm-lock.yaml

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot reset the lock file to main version and rerun pnpm install, it seems to be in a bad state

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reset to main's lockfile and regenerated — the lockfile is now clean. The only diff vs main is the two new workspace links for @azure-tools/typespec-ts and @azure-tools/typespec-java in the benchmark importer.

@github-actions

Copy link
Copy Markdown
Contributor

No changes needing a change description found.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants