[ai] bootstrap ABOUT.md recursive summaries and support for context priming by pq · Pull Request #9115 · flutter/flutter-intellij

pq · 2026-09-07T23:40:44Z

See flutter/dart-intellij-third-party#637 for context.

Review the contribution guidelines below:

I’ve reviewed the contributor guide and applied the relevant portions to this PR.
I've included the required information in the description above.
My up-to-date information is in the AUTHORS file.
I've updated CHANGELOG.md if appropriate.

Contribution guidelines:

See
our contributor guide and
the Flutter organization contributor guide
for general expectations for PRs.
Larger or significant changes should be discussed in an issue before creating a PR.
Dart contributions to our repos should follow the Dart style guide and use
dart format.
Java and Kotlin contributions should strive to follow Java and Kotlin best
practices (discussion).

gemini-code-assist

Code Review

This pull request introduces a new agent skill, load-repo-context, and adds hierarchical ABOUT.md architecture manifests across various directories to map out subsystems, invariants, and side effects. The review feedback focuses on improving the usability of these manifests by suggesting that child manifest paths be formatted as clickable Markdown links to ensure consistency and facilitate easier navigation.

gemini-code-assist · 2026-09-07T23:42:14Z

+- `./icons/ABOUT.md`
+- `./io/ABOUT.md`
+- `./org/ABOUT.md`


[CONCERN] Format the child manifest paths as clickable Markdown links to maintain consistency with other ABOUT.md files in the repository (such as the root ABOUT.md and tool/ABOUT.md) and to facilitate easier navigation for both developers and AI agents.

Suggested change

- `./icons/ABOUT.md`

- `./io/ABOUT.md`

- `./org/ABOUT.md`

- [./icons/ABOUT.md](./icons/ABOUT.md)

- [./io/ABOUT.md](./io/ABOUT.md)

- [./org/ABOUT.md](./org/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+Structural package containing core input/output and Flutter integrations.
+
+## Interface
+- `./flutter/ABOUT.md`


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./flutter/ABOUT.md`

- [./flutter/ABOUT.md](./flutter/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+- `./actions/ABOUT.md`
+- `./analytics/ABOUT.md`
+- `./android/ABOUT.md`
+- `./console/ABOUT.md`
+- `./dart/ABOUT.md`
+- `./deeplinks/ABOUT.md`
+- `./devtools/ABOUT.md`
+- `./editor/ABOUT.md`
+- `./font/ABOUT.md`
+- `./inspections/ABOUT.md`
+- `./jxbrowser/ABOUT.md`
+- `./logging/ABOUT.md`
+- `./module/ABOUT.md`
+- `./project/ABOUT.md`
+- `./propertyeditor/ABOUT.md`
+- `./pub/ABOUT.md`
+- `./refactoring/ABOUT.md`
+- `./run/ABOUT.md`
+- `./samples/ABOUT.md`
+- `./sdk/ABOUT.md`
+- `./settings/ABOUT.md`
+- `./survey/ABOUT.md`
+- `./template/ABOUT.md`
+- `./test/ABOUT.md`
+- `./toolwindow/ABOUT.md`
+- `./utils/ABOUT.md`
+- `./view/ABOUT.md`
+- `./vmService/ABOUT.md`
+- `./widgetpreview/ABOUT.md`


[CONCERN] Format the child manifest paths as clickable Markdown links to maintain consistency with other ABOUT.md files in the repository.

- [./actions/ABOUT.md](./actions/ABOUT.md) - [./analytics/ABOUT.md](./analytics/ABOUT.md) - [./android/ABOUT.md](./android/ABOUT.md) - [./console/ABOUT.md](./console/ABOUT.md) - [./dart/ABOUT.md](./dart/ABOUT.md) - [./deeplinks/ABOUT.md](./deeplinks/ABOUT.md) - [./devtools/ABOUT.md](./devtools/ABOUT.md) - [./editor/ABOUT.md](./editor/ABOUT.md) - [./font/ABOUT.md](./font/ABOUT.md) - [./inspections/ABOUT.md](./inspections/ABOUT.md) - [./jxbrowser/ABOUT.md](./jxbrowser/ABOUT.md) - [./logging/ABOUT.md](./logging/ABOUT.md) - [./module/ABOUT.md](./module/ABOUT.md) - [./project/ABOUT.md](./project/ABOUT.md) - [./propertyeditor/ABOUT.md](./propertyeditor/ABOUT.md) - [./pub/ABOUT.md](./pub/ABOUT.md) - [./refactoring/ABOUT.md](./refactoring/ABOUT.md) - [./run/ABOUT.md](./run/ABOUT.md) - [./samples/ABOUT.md](./samples/ABOUT.md) - [./sdk/ABOUT.md](./sdk/ABOUT.md) - [./settings/ABOUT.md](./settings/ABOUT.md) - [./survey/ABOUT.md](./survey/ABOUT.md) - [./template/ABOUT.md](./template/ABOUT.md) - [./test/ABOUT.md](./test/ABOUT.md) - [./toolwindow/ABOUT.md](./toolwindow/ABOUT.md) - [./utils/ABOUT.md](./utils/ABOUT.md) - [./view/ABOUT.md](./view/ABOUT.md) - [./vmService/ABOUT.md](./vmService/ABOUT.md) - [./widgetpreview/ABOUT.md](./widgetpreview/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+- `./common/ABOUT.md`
+- `./coverage/ABOUT.md`
+- `./daemon/ABOUT.md`
+- `./test/ABOUT.md`


[CONCERN] Format the child manifest paths as clickable Markdown links to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./common/ABOUT.md`

- `./coverage/ABOUT.md`

- `./daemon/ABOUT.md`

- `./test/ABOUT.md`

- [./common/ABOUT.md](./common/ABOUT.md)

- [./coverage/ABOUT.md](./coverage/ABOUT.md)

- [./daemon/ABOUT.md](./daemon/ABOUT.md)

- [./test/ABOUT.md](./test/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+- `VMServiceManager`
+- `VmServiceConsumers`
+- `VmServiceWrapper`
+- `./frame/ABOUT.md` - Subpackage for stack frame evaluation and variable inspection


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./frame/ABOUT.md` - Subpackage for stack frame evaluation and variable inspection

- [./frame/ABOUT.md](./frame/ABOUT.md) - Subpackage for stack frame evaluation and variable inspection

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+Structural package for Dart language integration and analysis.
+
+## Interface
+- `./analysis/ABOUT.md`


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./analysis/ABOUT.md`

- [./analysis/ABOUT.md](./analysis/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+Structural package for Dart code analysis tools.
+
+## Interface
+- `./server/ABOUT.md`


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./server/ABOUT.md`

- [./server/ABOUT.md](./server/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:14Z

+Structural package for the Dart Analysis Server integration.
+
+## Interface
+- `./protocol/ABOUT.md`


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./protocol/ABOUT.md`

- [./protocol/ABOUT.md](./protocol/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:15Z

+Structural package containing JetBrains-specific IDE integrations.
+
+## Interface
+- `./android/ABOUT.md`


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./android/ABOUT.md`

- [./android/ABOUT.md](./android/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

gemini-code-assist · 2026-09-07T23:42:15Z

+Structural package containing JetBrains Android integrations.
+
+## Interface
+- `./facet/ABOUT.md`


[CONCERN] Format the child manifest path as a clickable Markdown link to maintain consistency with other ABOUT.md files in the repository.

Suggested change

- `./facet/ABOUT.md`

- [./facet/ABOUT.md](./facet/ABOUT.md)

References

Prefix every comment with a severity category such as [CONCERN] for maintainability issues. ^(link)

pq · 2026-09-21T13:59:51Z

Ready for review!

pq · 2026-09-21T14:14:09Z

Build greenness gated by: #9132

pq · 2026-09-21T15:48:40Z

RE: gemini review feedback: this format worked well in my evals so I'm hesitant to make any changes. That said, I'll loop back and test the proposed refinements in my next round of evalutaions.

helin24 · 2026-09-21T17:26:26Z

I don't have an opinion on whether these should be clickable. (or on the Dart repo comment of whether to split up the interface notes)

[ai] bootstrap ABOUT.md recursive summaries

8e44f53

pq requested a review from helin24 as a code owner September 7, 2026 23:40

pq marked this pull request as draft September 7, 2026 23:40

gemini-code-assist Bot reviewed Sep 7, 2026

View reviewed changes

pq marked this pull request as ready for review September 21, 2026 13:59

Merge branch 'main' into egm_bootstrap

334fac9

Merge branch 'main' into egm_bootstrap

e82da30

helin24 approved these changes Sep 21, 2026

View reviewed changes

pq merged commit 79f66e0 into flutter:main Sep 21, 2026
14 checks passed

pq deleted the egm_bootstrap branch September 21, 2026 18:18

	- `./flutter/ABOUT.md`
	- [./flutter/ABOUT.md](./flutter/ABOUT.md)

	- `./frame/ABOUT.md` - Subpackage for stack frame evaluation and variable inspection
	- [./frame/ABOUT.md](./frame/ABOUT.md) - Subpackage for stack frame evaluation and variable inspection

	- `./analysis/ABOUT.md`
	- [./analysis/ABOUT.md](./analysis/ABOUT.md)

	- `./server/ABOUT.md`
	- [./server/ABOUT.md](./server/ABOUT.md)

	- `./protocol/ABOUT.md`
	- [./protocol/ABOUT.md](./protocol/ABOUT.md)

	- `./android/ABOUT.md`
	- [./android/ABOUT.md](./android/ABOUT.md)

Conversation

pq commented Sep 7, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

gemini-code-assist Bot Sep 7, 2026

Choose a reason for hiding this comment

Uh oh!

pq commented Sep 21, 2026

Uh oh!

pq commented Sep 21, 2026

Uh oh!

pq commented Sep 21, 2026

Uh oh!

helin24 commented Sep 21, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

pq commented Sep 7, 2026 •

edited

Loading