Skip to content

chore: add AGENTS.md with project conventions and verification steps - #30

Merged
Nivmizz7 merged 1 commit into
mainfrom
chore/add-agents-md
Jul 9, 2026
Merged

chore: add AGENTS.md with project conventions and verification steps#30
Nivmizz7 merged 1 commit into
mainfrom
chore/add-agents-md

Conversation

@DysektAI

@DysektAI DysektAI commented Jul 9, 2026

Copy link
Copy Markdown
Member

User description

Summary

  • Adds AGENTS.md documenting the project architecture, handler/env conventions, verification commands, and git/PR workflow.
  • Helps future agent (and human) sessions ramp up quickly without re-discovering conventions.

Test plan

  • npm test passes
  • No code changes — documentation only

CodeAnt-AI Description

Add repository guidance for future contributors and AI agents

What Changed

  • Added a single reference file that explains the bot’s structure, main entry points, and where key code lives
  • Documented project rules for environment variables, handler setup, error handling, and where shared settings belong
  • Added the expected checks to run before finishing work, plus the branch and commit style to use

Impact

✅ Easier onboarding for new contributors
✅ Fewer missed setup steps
✅ More consistent code changes and reviews

💡 Usage Guide

Checking Your Pull Request

Every time you make a pull request, our system automatically looks through it. We check for security issues, mistakes in how you're setting up your infrastructure, and common code problems. We do this to make sure your changes are solid and won't cause any trouble later.

Talking to CodeAnt AI

Got a question or need a hand with something in your pull request? You can easily get in touch with CodeAnt AI right here. Just type the following in a comment on your pull request, and replace "Your question here" with whatever you want to ask:

@codeant-ai ask: Your question here

This lets you have a chat with CodeAnt AI about your pull request, making it easier to understand and improve your code.

Example

@codeant-ai ask: Can you suggest a safer alternative to storing this secret?

Preserve Org Learnings with CodeAnt

You can record team preferences so CodeAnt AI applies them in future reviews. Reply directly to the specific CodeAnt AI suggestion (in the same thread) and replace "Your feedback here" with your input:

@codeant-ai: Your feedback here

This helps CodeAnt AI learn and adapt to your team's coding style and standards.

Example

@codeant-ai: Do not flag unused imports.

Retrigger review

Ask CodeAnt AI to review the PR again, by typing:

@codeant-ai: review

Check Your Repository Health

To analyze the health of your code repository, visit our dashboard at https://app.codeant.ai. This tool helps you identify potential issues and areas for improvement in your codebase, ensuring your repository maintains high standards of code health.

Documents architecture, handler/env conventions, verification commands,
and git/PR workflow so future agent sessions can ramp up quickly.
@coderabbitai

coderabbitai Bot commented Jul 9, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Adds a new AGENTS.md file documenting repository guidance for AI agents, including project overview, architecture, environment-variable validation conventions, handler registration, error-handling expectations, verification steps, and Git/PR workflow practices.

Changes

AGENTS.md Documentation

Layer / File(s) Summary
Add AGENTS.md guidance document
AGENTS.md
New documentation file describing project purpose, architecture, conventions for handlers and env-var validation, error handling, verification steps, and Git/PR workflow.

Estimated code review effort: 1 (Trivial) | ~2 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly summarizes the main documentation change and its purpose.
Description check ✅ Passed The description is directly related to the AGENTS.md documentation update.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/add-agents-md

Comment @coderabbitai help to get the list of available commands.

@codeant-ai codeant-ai Bot added the size:M This PR changes 30-99 lines, ignoring generated files label Jul 9, 2026
@kilo-code-bot

This comment has been minimized.

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@AGENTS.md`:
- Around line 16-28: The architecture section uses a fenced code block without a
language tag, which trips markdownlint. Update the fenced block in AGENTS.md to
include a language identifier such as text while keeping the existing content
and formatting intact. Use the surrounding architecture listing to locate the
block and ensure all fenced blocks in that section are properly tagged.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 25933c1b-5bfe-42aa-b08c-ce4d9e02b952

📥 Commits

Reviewing files that changed from the base of the PR and between 433ba5d and 935aa16.

📒 Files selected for processing (1)
  • AGENTS.md

Comment thread AGENTS.md
Comment on lines +16 to +28
```
bot.js # Entry point: ensureEnvVars -> createClient -> register handlers -> login
webserver.js # Express issue intake server
src/
client/ # Discord client setup (intents, partials)
commands/ # Slash command registration + command definitions
config/ # env.js (required-var validation), constants.js (role/config derivation)
handlers/ # Event-driven handlers registered via registerXxxHandlers(client)
interactions/ # Slash + button interaction logic
utils/ # Shared helpers (embeds)
web/ # Static issue/data report forms
test/ # Node --test files
```

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🔵 Trivial | 💤 Low value

Label the architecture fence.

This block should include a language tag (for example, text) to satisfy markdownlint and keep the docs build clean.

🧰 Tools
🪛 markdownlint-cli2 (0.22.1)

[warning] 16-16: Fenced code blocks should have a language specified

(MD040, fenced-code-language)

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@AGENTS.md` around lines 16 - 28, The architecture section uses a fenced code
block without a language tag, which trips markdownlint. Update the fenced block
in AGENTS.md to include a language identifier such as text while keeping the
existing content and formatting intact. Use the surrounding architecture listing
to locate the block and ensure all fenced blocks in that section are properly
tagged.

Source: Linters/SAST tools

@DysektAI DysektAI mentioned this pull request Jul 9, 2026
3 tasks
@DysektAI

DysektAI commented Jul 9, 2026

Copy link
Copy Markdown
Member Author

Hey @Nivmizz7 — this housekeeping PR (AGENTS.md) is ready to merge: CI green, documentation-only, no code changes. Safe to merge anytime. #31 (honeypot) is also approved and ready — see comment there for post-merge verification steps.

@Nivmizz7

Nivmizz7 commented Jul 9, 2026

Copy link
Copy Markdown
Member

LGTM

@Nivmizz7
Nivmizz7 merged commit 5d835f7 into main Jul 9, 2026
10 of 11 checks passed
@DysektAI
DysektAI deleted the chore/add-agents-md branch July 9, 2026 13:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants