Suppress webpack warnings in example and devtools builds

[frankmueller-msft]

Summary

• shared-tree-demo: Adds performance: { hints: false } to suppress webpack's default 244 KiB asset/entrypoint size warnings (not actionable for an example app). Adds silent: Boolean(process.env.CI || process.env.TF_BUILD) to dotenv-webpack so the missing .env warning is suppressed in CI but preserved locally for developers.
• devtools-browser-extension: Same conditional silent flag on dotenv-webpack.

Together these eliminate ~4 warnings per build.

Test plan

• Build - client packages pipeline passes (build, policy checks, all test stages)
• repo-policy-check passes
• Verify CI build logs no longer show asset size or .env warnings for these packages

🤖 Generated with Claude Code

[Copilot]

Pull request overview

This PR reduces webpack noise in CI/dev builds by suppressing expected warnings in an example app and the devtools browser extension build configurations.

Changes:

• Add silent: true to dotenv-webpack usage to suppress missing .env warnings.
• Disable webpack performance size hints in shared-tree-demo to avoid default asset/entrypoint size warnings.

Reviewed changes

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

File	Description
packages/tools/devtools/devtools-browser-extension/webpack.config.cjs	Suppresses dotenv-webpack missing .env warning via silent: true.
examples/service-clients/odsp-client/shared-tree-demo/webpack.config.cjs	Suppresses dotenv-webpack missing .env warning and disables webpack performance hints.

Comments suppressed due to low confidence (1)

examples/service-clients/odsp-client/shared-tree-demo/webpack.config.cjs:71

• Setting performance.hints = false disables all webpack performance hints. If the intent is just to avoid the default ~244KiB warning for this example, consider instead raising maxAssetSize / maxEntrypointSize to an expected upper bound so truly unexpected bundle growth still produces a warning.

		// This is an example app — disable webpack's default 244 KiB asset/entrypoint size warnings.
		performance: {
			hints: false,
		},

[Copilot]

silent: true will suppress the warning that the README relies on to prompt users to copy .env.template to .env. Consider making silent conditional (e.g., only in CI) so local devs still get an explicit signal when .env is missing.

Suggested change

	silent: true,
	// Show warnings locally to prompt copying `.env.template` to `.env`,
	// but stay silent in CI environments where CI logs should be quieter.
	silent: Boolean(process.env.CI),

[frankmueller-msft]

Addressed — now uses silent: Boolean(process.env.CI || process.env.TF_BUILD) so local devs still see the warning.

[Copilot]

silent: true will also suppress the helpful warning when a developer forgets to create the local .env file. Since this config is used for local development as well as CI, consider making silent conditional (e.g., only in CI) so missing .env is still surfaced locally.

Suggested change

	silent: true,
	silent: Boolean(process.env.CI),

[frankmueller-msft]

Addressed — same conditional silent: Boolean(process.env.CI || process.env.TF_BUILD) applied here.

[alexvy86]

Seems fine to me. Left one suggestion below.

For reference, this is how logs look like without the change (devtools one further down), and in the corresponding logs for this PR run no more warnings show up in for shared-tree-demo nor devtools.

[alexvy86]

Suggested change

	// This is an example app — disable webpack's default 244 KiB asset/entrypoint size warnings.
	// This is an example app — disable webpack's default 244 KiB asset/entrypoint size warnings to keep webpack output cleaner (we don't need to address this for a sample app)