Skip to content

Fix inaccurate frontmatter null-safety description in AGENTS.md#9

Merged
airtonix merged 3 commits intodocs/update-agents-mdfrom
copilot/sub-pr-7
Nov 17, 2025
Merged

Fix inaccurate frontmatter null-safety description in AGENTS.md#9
airtonix merged 3 commits intodocs/update-agents-mdfrom
copilot/sub-pr-7

Conversation

Copy link
Contributor

Copilot AI commented Nov 17, 2025

The frontmatter null-safety description in AGENTS.md incorrectly stated that moveCard() "throws on undefined frontmatter", which misrepresents the actual code behavior.

Changes

  • Updated line 139 to clarify that moveCard() doesn't check for null/undefined before calling IsObject(), which causes the error
  • Specified the fix more precisely: initialize empty object when frontmatter is null/undefined

Before:

- Frontmatter null-safety: moveCard() throws on undefined frontmatter (should initialize empty object)

After:

- Frontmatter null-safety: moveCard() doesn't handle null/undefined frontmatter before IsObject() check (should initialize empty object when null/undefined)

The updated description matches the actual implementation at KanbanStateController.ts:172 where IsObject(frontmatter) is called without prior null/undefined guards.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Copilot AI changed the title [WIP] Update agent guidelines and manifest.json schema Fix inaccurate frontmatter null-safety description in AGENTS.md Nov 17, 2025
Copilot AI requested a review from airtonix November 17, 2025 12:28
@airtonix airtonix marked this pull request as ready for review November 17, 2025 12:29
@airtonix airtonix merged commit 7df6e16 into docs/update-agents-md Nov 17, 2025
1 of 2 checks passed
@airtonix airtonix deleted the copilot/sub-pr-7 branch November 17, 2025 12:29
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