Skip to content

flutter-di: point state reads to flutter-bloc - #21

Closed
wzslr321 wants to merge 1 commit into
mainfrom
fix/di-bloc-reads
Closed

wzslr321 wants to merge 1 commit into
mainfrom
fix/di-bloc-reads

Conversation

@wzslr321

Copy link
Copy Markdown
Member

references/dependency-injection.md told Claude to subscribe to cubits with BlocBuilder, BlocListener or BlocConsumer. That contradicts flutter-bloc, which reads state with context.watch instead of a top-level BlocBuilder (the flutter-context-watch-instead-of-top-builder skill) and handles side effects with bloc_presentation. With both plugins installed, Claude got two conflicting rules.

The DI reference now leaves state reads to flutter-bloc and names its two rules. Version 0.1.0 to 0.1.1, with a new CHANGELOG.md.

🤖 Generated with Claude Code

The DI reference told Claude to subscribe with BlocBuilder, BlocListener
or BlocConsumer, which contradicts the flutter-bloc conventions.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@github-actions

Copy link
Copy Markdown

Hi @wzslr321! 👋 You're a LeanCode member, so a quick sanity check before this ships: is this PR meant for the public marketplace?

Everything merged here becomes publicly visible and is automatically synced into ai-plugins-internal.

  • Internal-only content (client-specific guidance, internal processes, backend/PMO plugins, anything not ready to be public) belongs in ai-plugins-internal instead.
  • Generally useful, public-ready content is right at home here — the internal repo will pick it up through the sync.

If public is what you intended, just ignore this comment. 🙂

@wzslr321 wzslr321 closed this Sep 23, 2026
@wzslr321
wzslr321 deleted the fix/di-bloc-reads branch September 23, 2026 13:11
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.

1 participant