Skip to content

Add AGENTS.md - #3445

Merged
maxkfranz merged 1 commit into
unstablefrom
feature/agents
Apr 14, 2026
Merged

Add AGENTS.md#3445
maxkfranz merged 1 commit into
unstablefrom
feature/agents

Conversation

@maxkfranz

Copy link
Copy Markdown
Member

Ref: Add AGENTS.md #3441

Cross-references to related issues. If there is no existing issue that describes your bug or feature request, then create an issue before making your pull request.

Associated issues:

Notes re. the content of the pull request. Give context to reviewers or serve as a general record of the changes made. Add a screenshot or video to demonstrate your new feature, if possible.

  • Adds AGENTS.md for use by Codex, Claude, Copilot, etc.
  • Doesn't change any API so may as well be included in a patch release.

Checklist

Author:

  • The proper base branch has been selected. New features go on unstable. Bug-fix patches can go on either unstable or master.
  • N/A -- Automated tests have been included in this pull request, if possible, for the new feature(s) or bug fix. Check this box if tests are not pragmatically possible (e.g. rendering features could include screenshots or videos instead of automated tests).
  • The associated GitHub issues are included (above).
  • Notes have been included (above).
  • For new or updated API, the index.d.ts Typescript definition file has been appropriately updated.

Reviewers:

  • All automated checks are passing (green check next to latest commit).
  • At least one reviewer has signed off on the pull request.
  • For bug fixes: Just after this pull request is merged, it should be applied to both the master branch and the unstable branch. Normally, this just requires cherry-picking the corresponding merge commit from master to unstable -- or vice versa.

@maxkfranz
maxkfranz requested review from chrtannus and d2fong April 10, 2026 17:33
@maxkfranz

Copy link
Copy Markdown
Member Author

Adding @chrtannus & @d2fong as reviewers. Let me know if you think anything should be added or amended.

@maxkfranz maxkfranz linked an issue Apr 10, 2026 that may be closed by this pull request
4 tasks
@d2fong

d2fong commented Apr 10, 2026

Copy link
Copy Markdown
Member

Looks good to me as a starting point. Because cytoscape.js is a large codebase. It may be worth exploring progressive disclosure techniques. Here is this one idea I found in this talk:

Screenshot 2026-04-10 at 3 12 34 PM

@chrtannus chrtannus left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can't think of anything else to add.

@maxkfranz

Copy link
Copy Markdown
Member Author

Looks good to me as a starting point. Because cytoscape.js is a large codebase. It may be worth exploring progressive disclosure techniques. Here is this one idea I found in this talk:

Screenshot 2026-04-10 at 3 12 34 PM

This is a really good point. I did some research on this, and I don't think it's yet safe to depend on it unless you assume everyone uses Claude or Codex. E.g. Copilot doesn't yet have full support for progressive disclosure in the same way.

We should definitely add support for this later, but right now it could be a bit hit or miss.

In any case, for now I just want something good enough as compared to what we have now for agents (nothing).

@maxkfranz

Copy link
Copy Markdown
Member Author

@copilot, please create an issue re. @d2fong's feedback re. progressive disclosure

@maxkfranz
maxkfranz merged commit d8f19f2 into unstable Apr 14, 2026
2 of 3 checks passed
Copilot stopped work on behalf of maxkfranz due to an error April 14, 2026 17:39
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.

Add AGENTS.md

3 participants