Skip to content

Conversation

@ohah
Copy link
Owner

@ohah ohah commented Oct 19, 2025

  • Add comprehensive Cheolsu-Query language documentation (Korean/English)
  • Fix markdown table rendering issues by escaping pipe characters
  • Document filter keywords, operators, usage examples, and tips
  • Update guide and index pages with new documentation

- Add comprehensive Cheolsu-Query language documentation (Korean/English)
- Fix markdown table rendering issues by escaping pipe characters
- Document filter keywords, operators, usage examples, and tips
- Update guide and index pages with new documentation
@ohah ohah self-assigned this Oct 19, 2025
@ohah ohah added the documentation Improvements or additions to documentation label Oct 19, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds comprehensive documentation for Cheolsu-Query, a dedicated query language for filtering network requests in Cheolsu Proxy, available in both Korean and English. The documentation includes syntax reference, usage examples, keyboard shortcuts, auto-completion features, and troubleshooting guides. Additionally, the PR fixes markdown table rendering by properly escaping pipe characters that would otherwise be interpreted as table delimiters.

Key Changes:

  • Added complete Cheolsu-Query documentation files for both Korean and English versions
  • Updated index and guide pages to include references to the new query language documentation
  • Escaped pipe characters in markdown tables to prevent rendering issues

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
docs/ko/index.md Added link to Cheolsu-Query documentation in Korean features section
docs/ko/guide/index.md Added advanced features section with Cheolsu-Query overview and bullet points
docs/ko/features/cheolsu-query.md New comprehensive Korean documentation for Cheolsu-Query language
docs/en/index.md Added link to Cheolsu-Query documentation in English features section
docs/en/guide/index.md Added advanced features section with Cheolsu-Query overview and bullet points
docs/en/features/cheolsu-query.md New comprehensive English documentation for Cheolsu-Query language

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

| `\|=` | 포함 (대소문자 구분) | `url \|= "api"` |
| `\|~` | 포함 (대소문자 무시) | `url \|~ "API"` |
| `!=` | 일치하지 않음 | `method!="OPTIONS"` |
| `!~` | 포함하지 않음 | `url!~="static"` |
Copy link

Copilot AI Oct 19, 2025

Choose a reason for hiding this comment

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

The operator !~ in the example uses != syntax (url!~=\"static\"). Based on the operator definition and other examples in the document, this should be url!~\"static\" (without the = after !~).

Suggested change
| `!~` | 포함하지 않음 | `url!~="static"` |
| `!~` | 포함하지 않음 | `url!~"static"` |

Copilot uses AI. Check for mistakes.
| `\|=` | Contains (case-sensitive) | `url \|= "api"` |
| `\|~` | Contains (case-insensitive) | `url \|~ "API"` |
| `!=` | Not equal | `method!="OPTIONS"` |
| `!~` | Does not contain | `url!~="static"` |
Copy link

Copilot AI Oct 19, 2025

Choose a reason for hiding this comment

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

The operator !~ in the example uses != syntax (url!~=\"static\"). Based on the operator definition and other examples in the document, this should be url!~\"static\" (without the = after !~).

Suggested change
| `!~` | Does not contain | `url!~="static"` |
| `!~` | Does not contain | `url!~"static"` |

Copilot uses AI. Check for mistakes.
@ohah ohah merged commit 311ad22 into master Oct 19, 2025
2 checks passed
@ohah ohah deleted the docs/search-ql branch October 19, 2025 16:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants