Fix malformed org config and add pre-merge lint guard - #97
Merged
justaugustus merged 1 commit intoAug 8, 2026
Merged
Conversation
The stray backtick on the final line of orgs/uwu-tools/org.yaml (merged via uwu-tools#93) made the file invalid YAML, so peribolos exited fatal on every invocation. This broke both the hourly Production sync and the GitHub sync tests: loading orgs: error in orgs/uwu-tools/org.yaml: unmarshal: error converting YAML to JSON: yaml: line 27: found character that cannot start any token Remove the backtick to restore a parseable config. The bug reached main because the existing peribolos checks only run on push to main -- i.e. after merge -- so nothing validated the config on the pull request. Add a Lint workflow that gates on pull_request: - yamllint over all YAML (org/team configs, allstar, settings, workflows), configured via .yamllint.yml to extend the "relaxed" preset (tolerant of the peribolos dump format) and fail on genuine syntax errors while ignoring line-length style noise. - actionlint over the workflow files themselves. Both jobs follow the repo hardening conventions (harden-runner, SHA/digest-pinned actions and images). Assisted-by: Claude Code:claude-opus-4-8 Signed-off-by: Stephen Augustus <foo@auggie.dev>
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
orgs/uwu-tools/org.yaml(introduced by Org updates #93), which made the file invalid YAML.Lintworkflow (yamllint+actionlint) that runs onpull_requestso malformed config is caught before merge.Why
peribolos exited
fatalon every invocation, breaking both the hourly Production sync and the GitHub sync tests:The bug reached
mainbecause the existing peribolos checks only run onpushtomain— i.e. after merge — so nothing validated the config on the PR.The guard
yamllintover all YAML (org/team configs, allstar, settings, workflows), configured via.yamllint.ymlto extend therelaxedpreset (tolerant of the peribolos dump format) and fail on genuine syntax errors while ignoring line-length style noise.actionlintover the workflow files.Validation (local)
yamllint .on the fixed tree → exit 0 (clean).yamllinton the pre-fix file →27:1 error syntax error: found character '' ...` (exit 1) — confirms the guard catches this class of bug.actionlinton all workflows incl. the newlint.yml→ exit 0.