Skip to content

[camera_android_camerax] Add AGENTS.md - #12066

Merged
auto-submit[bot] merged 13 commits into
flutter:mainfrom
camsim99:cos_agentsmd
Jul 20, 2026
Merged

[camera_android_camerax] Add AGENTS.md#12066
auto-submit[bot] merged 13 commits into
flutter:mainfrom
camsim99:cos_agentsmd

Conversation

@camsim99

@camsim99camsim99 commented Jun 30, 2026

Copy link
Copy Markdown
Contributor

Adds AGENTS.md to give camera_android_camerax-specific context and guidance on how to make high quality contributions to the package.

Pre-Review Checklist

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note: The Flutter team is currently trialing the use of Gemini Code Assist for GitHub. Comments from the gemini-code-assist bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.

Footnotes

  1. Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. 2

@camsim99camsim99 added the CICD Run CI/CD label Jul 13, 2026
@camsim99
camsim99 marked this pull request as ready for review July 13, 2026 18:38

@gemini-code-assistgemini-code-assistBot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds an AGENTS.md guide to provide AI agents with repository-specific context for the camera_android_camerax package, and updates the package version to 0.7.5. Review feedback focuses on correcting relative links to the .agents directory to ensure they resolve properly from the nested package directory, along with minor grammatical corrections.

Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
@github-actionsgithub-actionsBot removed the CICD Run CI/CD label Jul 13, 2026
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/AGENTS.md Outdated
Comment threadpackages/camera/camera_android_camerax/CHANGELOG.md Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

High level feedback this is much longer than an initial draft I thinks should be. Following the guidance in https://www.kaggle.com/whitepaper-the-new-SDLC-with-vibe-coding I think we should try to make this as short as we can since every token is an overhead in every conversation.

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Might have gone extreme in the other direction but the agent I edited with made it seem like I can get rid of all the wordiness and focus on the core operations/skills it needs to know so edited it down to this. PTAL!

camsim99and others added 3 commits July 15, 2026 11:58
Co-authored-by: Reid Baker <1063596+reidbaker@users.noreply.github.com>
@camsim99
camsim99 requested a review from reidbakerJuly 15, 2026 19:42
@camsim99camsim99 added the CICD Run CI/CD label Jul 16, 2026

@reidbakerreidbaker left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is wonderful. I don't think you went too far in the brevity direction.

@camsim99camsim99 added the autosubmit Merge PR when tree becomes green via auto submit App label Jul 20, 2026
@auto-submit

Copy link
Copy Markdown
Contributor

autosubmit label was removed for flutter/packages/12066, because - The status or check suite Dashboard Checks has failed. Please fix the issues identified (or deflake) before re-applying this label.

@auto-submitauto-submitBot removed the autosubmit Merge PR when tree becomes green via auto submit App label Jul 20, 2026
@camsim99camsim99 added override: no versioning needed Override the check requiring version bumps for most changes override: no changelog needed Override the check requiring CHANGELOG updates for most changes autosubmit Merge PR when tree becomes green via auto submit App labels Jul 20, 2026
@auto-submit
auto-submitBot merged commit f555f10 into flutter:mainJul 20, 2026
14 checks passed
pullBot pushed a commit to Klomgor/flutter that referenced this pull request Jul 21, 2026
…er#189782)
flutter/packages@611899b...8260a1e
2026-07-20 jmccandless@google.com [cupertino_ui] Publish pre-release
0.0.2 (flutter/packages#12251)
2026-07-20 43054281+camsim99@users.noreply.github.com
[camera_android_camerax] Add `AGENTS.md` (flutter/packages#12066)
2026-07-20 jmccandless@google.com [cupertino_ui] Undo unpublished
version info (flutter/packages#12252)
2026-07-20 42726313+dao-wkm@users.noreply.github.com
[video_player_android] Avoid sending unset duration on initialization
(flutter/packages#11709)
2026-07-20 stuartmorgan@google.com [webview_flutter] Convert from XCTest
to Swift Testing (flutter/packages#12243)
2026-07-20 stuartmorgan@google.com [various] Convert example boilerplate
from Java to Kotlin (flutter/packages#12201)
2026-07-20 jmccandless@google.com [material_ui, cupertino_ui] Update
changelogs for the prerelease (flutter/packages#12249)
2026-07-20 engine-flutter-autoroll@skia.org Roll Flutter from
8005793 to cab057d (2 revisions) (flutter/packages#12248)
2026-07-20 shuminlian@gmail.com [in_app_purchase_android] Add obfuscated
profile ID purchase param (flutter/packages#12080)
2026-07-20 jmccandless@google.com [material_ui] Unpin
material_color_utilities (flutter/packages#12225)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem@google.com on the revert to ensure that a
human
is aware of the problem.
To file a bug in Flutter:
https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md
Zubii12 pushed a commit to Zubii12/packages that referenced this pull request Jul 23, 2026
Adds `AGENTS.md` to give `camera_android_camerax`-specific context and guidance on how to make high quality contributions to the package.
## Pre-Review Checklist
**Note**: The Flutter team is currently trialing the use of [Gemini Code Assist for GitHub](https://developers.google.com/gemini-code-assist/docs/review-github-code). Comments from the `gemini-code-assist` bot should not be taken as authoritative feedback from the Flutter team. If you find its comments useful you can update your code accordingly, but if you are unsure or disagree with the feedback, please feel free to wait for a Flutter team member's review for guidance on which automated comments should be addressed.
[^1]: Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autosubmitMerge PR when tree becomes green via auto submit AppCICDRun CI/CDoverride: no changelog neededOverride the check requiring CHANGELOG updates for most changesoverride: no versioning neededOverride the check requiring version bumps for most changesp: camera

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@camsim99@reidbaker