Skip to content

Update AGENTS project guidance - #240

Merged
VaggelisKa merged 2 commits into
mainfrom
change/update-agents-project-patterns-20260703
Jul 9, 2026
Merged

VaggelisKa merged 2 commits into
mainfrom
change/update-agents-project-patterns-20260703

Conversation

@VaggelisKa

@VaggelisKa VaggelisKa commented Jul 4, 2026 •

Copy link
Copy Markdown
Owner

Summary

  • Rebase the automation branch onto current origin/main so the open AGENTS maintenance PR reflects the latest table/theming work.
  • Add AGENTS guidance that column reordering stays opt-in through [enableReordering] on NatTableSurface.
  • Replace stale stock-theme guidance with the current headless-by-default model: no public --nat-table-* defaults in component CSS, bridge tokens in table-surface.css, and the opt-in stock theme in ng-advanced-table/components/theme.css.

Evidence

Validation

  • git diff --check
  • git diff --cached --check
  • pnpm exec prettier --check AGENTS.md could not run because prettier is not installed in this worktree.

@vercel

vercel Bot commented Jul 4, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
angular-advanced-table Ready Ready Preview, Comment Jul 8, 2026 8:25pm

@VaggelisKa
VaggelisKa force-pushed the change/update-agents-project-patterns-20260703 branch from 124e207 to 933924c Compare July 4, 2026 13:04
@VaggelisKa VaggelisKa changed the title Update AGENTS project guidance Update AGENTS discovery guidance Jul 4, 2026
@VaggelisKa
VaggelisKa force-pushed the change/update-agents-project-patterns-20260703 branch from 933924c to 71c6564 Compare July 6, 2026 18:05
@VaggelisKa
VaggelisKa force-pushed the change/update-agents-project-patterns-20260703 branch from 71c6564 to 26802b6 Compare July 7, 2026 15:50
@VaggelisKa VaggelisKa changed the title Update AGENTS discovery guidance Update AGENTS project guidance Jul 7, 2026
@VaggelisKa
VaggelisKa force-pushed the change/update-agents-project-patterns-20260703 branch from 26802b6 to 0f7dcd5 Compare July 8, 2026 18:56
@VaggelisKa
VaggelisKa marked this pull request as ready for review July 8, 2026 19:03
@greptile-apps

greptile-apps Bot commented Jul 8, 2026 •

Copy link
Copy Markdown
Contributor

Confidence Score: 5/5

Documentation-only change to AGENTS.md with no code or build impact; safe to merge.

The only file changed is AGENTS.md. The new reordering rule accurately reflects the opt-in behavior introduced in #252, and the theming rule still preserves the authoritative three-tier CSS variable fallback chain agents need to follow. No executable code, types, styles, or configuration were modified.

No files require special attention.

Important Files Changed

Filename Overview
AGENTS.md Documentation-only update adding column-reordering opt-in guidance and removing the inline stylelint reference from the theming rule; no code changed.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["Consumer CSS / wrapper element"] -->|"sets --nat-table-x"| B["table-surface.css bridge"]
    B -->|"forwards to --sys-nat-table-x"| C["Component CSS property"]
    D["ng-advanced-table/components/theme.css\n(opt-in stock defaults)"] -->|"declares --nat-table-x defaults"| B
    C -->|"resolves via\nvar(--nat-table-x,\n  var(--sys-nat-table-x,\n    neutral literal))"| E["Final computed value"]
    F["[enableReordering]='true' on NatTableSurface"] -->|"opt-in gate"| G["Drag/drop, keyboard moves\nmove-menu, grab affordances\nreorder instructions"]
    H["Default (no enableReordering)"] -->|"all reorder UI hidden"| I["Headless / no reorder UI"]
Loading
%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A["Consumer CSS / wrapper element"] -->|"sets --nat-table-x"| B["table-surface.css bridge"]
    B -->|"forwards to --sys-nat-table-x"| C["Component CSS property"]
    D["ng-advanced-table/components/theme.css\n(opt-in stock defaults)"] -->|"declares --nat-table-x defaults"| B
    C -->|"resolves via\nvar(--nat-table-x,\n  var(--sys-nat-table-x,\n    neutral literal))"| E["Final computed value"]
    F["[enableReordering]='true' on NatTableSurface"] -->|"opt-in gate"| G["Drag/drop, keyboard moves\nmove-menu, grab affordances\nreorder instructions"]
    H["Default (no enableReordering)"] -->|"all reorder UI hidden"| I["Headless / no reorder UI"]
Loading

Reviews (3): Last reviewed commit: "Merge remote-tracking branch 'origin/mai..." | Re-trigger Greptile

Comment thread AGENTS.md Resolved
@VaggelisKa
VaggelisKa force-pushed the change/update-agents-project-patterns-20260703 branch from 0f7dcd5 to e6d7067 Compare July 8, 2026 19:30
…project-patterns-20260708

# Conflicts:
#	AGENTS.md
@VaggelisKa
VaggelisKa merged commit 78035be into main Jul 9, 2026
8 checks passed
@VaggelisKa
VaggelisKa deleted the change/update-agents-project-patterns-20260703 branch July 9, 2026 06:16

This branch was successfully deployed

1 active deployment
Preview — e163e40b Deployed Jul 8, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant