Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 23, 2026

Addresses review feedback on the CRSLang announcement blog post regarding schema inconsistencies and URL conventions.

Schema standardization

  • Unified YAML structure across all examples: metadata wrapper for id/phase, variables array with name field, nested operator syntax, and disruptive/non-disruptive action categories
  • Applied consistently to logical expression example (lines 84-99) and template functions example (lines 110-133)

Before:

rule:
  id: 1
  phase: 1
  conditions:
    - variable: ARGS:user
      operator: "@streq"
      pattern: "admin"
  actions:
    - action: block

After:

rule:
  metadata:
    id: 1
    phase: 1
  conditions:
    - variables:
        - name: ARGS:user
      operator:
        streq: "admin"
  actions:
    disruptive: block

URL corrections

  • Updated Slack URL to /slack/invite endpoint (matches site convention)
  • Removed mailing list bullet with no target URL

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: fzipi <3012076+fzipi@users.noreply.github.com>
Copilot AI changed the title [WIP] Add blogpost on CRSLang to documentation Standardize CRSLang YAML schema and fix URLs in blog post Jan 23, 2026
Copilot AI requested a review from fzipi January 23, 2026 02:42
@cloudflare-workers-and-pages
Copy link

Deploying website with  Cloudflare Pages  Cloudflare Pages

Latest commit: bf72729
Status: ✅  Deploy successful!
Preview URL: https://23bdb83b.website-1u6.pages.dev
Branch Preview URL: https://copilot-sub-pr-449.website-1u6.pages.dev

View logs

@fzipi fzipi marked this pull request as ready for review January 23, 2026 02:57
@fzipi fzipi merged commit 9628510 into feat/add-blogpost-crslang Jan 23, 2026
1 check passed
@fzipi fzipi deleted the copilot/sub-pr-449 branch January 23, 2026 02:58
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.

2 participants