Skip to content

fix: escape label names for path APIs#83

Merged
haasonsaas merged 1 commit into
mainfrom
codex/fix-label-path-encoding-20260515
May 15, 2026
Merged

fix: escape label names for path APIs#83
haasonsaas merged 1 commit into
mainfrom
codex/fix-label-path-encoding-20260515

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary

  • fix label update URL encoding so path components use %20 for spaces instead of query-string +
  • add regression coverage for labels like autorelease: pending and slash-containing names

Verification

  • ruby -Itest test/sync_labels_test.rb
  • ruby .github/scripts/sync-labels.rb --owner evalops --repos evalops/agentd --dry-run --json-output /tmp/label-sync-agentd.json --markdown-output /tmp/label-sync-agentd.md
  • git diff --check

Refs #6.

@cursor
Copy link
Copy Markdown

cursor Bot commented May 15, 2026

PR Summary

Low Risk
Low risk: small URL-encoding change for GitHub API path construction plus a focused regression test; behavior only differs for names/paths containing spaces or other special characters.

Overview
Fixes GitHub API path escaping in .github/scripts/sync-labels.rb by introducing path_component_escape so spaces are encoded as %20 (not +) when building contents and labels/{name} URLs.

Adds a regression test covering label names with spaces/colons and slashes to ensure path-component encoding is applied consistently.

Reviewed by Cursor Bugbot for commit dc1ac63. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas
Copy link
Copy Markdown
Contributor Author

EvalOps Label Sync Report

  • Generated at: 2026-05-15T20:03:40Z
  • Mode: dry-run
  • Labels: 24
  • Target repos: 60
  • Additions: 760
  • Updates: 155
  • Errors: 0
Repo Status Add Update Notes
evalops/.github in_sync 0 0
evalops/admin in_sync 0 0
evalops/agent-harness in_sync 0 0
evalops/agent-pm in_sync 0 0
evalops/agentd planned 0 2
evalops/bandit_dspy planned 15 2
evalops/cadence planned 13 3
evalops/cerebro planned 15 2
evalops/chat planned 12 4
evalops/cognitive-dissonance-dspy planned 15 2
evalops/conductor planned 13 3
evalops/console planned 15 2
evalops/cypher-llm-compiler planned 15 2
evalops/dagster-user-code planned 14 3
evalops/daybook planned 15 2
evalops/deep-code-reasoning-mcp planned 13 3
evalops/deliberate-reasoning-engine planned 13 3
evalops/deploy planned 11 6
evalops/diagrams planned 15 2
evalops/diffscope planned 12 4
evalops/docs planned 15 2
evalops/dspy-0to1-guide planned 15 2
evalops/dspy-advanced-prompting planned 15 2
evalops/dspy-micro-agent planned 15 2
evalops/ensemble planned 12 5
evalops/eval2otel planned 15 2
evalops/evalops-anthropic planned 13 4
evalops/evalops-google-generativeai planned 13 4
evalops/evalops-openai planned 13 4
evalops/experiments planned 15 2
evalops/explorer planned 15 2
evalops/fabric planned 13 3
evalops/fathom planned 15 2
evalops/finance planned 15 2
evalops/folie-a-deux-dspy planned 15 2
evalops/founder-email-optimizer planned 15 2
evalops/gam-workspace-config planned 15 2
evalops/garak-skill planned 15 2
evalops/gemini-exfil-detector planned 15 2
evalops/hopper planned 13 3
evalops/keep planned 13 3
evalops/kestrel planned 13 3
evalops/lark planned 15 2
evalops/maestro planned 14 2
evalops/maestro-internal planned 11 6
evalops/mcp-firewall planned 15 2
evalops/mcp-openapi planned 13 3
evalops/mocktopus planned 15 2
evalops/nimbus planned 13 3
evalops/openclaw-safety-harness planned 13 3
evalops/orbit-agent planned 14 2
evalops/override-cascade-dspy planned 15 2
evalops/personas planned 15 2
evalops/platform planned 0 11
evalops/proto planned 15 2
evalops/proto-docs planned 15 2
evalops/relay planned 15 2
evalops/service-runtime planned 15 2
evalops/shared-memory-mcp planned 13 3
evalops/template-go-service planned 15 2

@haasonsaas haasonsaas merged commit d36828c into main May 15, 2026
5 checks passed
@haasonsaas haasonsaas deleted the codex/fix-label-path-encoding-20260515 branch May 15, 2026 20:04
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