Skip to content

Add AGENTS.md#317

Merged
sanmai merged 4 commits into
mainfrom
feat/agents-md
Jun 10, 2026
Merged

Add AGENTS.md#317
sanmai merged 4 commits into
mainfrom
feat/agents-md

Conversation

@sanmai

@sanmai sanmai commented Jun 10, 2026

Copy link
Copy Markdown
Owner

No description provided.

github-actions[bot]
github-actions Bot previously approved these changes Jun 10, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Consider it done!

Comment thread AGENTS.md Outdated
github-actions[bot]
github-actions Bot previously approved these changes Jun 10, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I love it when you take charge! Let's make it happen.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds an AGENTS.md guide intended to orient contributors/AI assistants with development workflows, architecture notes, and testing/quality expectations for the sanmai/pipeline library.

Changes:

  • Introduces AGENTS.md describing project purpose, core components, and key design principles.
  • Documents local development commands for testing, code style, static analysis, and validation.
  • Summarizes the repository’s testing philosophy (coverage/mutation testing) and performance considerations.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment thread AGENTS.md Outdated
Comment on lines +87 to +88
- `DocumentationTest` and `DocumentationMethodsTest` enforce that README and `docs/` stay in sync with the actual public API: every public method needs a README mention and header, and documented methods must exist

Comment thread AGENTS.md
### Core Components


1. **Main Pipeline Class**: `src/Standard.php` - see main @README.md
Comment thread AGENTS.md
- `peek()` is destructive: it eagerly consumes the first N items (as tuples, to survive duplicate keys) and leaves the rest in the pipeline.
- Callbacks for `each()`/`tap()`/`select(onReject:)` receive `($value, $key)`; an `ArgumentCountError` from arity-sensitive internal functions (e.g. `printf(...)`) is caught once and the callable permanently wrapped to single-argument form (`callWithValueKey()`).

2. **Helper Functions**: @src/functions.php

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I like your style! Let's roll with it.

@sanmai sanmai enabled auto-merge (squash) June 10, 2026 10:07
@sanmai sanmai merged commit b2bfc50 into main Jun 10, 2026
10 checks passed
@sanmai sanmai deleted the feat/agents-md branch June 10, 2026 10:07
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