Skip to content

docs: add build/test commands and architecture overview to CLAUDE.md - #2760

Closed
9Pie-893 wants to merge 1 commit into
NVIDIA:mainfrom
9Pie-893:claude/review-complete-2fix1e
Closed

docs: add build/test commands and architecture overview to CLAUDE.md#2760
9Pie-893 wants to merge 1 commit into
NVIDIA:mainfrom
9Pie-893:claude/review-complete-2fix1e

Conversation

@9Pie-893

Copy link
Copy Markdown

Expand CLAUDE.md beyond the existing AGENTS.md import with a Commands section (build, test, single-test invocations, e2e, lint/format, Go SDK CI) sourced from tasks/.toml, and an Architecture section synthesizing the control-plane/data-plane split, egress policy enforcement path, gateway RPC authorization/interceptors, OCSF logging, and proto/SDK relationships from architecture/.md.

Summary

  • 1-3 sentences: what this PR does and why

Related Issue

  • Required for features, user-visible behavior changes, public API changes,
    architecture changes, and multi-PR efforts: Fixes #NNN or Closes #NNN.
    For an exempt small docs fix, mechanical change, or obvious localized bug fix:
    No issue required:

Changes

  • Bullet list of key changes -->

Testing

  • Report current verification performed for this implementation. Do not copy diagnostics from the original issue. -->
  • mise run pre-commit passes
  • Unit tests added/updated
  • E2E tests added/updated (if applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)
  • Architecture docs updated (if applicable)

Expand CLAUDE.md beyond the existing AGENTS.md import with a Commands
section (build, test, single-test invocations, e2e, lint/format,
Go SDK CI) sourced from tasks/*.toml, and an Architecture section
synthesizing the control-plane/data-plane split, egress policy
enforcement path, gateway RPC authorization/interceptors, OCSF
logging, and proto/SDK relationships from architecture/*.md.

Signed-off-by: Claude <noreply@anthropic.com>
@copy-pr-bot

copy-pr-bot Bot commented Aug 15, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

Thank you for your interest in contributing to OpenShell, @9Pie-893.

This project uses a vouch system for first-time contributors. Before submitting a pull request, you need to be vouched by a maintainer.

To get vouched:

  1. Open a Vouch Request discussion.
  2. Describe what you want to change and why.
  3. Write in your own words — do not have an AI generate the request.
  4. A maintainer will comment /vouch if approved.
  5. Once vouched, open a new PR (preferred) or reopen this one after a few minutes.

See CONTRIBUTING.md for details.

@github-actions github-actions Bot closed this Aug 15, 2026
@github-actions

Copy link
Copy Markdown

Thank you for your submission! We ask that you sign our Developer Certificate of Origin before we can accept your contribution. You can sign the DCO by adding a comment below using this text:


I have read the DCO document and I hereby sign the DCO.


You can retrigger this bot by commenting recheck in this Pull Request. Posted by the DCO Assistant Lite bot.

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