You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "choo121600 (via GitHub)" <gi...@apache.org> on 2026/05/11 08:29:31 UTC

[PR] skills/setup-shared-config-sync: trim frontmatter to fit metadata budget [airflow-steward]

choo121600 opened a new pull request, #127:
URL: https://github.com/apache/airflow-steward/pull/127

   ## Summary
   
   Trims `setup-shared-config-sync` frontmatter (`description` + `when_to_use`) from **1,138 → 831** characters.
   
   Same principle as #103 and the rest of #118's audit pass: frontmatter is the routing layer. The body already cross-references `docs/setup/secure-agent-setup.md` and details the per-file flow.
   
   Tracking: #118
   
   ## Before / after
   
   |              | before | after | Δ      |
   |--------------|-------:|------:|-------:|
   | description  | 610    | 474   | -136   |
   | when_to_use  | 528    | 357   | -171   |
   | **total**    | **1,138** | **831** | **-307** |
   | budget margin | 398   | 705   | +307   |
   | budget        | 1,536  | 1,536 |        |
   
   ## What moved where
   
   | Detail                                                                                                                       | Where it lives now                                  |
   |------------------------------------------------------------------------------------------------------------------------------|-----------------------------------------------------|
   | `(per docs/setup/secure-agent-setup.md → Syncing user-scope config across machines)` doc cross-reference                     | Body L34-38 (counterpart-to-sync.sh note + cross-references) |
   | Explicit file glob list (`~/.claude-config/scripts/*.sh`, `~/.claude-config/CLAUDE.md`, `~/.claude-config/commands/*`, etc.) | Tightened to a categorical list: *"scripts, CLAUDE.md, commands, sync.sh"* |
   | Long "Also appropriate after the `setup-isolated-setup-update` skill surfaces drift on a script that the user keeps in `~/.claude-config/` and the user wants the framework's update propagated to every machine the sync repo is checked out on" | Tightened to *"propagated to other machines"* |
   
   ## Trigger-phrase preservation
   
   Every literal trigger phrase from the original `when_to_use` is preserved verbatim:
   
   - `"sync my Claude config"`
   - `"push my ~/.claude-config"`
   - `"commit shared Claude config"`
   
   Behavioral routing signals preserved:
   
   - *"after modifying a file in `~/.claude-config/`"*
   - *"after `setup-isolated-setup-update` surfaces drift"*
   
   Safety rails preserved verbatim:
   
   - *Never force-pushes; never rewrites already-pushed history; never modifies files outside `~/.claude-config/`*
   - Pull-rebase-first ordering (*"`git pull --rebase` first if the local checkout is behind"*)
   
   Routing recall does not regress.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] skills/setup-shared-config-sync: trim frontmatter to fit metadata budget [airflow-steward]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #127:
URL: https://github.com/apache/airflow-steward/pull/127


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org