Skip to content

fix(rwa-alpha): replace UTF-8 box-drawing chars with ASCII#332

Merged
mig-pre merged 1 commit intookx:mainfrom
VibeCodeDaddy69:fix-rwa-alpha-utf8
Apr 24, 2026
Merged

fix(rwa-alpha): replace UTF-8 box-drawing chars with ASCII#332
mig-pre merged 1 commit intookx:mainfrom
VibeCodeDaddy69:fix-rwa-alpha-utf8

Conversation

@VibeCodeDaddy69
Copy link
Copy Markdown
Contributor

Summary

  • Replace box-drawing characters (, , , , ) in skill.md with ASCII equivalents (+--, |--, +, |, -)
  • Replace Unicode arrow with -> in both skill.md and SUMMARY.md
  • Fix CI Phase 1 structure validation panic: the Rust linter crashes on multi-byte UTF-8 chars at non-char-boundary byte offsets (lint.rs:873)

Context

PR #287 was merged with box-drawing characters that cause the CI linter to panic. This follow-up fixes the encoding issue.

Test plan

  • CI Phase 1 structure validation passes without panic
  • skill.md renders correctly with ASCII tree/box chars
  • All content preserved — only decorative characters changed

🤖 Generated with Claude Code

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Apr 23, 2026

✅ Phase 1: Structure Validation — PASSED

Linting skills/rwa-alpha...

  ⚠️  [W010] description is 285 chars (recommended < 200)
  ⚠️  [W140] SKILL.md references 3 external URL(s) not listed in api_calls: 'http://localhost:3249', 'http://localhost:3249', 'http://localhost:3249'. Add them to api_calls in plugin.yaml so reviewers can verify them.

✓ Plugin 'rwa-alpha' passed with 2 warning(s)

→ Proceeding to Phase 2: Build Verification

- Replace box-drawing chars and arrows in SKILL.md with ASCII equivalents
- Remove duplicate skill.md (lowercase) — linter reads SKILL.md (uppercase)
- Fix plugin.yaml description: remove em dash and arrow chars
- Add newsnow.busiyi.world to api_calls

Fixes CI linter panic at lint.rs:873 (non-char-boundary byte index)
@mig-pre mig-pre merged commit e883cac into okx:main Apr 24, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants