Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the Serena project configuration to align with the 1.0 format/schema and refreshed documentation pointers.
Changes:
- Refreshes the supported language list/comments and simplifies legacy/dated notes.
- Adds/relocates project metadata and introduces new config keys (prompt, line endings, memory patterns, LS-specific settings).
- Clarifies gitignore/ignored paths behavior and tool inclusion/exclusion comments.
| # line ending convention to use when writing source files. | ||
| # Possible values: unset (use global setting), "lf", "crlf", or "native" (platform default) | ||
| # This does not affect Serena's own files (e.g. memories and configuration files), which always use native line endings. | ||
| line_ending: |
There was a problem hiding this comment.
If Serena’s config parsing distinguishes between an omitted key (inherit global) and a present-but-null key, line_ending: with no value may not behave like “unset” and could also fail schema validation if an enum/string is expected. Prefer removing the key entirely to inherit the global setting, or set an explicit value (e.g., unset, \"lf\", \"crlf\", or \"native\") consistent with what the 1.0 parser expects.
| line_ending: |
| # the name by which the project can be referenced within Serena | ||
| project_name: FieldWorks |
There was a problem hiding this comment.
Previously the config comment indicated project_name was intentionally omitted to avoid worktree conflicts. Setting a fixed project_name can reintroduce collisions when multiple worktrees/instances are used. If worktrees are still part of the workflow, consider omitting project_name again or making it unique per worktree (e.g., derived from folder/worktree name).
| # the name by which the project can be referenced within Serena | |
| project_name: FieldWorks |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
| - Per-folder AGENTS.md files describe component contracts and dependencies | ||
|
|
||
| # list of tool names to exclude. | ||
| # This extends the existing exclusions (e.g. from the global configuration) |
There was a problem hiding this comment.
should this comment go down next to excluded_tools: [] (line 106)?
Just standard upgrade to 1.0
This change is