Improved create-twenty-app documentation for AI coding agents#20325
Open
Bredo wants to merge 3 commits intotwentyhq:mainfrom
Open
Improved create-twenty-app documentation for AI coding agents#20325Bredo wants to merge 3 commits intotwentyhq:mainfrom
Bredo wants to merge 3 commits intotwentyhq:mainfrom
Conversation
Contributor
Welcome!
Hello there, congrats on your first PR! We're excited to have you contributing to this project. |
Contributor
There was a problem hiding this comment.
2 issues found across 3 files
Prompt for AI agents (unresolved issues)
Check if these issues are valid — if so, understand the root cause of each and fix them. If appropriate, use sub-agents to investigate and fix each issue separately.
<file name="packages/create-twenty-app/src/constants/template/AGENT.md">
<violation number="1" location="packages/create-twenty-app/src/constants/template/AGENT.md:31">
P2: Logic functions are documented with the wrong filename pattern; the generator uses `*.function.ts`, not plain `*.ts`.</violation>
<violation number="2" location="packages/create-twenty-app/src/constants/template/AGENT.md:37">
P3: Navigation menu items are documented with an incomplete filename pattern; the actual convention includes the `.navigation-menu-item.ts` suffix.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
| | -------------------- | ------------------------------------ | ------------------------------------- | | ||
| | Object | `yarn twenty add object` | `src/objects/<name>.ts` | | ||
| | Field | `yarn twenty add field` | `src/fields/<name>.ts` | | ||
| | Logic function | `yarn twenty add logicFunction` | `src/logic-functions/<name>.ts` | |
Contributor
There was a problem hiding this comment.
P2: Logic functions are documented with the wrong filename pattern; the generator uses *.function.ts, not plain *.ts.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/create-twenty-app/src/constants/template/AGENT.md, line 31:
<comment>Logic functions are documented with the wrong filename pattern; the generator uses `*.function.ts`, not plain `*.ts`.</comment>
<file context>
@@ -12,3 +19,22 @@
+| -------------------- | ------------------------------------ | ------------------------------------- |
+| Object | `yarn twenty add object` | `src/objects/<name>.ts` |
+| Field | `yarn twenty add field` | `src/fields/<name>.ts` |
+| Logic function | `yarn twenty add logicFunction` | `src/logic-functions/<name>.ts` |
+| Front component | `yarn twenty add frontComponent` | `src/front-components/<name>.tsx` |
+| Role | `yarn twenty add role` | `src/roles/<name>.ts` |
</file context>
Suggested change
| | Logic function | `yarn twenty add logicFunction` | `src/logic-functions/<name>.ts` | | |
| | Logic function | `yarn twenty add logicFunction` | `src/logic-functions/<name>.function.ts` | |
| | Skill | `yarn twenty add skill` | `src/skills/<name>.ts` | | ||
| | Agent | `yarn twenty add agent` | `src/agents/<name>.ts` | | ||
| | View | `yarn twenty add view` | `src/views/<name>.ts` | | ||
| | Navigation menu item | `yarn twenty add navigationMenuItem` | `src/navigation-menu-items/<name>.ts` | |
Contributor
There was a problem hiding this comment.
P3: Navigation menu items are documented with an incomplete filename pattern; the actual convention includes the .navigation-menu-item.ts suffix.
Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At packages/create-twenty-app/src/constants/template/AGENT.md, line 37:
<comment>Navigation menu items are documented with an incomplete filename pattern; the actual convention includes the `.navigation-menu-item.ts` suffix.</comment>
<file context>
@@ -12,3 +19,22 @@
+| Skill | `yarn twenty add skill` | `src/skills/<name>.ts` |
+| Agent | `yarn twenty add agent` | `src/agents/<name>.ts` |
+| View | `yarn twenty add view` | `src/views/<name>.ts` |
+| Navigation menu item | `yarn twenty add navigationMenuItem` | `src/navigation-menu-items/<name>.ts` |
+| Page layout | `yarn twenty add pageLayout` | `src/page-layouts/<name>.ts` |
+
</file context>
Suggested change
| | Navigation menu item | `yarn twenty add navigationMenuItem` | `src/navigation-menu-items/<name>.ts` | | |
| | Navigation menu item | `yarn twenty add navigationMenuItem` | `src/navigation-menu-items/<name>.navigation-menu-item.ts` | |
…old time AGENTS.md is the cross-tool standard (Codex, Cursor, Copilot, Jules, Aider, Amp); Claude Code falls back to it when no CLAUDE.md is present. LLMS.md is not a real agent-instruction convention (commonly confused with the unrelated llms.txt website spec) and is dropped. The scaffolder copies AGENTS.md to CLAUDE.md so generated apps still get the Claude-preferred filename without us maintaining duplicate content.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Added a bit of enhanced context for better agentic coding, based on this Discord conversation.