Skip to content

1.0.0 Discord Random Text Formatter - #245

Merged
smeech merged 3 commits into
espanso:mainfrom
kreezxil:main
Oct 14, 2026
Merged

smeech merged 3 commits into
espanso:mainfrom
kreezxil:main

Conversation

@kreezxil

Copy link
Copy Markdown
Contributor

an image is worth a 1000 words. No one needs it but here it is.

image

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The manifest fails repository validation and the trigger references a nonexistent script path.

3 open findings
What changed in this PR

Adds an Espanso package that opens a Tkinter prompt and applies randomized Discord formatting.

Changes:

  • Adds the formatting GUI script.
  • Registers the :randytext trigger.
  • Documents setup and usage.
File Description
scripts/​random_discord_formatter.py Implements formatting and GUI input.
README.md Documents behavior and usage.
package.yml Defines the Espanso trigger.
_manifest.yml Provides package metadata.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread packages/random-discord-text-formatter/1.0.0/_manifest.yml Outdated
Comment thread packages/random-discord-text-formatter/1.0.0/package.yml Outdated
Comment thread packages/random-discord-text-formatter/1.0.0/random_discord_text_formatter.py Outdated

@smeech smeech left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A fun package to reformat text in a markdown-capable environment such as Discord. Uses a benign-looking Python script.

I'll run the automated checks and let Copilot comment.

Comment thread packages/random-discord-text-formatter/1.0.0/README.md Outdated
Comment thread packages/random-discord-text-formatter/1.0.0/README.md Outdated
Comment thread packages/random-discord-text-formatter/1.0.0/README.md Outdated
Comment thread packages/random-discord-text-formatter/1.0.0/package.yml Outdated
again, with more smeechiness
this should be it.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

Unescaped Markdown delimiters can produce invalid output, and boundary whitespace is not preserved as documented.

2 open findings
3 resolved since last review

🧠 Review effort: Balanced

Comment on lines +47 to +49
word = random_case_word(part)
formatter = random.choice(FORMATTERS)
result.append(formatter(word))

def main() -> None:
# Form field "text" is exposed as ESPANSO_FORM1_TEXT
raw = os.environ.get("ESPANSO_FORM1_TEXT", "").strip()
@smeech
smeech self-requested a review October 14, 2026 18:59

@smeech smeech left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thank you for the changes. Copilot has raised a couple of issues which I am happy for you to address or ignore. They depend on the user input, so don't matter much.

I'm re-running the automated checks to make sure it will merge successfully. If that's clear and you're happy with the package as it is, I'll go ahead and merge. It'll take a couple of hours to appear in the Hub.

@kreezxil

Copy link
Copy Markdown
Contributor Author

merge them. I tested in discord what copilot thought might happen with backticks and it's not happening. I believe that is because of the lambdas operating on each character one by one.

@smeech
smeech merged commit d83491a into espanso:main Oct 14, 2026
1 check passed
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.

3 participants