Skip to content

Conversation

@hellovai
Copy link
Contributor

@hellovai hellovai commented Dec 1, 2025

  • Initial research
  • Adding readme

Note

Adds comprehensive BEP-001/002 docs, introduces MkDocs diff/TOC/nav enhancements, and updates CI to send Slack notifications with canary-aware indexing.

  • Docs:
    • BEP-001: Exception Handling: New proposal with background, syntax, guides, alternatives, tooling, and TypeScript deviations.
    • BEP-002: match: New proposal and context docs; README index updated to list both BEPs.
  • MkDocs Hooks/UI:
    • Auto-generates TOCs, highlights inline diffs vs canary and recent commits, adds nav change indicators, and a compare dropdown with summaries.
  • CI/Workflows:
    • Fetches canary explicitly and posts Slack notifications for canary and preview deployments using generated payloads.
  • Scripts/Automation:
    • summarize_changes.py: Summarizes BEP deltas and builds Slack Block Kit payloads.
    • update_bep_readme.py: Canary-aware dates, TBD handling, stricter metadata; regenerates README table.
  • Config/Dev:
    • MkDocs config tweaks; add debug hook; update mise.toml (Node 22, tools).

Written by Cursor Bugbot for commit cd9a6cd. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Dec 1, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
promptfiddle Ready Ready Preview Comment Dec 5, 2025 0:18am

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

Latest build completeView logs

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 75ba8164eaa1db7464df3b8798c6271b5ab07577Workflow run

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 7863ce626985b72df6c3e77b9cb20fc4042ea5b3Workflow run

@github-actions
Copy link

github-actions bot commented Dec 1, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 80c29dcce45ce6f4fc88290e1d54b65b0d9d3013Workflow run

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 2285593eeaa5025033cb2133e8a588dc00452748Workflow run

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 17d376d965071b83ea6f158b7b1863310f09cadbWorkflow run

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

is_safe = False
elif stripped.startswith('- ') or stripped.startswith('* ') or stripped.startswith('+ '): # Unordered list
is_safe = False
elif stripped[0].isdigit() and '. ' in stripped[:5]: # Ordered list (heuristic)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Missing empty string guard causes potential IndexError

The code accesses stripped[0] without checking if stripped is non-empty, which can raise an IndexError. This happens when a line contains only Unicode whitespace characters (like non-breaking space \u00A0) that aren't regular space or tab - such lines pass the not line and line.startswith(' ') checks, but strip() results in an empty string. The similar check at line 247 correctly guards with next_stripped and next_stripped[0].isdigit(), but line 210 is missing this guard.

Fix in Cursor Fix in Web

pass
else:
# Context line (starts with space)
current_new_line += 1
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Diff parser mishandles "No newline" markers causing offset

The else branch in _parse_unified_diff assumes any unmatched line is a context line and increments current_new_line. However, unified diffs can contain \ No newline at end of file markers that start with \, not space. These markers are not actual lines in the file, so incrementing the counter for them causes all subsequent line numbers to be off by one. The code comment says "Context line (starts with space)" but doesn't actually check for that prefix.

Fix in Cursor Fix in Web

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 0a77b82b757d5775d9c8901ba7d251efb4a85ac2Workflow run

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: dc6877bf671d6479a79e9f35b6211d04827c4264Workflow run

summaries = []
# Canary Summary
c_canary = len(parsed_canary["added"]) + len(parsed_canary["removed"])
c_canary = len(parsed_canary)
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Missing HTML escaping for commit subjects in output

Commit subjects from c['subject'] are inserted directly into HTML option elements without escaping. If a commit message contains characters like <, >, &, or ", this could produce malformed HTML output that breaks the dropdown rendering. The previous code only used static option labels without commit subjects, so this is a new issue introduced by this change.

Fix in Cursor Fix in Web

@github-actions
Copy link

github-actions bot commented Dec 3, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 8e92cff51b0183da9930af103df03f6990018cccWorkflow run


# Read the actual file content
file_path = Path("docs/proposals/BEP-001-exceptions/context/go.md")
with open(file_path, "r") as f:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Debug script references deleted file path

This debug script references docs/proposals/BEP-001-exceptions/context/go.md, but this file was deleted in the same commit. The script will crash with FileNotFoundError when executed. This appears to be a development debugging tool that was accidentally committed alongside the production code changes.

Fix in Cursor Fix in Web

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: fee6552326f4732686930f4a0538e2d198af3d7aWorkflow run

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 3ea9af24d70d47a27ba427fcaaf6f2cb90c1b693Workflow run

@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 924d7867b4de2a8ad9705ed48da055125498d39dWorkflow run

- name: Fetch base branch history
run: |
git fetch --depth=500 origin canary
- name: Fetch canary ref
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Shallow clone breaks git history lookup in check script

The removal of fetch-depth: 0 from the check-beps job causes a shallow clone (depth 1 by default). The update_bep_readme.py --check script runs git log {canary_ref}..HEAD to determine file modification dates, which requires commit history between canary and HEAD. With a shallow clone, this history is unavailable, causing the script to incorrectly return "TBD" for last_modified dates instead of actual values, leading to potential check failures or inconsistent behavior between CI and local development.

Fix in Cursor Fix in Web

env: beps-canary

- name: Install uv
uses: astral-sh/setup-uv@v5

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium

Unpinned 3rd party Action 'BEPs (Reusable)' step
Uses Step
uses 'astral-sh/setup-uv' with ref 'v5', not a pinned commit hash
run: git fetch origin canary

- name: Install uv
uses: astral-sh/setup-uv@v5

Check warning

Code scanning / CodeQL

Unpinned tag for a non-immutable Action in workflow Medium

Unpinned 3rd party Action 'BEPs (Reusable)' step
Uses Step
uses 'astral-sh/setup-uv' with ref 'v5', not a pinned commit hash
@github-actions
Copy link

github-actions bot commented Dec 4, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: 9d221e085f4ff604c9915183fda82474a73297c6Workflow run

@github-actions
Copy link

github-actions bot commented Dec 5, 2025

🐑 BEPs Preview Ready

Preview URL: https://dj7ggjkp4tlhz.cloudfront.net/hellovai-beps-exceptions/

Commit: cd9a6cdd36a829ef11b1a16b9f4d80bcd9023809Workflow run

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