Skip to content

Conversation

@BillyGoatCapital
Copy link

Summary

Related Linear tickets, Github issues, and Community forum posts

Review / Merge checklist

  • PR title and summary are descriptive. (conventions)
  • Docs updated or follow-up ticket created.
  • Tests included.
  • PR Labeled with release/backport (if the PR is an urgent fix that needs to be backported)

claude and others added 5 commits November 17, 2025 10:19
Updated both main and frontend CLAUDE.md files to reflect the current state of the n8n repository:

Main CLAUDE.md updates:
- Added Node.js >=22.16 and pnpm >=10.18.3 requirements
- Expanded package structure documentation to include all 50+ packages
- Organized packages into logical categories (Core, Frontend, Backend, Nodes, Task Runners, Development Tools, Testing, Enterprise)
- Added comprehensive development commands (dev, dev:be, dev:fe, dev:ai, build:docker, test:ci:*)
- Added code quality commands (lint:fix, format, format:check)
- Documented task runner packages for Python support
- Updated technology stack with specific version numbers
- Added backend module development section with scaffolding commands
- Fixed path reference to backend-module-guide.md
- Added dependency catalog information from pnpm-workspace.yaml
- Updated frontend package paths to reflect packages/frontend/ structure

Frontend CLAUDE.md updates:
- Added comprehensive frontend architecture overview
- Documented all 8 frontend packages and their purposes
- Added development guidelines for component location, state management, API calls
- Clarified when to use @n8n/design-system vs editor-ui
- Added guidance on using @n8n/stores, @n8n/composables, and @n8n/rest-api-client

These updates provide AI assistants with accurate, comprehensive guidance for working with the n8n codebase.
Copilot AI review requested due to automatic review settings November 23, 2025 16:21
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ BillyGoatCapital
❌ claude
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Contributor

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

This PR enhances the documentation structure for the n8n workflow automation platform by adding comprehensive AI-assisted development documentation and expanding existing documentation with detailed technical information.

Key Changes:

  • Added a new enterprise refinement analysis document for Gemini Pro AI analysis
  • Significantly expanded the main CLAUDE.md with detailed package structure, version requirements, and development guidelines
  • Added frontend architecture documentation to packages/frontend/CLAUDE.md

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
GEMINI_ENTERPRISE_REFINEMENT.md New 359-line document providing comprehensive enterprise system analysis request for Gemini Pro, covering architecture, security, scalability, and deployment considerations
CLAUDE.md Expanded with detailed package organization (40+ packages categorized), version requirements, build/test commands, and backend module development guidelines
packages/frontend/CLAUDE.md Added frontend architecture overview documenting package structure and development guidelines for Vue 3 components, state management, and API integration

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

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@n8n-assistant n8n-assistant bot added community Authored by a community member in linear Issue or PR has been created in Linear for internal review labels Nov 23, 2025
@n8n-assistant
Copy link

n8n-assistant bot commented Nov 23, 2025

Hey @BillyGoatCapital,

Thank you for your contribution. We appreciate the time and effort you’ve taken to submit this pull request.

Before we can proceed, please ensure the following:
• Tests are included for any new functionality, logic changes or bug fixes.
• The PR aligns with our contribution guidelines.

Regarding new nodes:
We no longer accept new nodes directly into the core codebase. Instead, we encourage contributors to follow our Community Node Submission Guide to publish nodes independently.

If your node integrates with an AI service that you own or represent, please email [email protected] and we will be happy to discuss the best approach.

About review timelines:
This PR has been added to our internal tracker as "GHC-5606". While we plan to review it, we are currently unable to provide an exact timeframe. Our goal is to begin reviews within a month, but this may change depending on team priorities. We will reach out when the review begins.

Thank you again for contributing to n8n.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community Authored by a community member in linear Issue or PR has been created in Linear for internal review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants