Switch to named imports without subpaths

[Jesperholmbergmsft]

While setting up some sample projects to repro & test a fix for bug 542, I realized that on older webpack I had to add babel configuration to work around subpath imports inside of the teams.ts libraries. I had to add the alias section here to my webpack.frontend.config.js just for a basic tab test app. Which is doable, but also unnecessary friction:

  resolve: {
    extensions: [".tsx", ".ts", ".js"],
    mainFields: ["main", "module", "browser"],
    alias: {
      "@microsoft/teams.common/http$": path.resolve(
        __dirname,
        "node_modules/@microsoft/teams.common/dist/http/index.js"
      ),
      "@microsoft/teams.common/logging$": path.resolve(
        __dirname,
        "node_modules/@microsoft/teams.common/dist/logging/index.js"
      ),
    },
  },

In addition, I noticed that we often use wildcard imports for readability, but on older bundlers this may impact tree shakability.

Let's clean up both of these and switch to named imports without subpaths. There are no code changes from this, just a bunch of routine imports & type alias changes.

How tested:

• Built & packed the packages locally; npm installed them into my test project; verified that I could now remove the alias section
• Build passes, linter passes, unit tests pass.
• Verified that the diffs detected by Template Sync Verification are false positives, as they didn't have the same imports in the first place. skip-test-verification

[Copilot]

Pull request overview

This PR standardizes imports across the repo by removing @microsoft/* subpath imports (e.g. @microsoft/teams.common/http, @microsoft/teams.common/logging) and replacing wildcard imports with named imports to reduce bundler friction (notably with older webpack setups) and improve tree-shakability.

Changes:

• Replaced @microsoft/teams.common/* subpath imports with @microsoft/teams.common named imports across packages, templates, and examples.
• Replaced some namespace (import * as …) usage with named imports (notably @azure/msal-browser, @microsoft/teams-js, @microsoft/teams.graph, @microsoft/teams.api).
• Updated Jest mocks to target @microsoft/teams.common instead of subpath modules.

Reviewed changes

Copilot reviewed 55 out of 55 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
packages/openai/src/chat.ts	Switch logging imports to @microsoft/teams.common root.
packages/openai/src/audio.ts	Switch logging imports to @microsoft/teams.common root.
packages/graph/src/utils/url.ts	Replace http subpath type usage with RequestConfig from @microsoft/teams.common.
packages/graph/src/types.ts	Replace http subpath type usage with RequestConfig from @microsoft/teams.common.
packages/graph/src/index.ts	Replace http subpath module import with HttpClient/HttpClientOptions from @microsoft/teams.common.
packages/graph/src/index.spec.ts	Update Jest mocking/imports to mock @microsoft/teams.common root.
packages/devtools/src/components/Logger/Logger.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
packages/dev/src/routes/context.ts	Switch ILogger import to @microsoft/teams.common root.
packages/client/src/msal-utils.ts	Replace MSAL namespace import with named imports; switch ILogger import to @microsoft/teams.common root.
packages/client/src/graph-utils.ts	Switch http/logging imports to @microsoft/teams.common root and graph client to named import.
packages/client/src/graph-utils.spec.ts	Update Jest mocking to mock @microsoft/teams.common root.
packages/client/src/app.ts	Replace MSAL/teams-js/http/logging/graph namespace imports with named imports from package roots.
packages/client/src/app.spec.ts	Update Jest mocking to mock @microsoft/teams.common root.
packages/cli/templates/typescript/tab/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
packages/cli/templates/typescript/ai/src/index.ts	Switch LocalStorage import to @microsoft/teams.common root.
packages/botbuilder/src/plugin.ts	Replace http subpath import with HttpClient named import from @microsoft/teams.common.
packages/apps/src/middleware/strip-mentions-text.ts	Replace @microsoft/teams.api namespace import with named imports and alias.
packages/apps/src/contexts/activity.ts	Switch logging/storage imports to @microsoft/teams.common root.
packages/apps/src/contexts/activity.test.ts	Switch logging/storage imports to @microsoft/teams.common root.
packages/apps/src/app.ts	Collapse events/http/logging/storage imports into a single @microsoft/teams.common named import.
packages/apps/src/app.plugin.spec.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
packages/apps/src/app.oauth.ts	Switch graph namespace import to named Client import.
packages/apps/src/api.ts	Replace local re-aliasing with direct re-exports of Client from api/graph packages.
packages/apps/src/activity-sender.ts	Replace http subpath import with HttpClient named import from @microsoft/teams.common.
packages/apps/src/activity-sender.spec.ts	Replace http subpath import with HttpClient named import from @microsoft/teams.common.
packages/api/src/clients/user/token.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/user/token.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/user/index.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/team.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/team.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/reaction/reaction.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/reaction/reaction.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/meeting.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/meeting.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/index.ts	Switch http subpath imports to @microsoft/teams.common root and rename to HttpClient.
packages/api/src/clients/conversation/members.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/conversation/member.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/conversation/index.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/conversation/activity.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/conversation/activity.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/bot/sign-in.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/bot/sign-in.spec.ts	Switch http subpath imports to @microsoft/teams.common root.
packages/api/src/clients/bot/index.ts	Switch http subpath imports to @microsoft/teams.common root.
examples/threading/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/targeted-messages/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/tab/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/reactions/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/proactive-messaging/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/message-extensions/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/meetings/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/mcp-server/src/app.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/lights/src/index.ts	Switch ConsoleLogger/LocalStorage imports to @microsoft/teams.common root.
examples/graph/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root; minor comment whitespace fix.
examples/echo/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.
examples/botbuilder/src/index.ts	Switch ConsoleLogger import to @microsoft/teams.common root.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

Comments suppressed due to low confidence (4)

packages/client/src/graph-utils.ts:1

• There’s an extra nested block scope inside buildGraphClient ({ on line 10) that doesn’t provide value and adds visual noise. Removing the redundant block will simplify the function body without changing behavior.
  packages/client/src/app.ts:1
• This import has inconsistent spacing (import␠␠{ ... }). Please normalize spacing to match typical formatting (single space after import and inside braces) to avoid churn in auto-formatters and keep style consistent.
  packages/graph/src/index.spec.ts:1
• The named import is missing a space before } (HttpClient}). Please normalize spacing to keep imports consistent and reduce formatter diffs.
  packages/dev/src/routes/context.ts:1
• ILogger is a type-only symbol in this file context. Prefer import type { ILogger } ... to ensure the import is always elided under verbatimModuleSyntax/ESM settings and to make intent explicit.

[heyitsaamir]

Ty for cleaning up!

[corinagum]

Left a few nits. Thank you!