makememe 0.1.2__tar.gz → 0.1.4__tar.gz

{makememe-0.1.2 → makememe-0.1.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: makememe
-Version: 0.1.2
+Version: 0.1.4
 Summary: A tiny zero-dependency CLI for generating memes via the free memegen.link API. Agent-friendly.
 Project-URL: Homepage, https://pypi.org/project/makememe/
 Project-URL: memegen.link API, https://api.memegen.link
@@ -67,7 +67,7 @@ meme <template> "top line" "bottom line" [-o out.png]
 
 | Flag | Meaning |
 |------|---------|
-| `-o, --out` | output file (default `meme.png`) |
+| `-o, --out` | output file (default: a unique file in a temp folder, so it never writes into your current directory) |
 | `--bg URL` | use a custom background image instead of a template |
 | `--ext` | `png` (default), `jpg`, `webp`, or `gif` |
 | `--style` / `--font` | template style variant / font override |
@@ -106,7 +106,7 @@ The tool is designed to be parsed:
 - **`--json` mode** prints a single JSON object:
 
   ```json
-  { "path": "meme.png", "bytes": 12345, "url": "https://api.memegen.link/..." }
+  { "path": "/tmp/makememe/meme-ab12cd.png", "bytes": 12345, "url": "https://api.memegen.link/..." }
   ```
 
   `--list --json` returns the template catalog as JSON; `--print-url --json`

{makememe-0.1.2 → makememe-0.1.4}/README.md

@@ -48,7 +48,7 @@ meme <template> "top line" "bottom line" [-o out.png]
 
 | Flag | Meaning |
 |------|---------|
-| `-o, --out` | output file (default `meme.png`) |
+| `-o, --out` | output file (default: a unique file in a temp folder, so it never writes into your current directory) |
 | `--bg URL` | use a custom background image instead of a template |
 | `--ext` | `png` (default), `jpg`, `webp`, or `gif` |
 | `--style` / `--font` | template style variant / font override |
@@ -87,7 +87,7 @@ The tool is designed to be parsed:
 - **`--json` mode** prints a single JSON object:
 
   ```json
-  { "path": "meme.png", "bytes": 12345, "url": "https://api.memegen.link/..." }
+  { "path": "/tmp/makememe/meme-ab12cd.png", "bytes": 12345, "url": "https://api.memegen.link/..." }
   ```
 
   `--list --json` returns the template catalog as JSON; `--print-url --json`

{makememe-0.1.2 → makememe-0.1.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "makememe"
-version = "0.1.2"
+version = "0.1.4"
 description = "A tiny zero-dependency CLI for generating memes via the free memegen.link API. Agent-friendly."
 readme = "README.md"
 requires-python = ">=3.8"

{makememe-0.1.2 → makememe-0.1.4}/src/memecli/SKILL.md

@@ -26,9 +26,13 @@ uv tool install makememe
    meme --list --json
    ```
 
-   Common ids: `drake`, `db` (distracted boyfriend), `buttons` (two buttons),
-   `gru` (gru's plan), `cmm` (change my mind), `fine` (this is fine),
-   `success` (success kid), `rollsafe`, `same` (same picture), `regret`.
+   Common ids: `drake`, `db` (distracted boyfriend), `ds` (daily struggle /
+   "two buttons"), `gru` (gru's plan), `cmm` (change my mind),
+   `fine` (this is fine), `success` (success kid), `rollsafe`,
+   `same` (same picture), `regret`.
+
+   **Always verify an id with `meme --list --json` before using it** — guessing
+   ids (e.g. `buttons`, `twobuttons`) leads to 404s. When unsure, list first.
 
 2. **Generate.** Pass the template id then the caption lines in order. Use
    `--json` so you can capture the output path reliably:
@@ -40,15 +44,17 @@ uv tool install makememe
    Output:
 
    ```json
-   { "path": "meme.png", "bytes": 12345, "url": "https://api.memegen.link/..." }
+   { "path": "/tmp/makememe/meme-ab12cd.png", "bytes": 12345, "url": "https://api.memegen.link/..." }
    ```
 
 3. **Tell the user the path** (and show the image if the surface supports it).
 
 ## Key flags
 
-- `-o out.png` — choose the output filename (default `meme.png`). Pick a
-  descriptive name when generating several.
+- By default the image is saved to a temp folder (`<tmp>/makememe/`) with a
+  unique name, so it never clutters the user's working directory. The path is
+  in the output — report it to the user. Use `-o path.png` only if the user
+  wants it saved somewhere specific.
 - `--bg <image-url>` — use a custom background image instead of a template;
   pass caption lines as usual.
 - `--ext png|jpg|webp|gif` — output format.

{makememe-0.1.2 → makememe-0.1.4}/src/memecli/cli.py

@@ -23,12 +23,23 @@ import argparse
 import json
 import os
 import sys
+import tempfile
 import urllib.parse
 import urllib.request
 
 API = "https://api.memegen.link"
 
 
+def default_output(ext):
+    """A unique output path in a grouped temp folder, so we never write into the
+    user's project/root dir and never clobber a previous meme."""
+    d = os.path.join(tempfile.gettempdir(), "makememe")
+    os.makedirs(d, exist_ok=True)
+    fd, path = tempfile.mkstemp(prefix="meme-", suffix=f".{ext}", dir=d)
+    os.close(fd)
+    return path
+
+
 def get_version():
     """Read the installed package version; fall back gracefully from source."""
     try:
@@ -134,8 +145,8 @@ def build_parser():
     ap.add_argument("template", nargs="?",
                     help="template id (see --list), or any id when using --bg")
     ap.add_argument("lines", nargs="*", help="text lines, in order")
-    ap.add_argument("-o", "--out", default="meme.png",
-                    help="output file (default meme.png)")
+    ap.add_argument("-o", "--out", default=None,
+                    help="output file (default: a temp folder, not your current dir)")
     ap.add_argument("--bg",
                     help="custom background image URL (uses the 'custom' template)")
     ap.add_argument("--ext", default="png", choices=["png", "jpg", "webp", "gif"])
@@ -190,18 +201,31 @@ def _run(argv=None):
             print(url)
         return
 
+    out = args.out if args.out else default_output(args.ext)
+
     try:
-        n = download(url, args.out)
+        n = download(url, out)
     except Exception as e:
+        # a 404 almost always means a bad template id — point the user/agent at --list
+        hint = None
+        if getattr(e, "code", None) == 404 and not args.bg:
+            hint = (f"template '{template}' not found (404). "
+                    "Run `meme --list` to find the correct id.")
         if args.json:
-            print(json.dumps({"error": str(e), "url": url}))
+            payload = {"error": str(e), "url": url}
+            if hint:
+                payload["hint"] = hint
+            print(json.dumps(payload))
             sys.exit(1)
-        sys.exit(f"download failed: {e}\nurl was: {url}")
+        msg = f"download failed: {e}\nurl was: {url}"
+        if hint:
+            msg += f"\nhint: {hint}"
+        sys.exit(msg)
 
     if args.json:
-        print(json.dumps({"path": args.out, "bytes": n, "url": url}))
+        print(json.dumps({"path": out, "bytes": n, "url": url}))
     else:
-        print(args.out)                   # stdout: just the path, easy to capture
+        print(out)                        # stdout: just the path, easy to capture
         print(f"saved {n} bytes from {url}", file=sys.stderr)
 
 

{makememe-0.1.2 → makememe-0.1.4}/tests/test_cli.py

@@ -93,6 +93,19 @@ class TestCrashSafety(unittest.TestCase):
         finally:
             cli.download = orig
 
+    def test_404_adds_list_hint(self):
+        class FakeHTTPError(Exception):
+            code = 404
+        orig = cli.download
+        cli.download = lambda *a, **k: (_ for _ in ()).throw(FakeHTTPError("404"))
+        try:
+            buf = io.StringIO()
+            with redirect_stdout(buf), redirect_stderr(io.StringIO()):
+                self.assertEqual(self._exit_code(["buttons", "a", "b", "--json"]), 1)
+            self.assertIn("meme --list", buf.getvalue())
+        finally:
+            cli.download = orig
+
     def test_download_failure_is_clean_exit_in_json(self):
         orig = cli.download
         cli.download = lambda *a, **k: (_ for _ in ()).throw(OSError("boom"))
@@ -103,6 +116,28 @@ class TestCrashSafety(unittest.TestCase):
             cli.download = orig
 
 
+class TestDefaultOutput(unittest.TestCase):
+    def test_default_is_in_tempdir_not_cwd(self):
+        import tempfile
+        p = cli.default_output("png")
+        self.assertTrue(p.startswith(tempfile.gettempdir()))
+        self.assertTrue(p.endswith(".png"))
+        # cleanup the empty file mkstemp created
+        try:
+            os.remove(p)
+        except OSError:
+            pass
+
+    def test_default_paths_are_unique(self):
+        a, b = cli.default_output("png"), cli.default_output("png")
+        self.assertNotEqual(a, b)
+        for p in (a, b):
+            try:
+                os.remove(p)
+            except OSError:
+                pass
+
+
 class TestNewCommands(unittest.TestCase):
     def test_version_flag_prints_and_exits(self):
         buf = io.StringIO()