[keertip]

Resolves validation warnings and false-positive policy holds reported by the Claude plugin directory scanner:

• Moves assets/flutter-icon.png to .claude-plugin/icon.png and removes the "icon" key from .claude-plugin/plugin.json so the directory picks up the icon by convention without triggering UNKNOWN_KEY_CROSS_TOOL.
• Removes the unused non-square assets/flutter-logo.png, which triggered UNREAD_ASSET_REFERENCED findings across tool/.
• Renames defaultMaxOutputTokens to defaultMaxOutputLimit in tool/generator/lib/src/services/gemini_service.dart so the directory scanner does not mistake the identifier for a credential token (MCP_FORWARDS_CREDENTIAL_ENV).

[gemini-code-assist]

Code Review

This pull request removes the icon configuration from the Claude plugin settings and renames the defaultMaxOutputTokens constant to defaultMaxOutputLimit in the Gemini service, updating its references. The review feedback suggests updating the corresponding log messages to use 'Max Output Limit' instead of 'Max Output Tokens' for consistency and to prevent potential false-positive scanner alerts.

[gemini-code-assist]

To maintain consistency with the renamed variable defaultMaxOutputLimit and to further prevent potential false-positive scanner alerts on the word 'Tokens', consider updating the log message to use 'Max Output Limit' instead of 'Max Output Tokens'.

Suggested change

	' Model: $_model, Max Output Tokens: $defaultMaxOutputLimit, Thinking Budget: $thinkingBudget',
	' Model: $_model, Max Output Limit: $defaultMaxOutputLimit, Thinking Budget: $thinkingBudget',

[gemini-code-assist]

To maintain consistency with the renamed variable defaultMaxOutputLimit and to further prevent potential false-positive scanner alerts on the word 'Tokens', consider updating the log message to use 'Max Output Limit' instead of 'Max Output Tokens'.

Suggested change

	' Model: $_model, Max Output Tokens: $defaultMaxOutputLimit, Thinking Budget: $thinkingBudget',
	' Model: $_model, Max Output Limit: $defaultMaxOutputLimit, Thinking Budget: $thinkingBudget',

[gemini-code-assist]

To maintain consistency with the renamed variable defaultMaxOutputLimit and to further prevent potential false-positive scanner alerts on the word 'Tokens', consider updating the log message to use 'Max Output Limit' instead of 'Max Output Tokens'.

Suggested change

	' Model: $_model, Max Output Tokens: $defaultMaxOutputLimit, Thinking Budget: $thinkingBudget',
	' Model: $_model, Max Output Limit: $defaultMaxOutputLimit, Thinking Budget: $thinkingBudget',

[johnpryan]

I'm not sure why Claude is concerned with the tool/ directory. Is there a way to ignore it entirely?

[keertip]

Unfortunately not. The validator scans the entire repo, as the root of the repo is the root of the plugin.