Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,10 @@ tool-owned format outside `.bat`/`.cmd`, or a byte-preserve data directory whose
consumer may depend on), still pair a `.gitattributes` pin with a matching `.editorconfig`
override, since the git pin alone is not enough there, `.gitattributes` governs git while the
editor follows `.editorconfig`. For a byte-preserve directory, disable all editor normalization,
not just EOL: `[<dir>/*]` with `charset = unset`, `end_of_line = unset`, `insert_final_newline =
not just EOL: `[<dir>/**]` with `charset = unset`, `end_of_line = unset`, `insert_final_newline =
false`, `trim_trailing_whitespace = false` (`unset` is EditorConfig's spec-defined special value
that removes an inherited property).
that removes an inherited property, and `**` is needed rather than `*` so a nested file under the
directory is covered too, since `*` excludes `/` and only matches one path component).

## Editing discipline

Expand Down
11 changes: 10 additions & 1 deletion .agents/skills/dotnet-codestyle/references/conventions.md
Original file line number Diff line number Diff line change
Expand Up @@ -122,5 +122,14 @@ parameters, return values, exceptions, and crefs.
/// <exception cref="System.ArgumentException">
/// Thrown when <paramref name="category"/> is not a supported value.
/// </exception>
public async Task<string> GetQuoteOfTheDayAsync(string category, CancellationToken cancellationToken) {}
public async Task<string> GetQuoteOfTheDayAsync(string category, CancellationToken cancellationToken)
{
if (category is not ("motivational" or "humor"))
{
throw new ArgumentException($"Unsupported category: {category}", nameof(category));
}

await Task.Delay(0, cancellationToken);
return $"Quote for {category}";
Comment thread
qodo-code-review[bot] marked this conversation as resolved.
}
```
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Two traps, both learned the hard way:
`git checkout -b promote/develop-to-main origin/main && git merge origin/develop`, take
`develop`'s side for the EOL-conflicted files (`git checkout --theirs <file>`) **after
confirming each is content-identical modulo EOL, or that `develop` is a strict superset**
(`diff <(git show :2:f | tr -d '\r') <(git show :3:f | tr -d '\r')`), then open that branch into
(`diff <(git show ":2:<file>" | tr -d '\r') <(git show ":3:<file>" | tr -d '\r')`), then open that branch into
`main`. Verify no genuine `main`-only content is dropped (build/test where the repo supports it).

## Why both rulesets omit "Require branches to be up to date before merging"
Expand Down
5 changes: 3 additions & 2 deletions .agents/skills/resync-a-repo/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,9 @@ Preserve the evidence RESYNC.md section 2 requires, and do not leave the finding
4. **Interface workflows.** Honor the named contract, required jobs, the ruleset-bound check name,
the artifact-name handoff, rather than copying bytes.
5. **Settings, rulesets, and secrets.** Run
`repo-config/configure.sh check <owner>/<repo> release|operational` from the hub at `main`,
then `apply` for what it reports, never from a carried copy.
`repo-config/configure.sh check "<owner>/<repo>" release` (substitute `operational` for an
operational repo) from the hub at `main`, then `apply` for what it reports, never from a
carried copy.
6. **Intent files last, and by hand,** since nothing mechanical judges these.

Reconcile the registry entry (`status`, `types`, `releaseTrigger`, `workflowModel`,
Expand Down
5 changes: 3 additions & 2 deletions .agents/skills/standup-a-repo/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,9 @@ maintainer can supply what section 0A lists.

8. **Settings, rulesets, and secrets.** STANDUP.md section 4: confirm the remote and the GitHub
repository agree before running anything else here, then apply with
`repo-config/configure.sh apply owner/repo release|operational` from the hub at `main` and check with the same
command's `check` subcommand, never from a hand-built or carried copy.
`repo-config/configure.sh apply owner/repo release` (substitute `operational` for an
operational repo) from the hub at `main` and check with the same command's `check` subcommand,
never from a hand-built or carried copy.

9. **Verify with the audit.** STANDUP.md section 5: run `AUDIT.md` end to end. The repo is stood
up only when it passes for its type, or its residual deltas are tracked in
Expand Down
2 changes: 1 addition & 1 deletion .claude-plugin/fleet-skills/.source-digest
Original file line number Diff line number Diff line change
@@ -1 +1 @@
b945e66c274cb82a
702692e5f8c8f60e
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,10 @@ tool-owned format outside `.bat`/`.cmd`, or a byte-preserve data directory whose
consumer may depend on), still pair a `.gitattributes` pin with a matching `.editorconfig`
override, since the git pin alone is not enough there, `.gitattributes` governs git while the
editor follows `.editorconfig`. For a byte-preserve directory, disable all editor normalization,
not just EOL: `[<dir>/*]` with `charset = unset`, `end_of_line = unset`, `insert_final_newline =
not just EOL: `[<dir>/**]` with `charset = unset`, `end_of_line = unset`, `insert_final_newline =
false`, `trim_trailing_whitespace = false` (`unset` is EditorConfig's spec-defined special value
that removes an inherited property).
that removes an inherited property, and `**` is needed rather than `*` so a nested file under the
directory is covered too, since `*` excludes `/` and only matches one path component).

## Editing discipline

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -122,5 +122,14 @@ parameters, return values, exceptions, and crefs.
/// <exception cref="System.ArgumentException">
/// Thrown when <paramref name="category"/> is not a supported value.
/// </exception>
public async Task<string> GetQuoteOfTheDayAsync(string category, CancellationToken cancellationToken) {}
public async Task<string> GetQuoteOfTheDayAsync(string category, CancellationToken cancellationToken)
{
if (category is not ("motivational" or "humor"))
{
throw new ArgumentException($"Unsupported category: {category}", nameof(category));
}

await Task.Delay(0, cancellationToken);
return $"Quote for {category}";
}
```
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Two traps, both learned the hard way:
`git checkout -b promote/develop-to-main origin/main && git merge origin/develop`, take
`develop`'s side for the EOL-conflicted files (`git checkout --theirs <file>`) **after
confirming each is content-identical modulo EOL, or that `develop` is a strict superset**
(`diff <(git show :2:f | tr -d '\r') <(git show :3:f | tr -d '\r')`), then open that branch into
(`diff <(git show ":2:<file>" | tr -d '\r') <(git show ":3:<file>" | tr -d '\r')`), then open that branch into
`main`. Verify no genuine `main`-only content is dropped (build/test where the repo supports it).

## Why both rulesets omit "Require branches to be up to date before merging"
Expand Down
5 changes: 3 additions & 2 deletions .claude-plugin/fleet-skills/skills/resync-a-repo/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,9 @@ Preserve the evidence RESYNC.md section 2 requires, and do not leave the finding
4. **Interface workflows.** Honor the named contract, required jobs, the ruleset-bound check name,
the artifact-name handoff, rather than copying bytes.
5. **Settings, rulesets, and secrets.** Run
`repo-config/configure.sh check <owner>/<repo> release|operational` from the hub at `main`,
then `apply` for what it reports, never from a carried copy.
`repo-config/configure.sh check "<owner>/<repo>" release` (substitute `operational` for an
operational repo) from the hub at `main`, then `apply` for what it reports, never from a
carried copy.
6. **Intent files last, and by hand,** since nothing mechanical judges these.

Reconcile the registry entry (`status`, `types`, `releaseTrigger`, `workflowModel`,
Expand Down
5 changes: 3 additions & 2 deletions .claude-plugin/fleet-skills/skills/standup-a-repo/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,9 @@ maintainer can supply what section 0A lists.

8. **Settings, rulesets, and secrets.** STANDUP.md section 4: confirm the remote and the GitHub
repository agree before running anything else here, then apply with
`repo-config/configure.sh apply owner/repo release|operational` from the hub at `main` and check with the same
command's `check` subcommand, never from a hand-built or carried copy.
`repo-config/configure.sh apply owner/repo release` (substitute `operational` for an
operational repo) from the hub at `main` and check with the same command's `check` subcommand,
never from a hand-built or carried copy.

9. **Verify with the audit.** STANDUP.md section 5: run `AUDIT.md` end to end. The repo is stood
up only when it passes for its type, or its residual deltas are tracked in
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -71,9 +71,10 @@ tool-owned format outside `.bat`/`.cmd`, or a byte-preserve data directory whose
consumer may depend on), still pair a `.gitattributes` pin with a matching `.editorconfig`
override, since the git pin alone is not enough there, `.gitattributes` governs git while the
editor follows `.editorconfig`. For a byte-preserve directory, disable all editor normalization,
not just EOL: `[<dir>/*]` with `charset = unset`, `end_of_line = unset`, `insert_final_newline =
not just EOL: `[<dir>/**]` with `charset = unset`, `end_of_line = unset`, `insert_final_newline =
false`, `trim_trailing_whitespace = false` (`unset` is EditorConfig's spec-defined special value
that removes an inherited property).
that removes an inherited property, and `**` is needed rather than `*` so a nested file under the
directory is covered too, since `*` excludes `/` and only matches one path component).

## Editing discipline

Expand Down
11 changes: 10 additions & 1 deletion .github/skills/dotnet-codestyle/references/conventions.md
Original file line number Diff line number Diff line change
Expand Up @@ -122,5 +122,14 @@ parameters, return values, exceptions, and crefs.
/// <exception cref="System.ArgumentException">
/// Thrown when <paramref name="category"/> is not a supported value.
/// </exception>
public async Task<string> GetQuoteOfTheDayAsync(string category, CancellationToken cancellationToken) {}
public async Task<string> GetQuoteOfTheDayAsync(string category, CancellationToken cancellationToken)
{
if (category is not ("motivational" or "humor"))
{
throw new ArgumentException($"Unsupported category: {category}", nameof(category));
}

await Task.Delay(0, cancellationToken);
return $"Quote for {category}";
}
Comment thread
coderabbitai[bot] marked this conversation as resolved.
```
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Two traps, both learned the hard way:
`git checkout -b promote/develop-to-main origin/main && git merge origin/develop`, take
`develop`'s side for the EOL-conflicted files (`git checkout --theirs <file>`) **after
confirming each is content-identical modulo EOL, or that `develop` is a strict superset**
(`diff <(git show :2:f | tr -d '\r') <(git show :3:f | tr -d '\r')`), then open that branch into
(`diff <(git show ":2:<file>" | tr -d '\r') <(git show ":3:<file>" | tr -d '\r')`), then open that branch into
`main`. Verify no genuine `main`-only content is dropped (build/test where the repo supports it).

## Why both rulesets omit "Require branches to be up to date before merging"
Expand Down
5 changes: 3 additions & 2 deletions .github/skills/resync-a-repo/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -67,8 +67,9 @@ Preserve the evidence RESYNC.md section 2 requires, and do not leave the finding
4. **Interface workflows.** Honor the named contract, required jobs, the ruleset-bound check name,
the artifact-name handoff, rather than copying bytes.
5. **Settings, rulesets, and secrets.** Run
`repo-config/configure.sh check <owner>/<repo> release|operational` from the hub at `main`,
then `apply` for what it reports, never from a carried copy.
`repo-config/configure.sh check "<owner>/<repo>" release` (substitute `operational` for an
operational repo) from the hub at `main`, then `apply` for what it reports, never from a
carried copy.
6. **Intent files last, and by hand,** since nothing mechanical judges these.

Reconcile the registry entry (`status`, `types`, `releaseTrigger`, `workflowModel`,
Expand Down
5 changes: 3 additions & 2 deletions .github/skills/standup-a-repo/SKILL.md
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,9 @@ maintainer can supply what section 0A lists.

8. **Settings, rulesets, and secrets.** STANDUP.md section 4: confirm the remote and the GitHub
repository agree before running anything else here, then apply with
`repo-config/configure.sh apply owner/repo release|operational` from the hub at `main` and check with the same
command's `check` subcommand, never from a hand-built or carried copy.
`repo-config/configure.sh apply owner/repo release` (substitute `operational` for an
operational repo) from the hub at `main` and check with the same command's `check` subcommand,
never from a hand-built or carried copy.

9. **Verify with the audit.** STANDUP.md section 5: run `AUDIT.md` end to end. The repo is stood
up only when it passes for its type, or its residual deltas are tracked in
Expand Down