Skip to content

chore: sync code standards from vault - #64

Merged
aliasunder merged 2 commits into
mainfrom
chore/sync-code-standards
Aug 24, 2026
Merged

chore: sync code standards from vault#64
aliasunder merged 2 commits into
mainfrom
chore/sync-code-standards

Conversation

@aliasunder

Copy link
Copy Markdown
Owner

Summary

  • Refresh AGENTS.md code standards from vault Reference notes (last synced 2026-07-22)
  • Comment decision tree: write-time checklist replacing the abstract "comments explain non-obvious context" rule
  • Conditional spreads / filter(Boolean) both acceptable — no mechanical conversion
  • Required inputs enforced at every entry point
  • Write-time format decision for docs (table/bullets/steps over prose blobs)
  • No internal references in any public artifact

Test plan

  • Review AGENTS.md diff for accuracy against vault notes
  • Confirm no repo-specific rules were overwritten

🤖 Generated with Claude Code

aliasunder and others added 2 commits August 24, 2026 13:51
Refresh AGENTS.md code standards section from vault Reference notes:
- Comment decision tree: write-time checklist (most functions need no
  comment; inline comments at the relevant line, not in the docstring)
- Conditional spreads and filter(Boolean) both fine — no mechanical
  conversion
- Required inputs enforced at every entry point
- Write-time format decision for docs (table/bullets/steps over prose)
- No internal references in any public artifact (issue/PR numbers,
  task-board IDs, incident dates)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aliasunder
aliasunder merged commit 0e16381 into main Aug 24, 2026
9 checks passed
@aliasunder
aliasunder deleted the chore/sync-code-standards branch August 24, 2026 18:00
Comment thread AGENTS.md
@umm-actually

umm-actually Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

umm-actually reviewed at 9d1398d

1 new finding(s) posted (1 tracked finding(s) across all runs).


umm-actually · deepseek/deepseek-v4-flash-0731

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.

1 participant