Skip to content

[ZEPPELIN-6405] Add AGENTS.md for AI coding agent guidance#5187

Open
jongyoul wants to merge 5 commits intoapache:masterfrom
jongyoul:ZEPPELIN-6405-agents-md
Open

[ZEPPELIN-6405] Add AGENTS.md for AI coding agent guidance#5187
jongyoul wants to merge 5 commits intoapache:masterfrom
jongyoul:ZEPPELIN-6405-agents-md

Conversation

@jongyoul
Copy link
Copy Markdown
Member

Summary

  • Add comprehensive AGENTS.md following the open standard to help AI coding agents understand and work effectively with the Zeppelin codebase
  • Covers module architecture, server-interpreter Thrift IPC communication, plugin system with custom classloading, reflection patterns, interpreter lifecycle, and contributing guide
  • Build/test instructions kept concise; focus on deep architectural context

Test plan

  • Verify AGENTS.md renders correctly on GitHub
  • Verify RAT license check passes (./mvnw clean org.apache.rat:apache-rat-plugin:check -Prat)

jongyoul and others added 4 commits March 18, 2026 10:52
Add comprehensive AGENTS.md following the open standard to help AI coding
agents understand and work effectively with the Zeppelin codebase. Includes
module architecture, server-interpreter Thrift IPC communication details,
plugin system with custom classloading, reflection patterns, interpreter
lifecycle, and contributing guide.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
… exclusion

Add the standard ASF license header to AGENTS.md instead of excluding
it from the RAT license check.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…s, and config map; add AI agent dirs to .gitignore

AGENTS.md additions:
- Build Gotchas: shaded JAR rebuild chain, module build order
- Configuration Files: conf/ file roles and template relationship
- Module Boundaries: where new code should go
- Thrift Code Generation: genthrift.sh workflow, generated files in git
- REST API Pattern: AbstractRestApi, JsonResponse, @ZeppelinApi conventions

.gitignore additions:
- AI coding agent personal config directories (CLAUDE.md, GEMINI.md,
  .claude/, .gemini/, .codex/, .cursor/, .windsurf/, .cline/,
  .continue/, .aider*, .augment/, .amazonq/, .junie/, .goose/, .roo/)
- AGENTS.md remains shared and tracked in git

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jongyoul jongyoul marked this pull request as ready for review April 4, 2026 05:46
Copilot AI review requested due to automatic review settings April 4, 2026 05:46
Copy link
Copy Markdown

Copilot AI left a comment

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 (per the agents-md standard) to document Apache Zeppelin’s architecture and contributor workflows for AI coding agents, plus updates .gitignore to exclude agent-specific local config files.

Changes:

  • Introduces AGENTS.md with build/test commands and deep architectural context (modules, Thrift IPC, plugin/classloading, interpreter lifecycle).
  • Adds .gitignore entries for various AI-agent local config files/directories to keep them untracked.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated 8 comments.

File Description
AGENTS.md New AI-agent guidance doc covering build/test workflow and Zeppelin architecture details.
.gitignore Ignores common AI-agent local configuration files/directories.

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

- Add --am flag to module-scoped Maven commands for clean checkout support
- Fix JUnit version: JUnit 4 → JUnit 5 (Jupiter) with legacy note
- Add git worktree recommendation in Development Workflow section

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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