Skip to content

feat: add specialized review, documentation, and CLI skills plus known adopters resource#181

Merged
reidbaker merged 2 commits into
flutter:mainfrom
reidbaker:feat/update-agent-review-skills
Jul 16, 2026
Merged

feat: add specialized review, documentation, and CLI skills plus known adopters resource#181
reidbaker merged 2 commits into
flutter:mainfrom
reidbaker:feat/update-agent-review-skills

Conversation

@reidbaker-agent

@reidbaker-agent reidbaker-agent commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

feat: Add Specialized Review, Documentation, and CLI Skills

Context & Goal

This PR augments our core agent skillset (reidbaker-agent) with specialized instruction profiles covering canonical API reviews, structured multi-language code documentation, precise test review execution, and reliable UNIX CLI best practices. Additionally, it provides the updated known adopters directory within dart-skills-lint-integration.

Source Repository: All expert reference skills adapted in this PR (api-review, code-documentation, unix-cli-best-practices, plus reference refinements) originate from gspencergoog/skills.


Key Improvements & Added Skills

  1. api-review: Comprehensive guidelines and canonical interface specifications (canonical_api_design.md) for evaluating and architecting robust APIs.
  2. code-documentation: Idiomatic documentation conventions and structural guidelines covering Dart, Python, and TypeScript (dart.md, python.md, typescript.md).
  3. code-review/references/reviewing_tests.md: Specialized verification checklist guaranteeing test structure, assertions, and isolation adhere to project testing standards during review workflows.
  4. unix-cli-best-practices: Best practices guide (SKILL.md) covering robust terminal operations, flag structures, and command-line execution conventions.
  5. known_adopters.md: Refined adopter index tracking projects employing dart_skills_lint within dart-skills-lint-integration/resources/.

Verification

  • Ran complete dart_skills_lint validation across all newly authored skills (.agents/agents/reidbaker-agent/skills/ and dart-skills-lint-integration). All skills verified (Skill is valid.).
  • Confirmed completely clean formatting and zero static analysis issues (dart format ., dart analyze --fatal-infos).

@gemini-code-assist gemini-code-assist Bot 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 several documentation files and skill guides, including API design principles, code documentation standards for Dart, Python, and TypeScript, Unix CLI best practices, and a list of adopters for the dart_skills_lint tool. The review identified an inaccuracy in the description of an awk command within the Unix CLI best practices guide, which has been addressed with a suggestion to clarify that the command prints unique lines rather than duplicates.

```bash
ls -l | awk '{print $1, $3}'
```
* Find and print duplicate lines in a file without sorting them first:

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.

medium

The description for the awk command on line 120 is incorrect. The command awk '!seen[$0]++' prints the first occurrence of each unique line, effectively removing duplicates while preserving order. It does not print the duplicate lines as the description claims.

To print only the duplicate lines (i.e., the second and subsequent occurrences), the command would be awk 'seen[$0]++' filename.txt.

I suggest updating the description to accurately reflect what the command does.

Suggested change
* Find and print duplicate lines in a file without sorting them first:
* Find and print unique lines in a file (first occurrence of each line), preserving order:

@reidbaker
reidbaker merged commit acb658a into flutter:main Jul 16, 2026
7 checks passed
@reidbaker
reidbaker deleted the feat/update-agent-review-skills branch July 16, 2026 19:41
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.

2 participants