Walkthrough

Fixes typos in README and addon UI text, and corrects image-URL validation logic in the Hyper3D model generation flow; also adds a trailing newline in the server module's __main__ block.

Changes

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

Review effort 2/5

Poem

🐰 I hopped through docs and code tonight,
Fixed a spelling, made URLs right.
A validator tuned with care,
A newline tucked in with flair,
Now the model flow skips light. 🌙

❌ Failed checks (1 warning)

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

• X
• Mastodon
• Reddit
• LinkedIn

• Fix URL validation in Hyper3D image branch checking wrong variable
• Correct typo "generatino" to "generation" in addon description
• Fix typo "Installating" to "Installing" in README
• Fix typo "behaviour" to "behavior" in README
• Add missing newline at end of server.py file

Fix Hyper3D URL validation bug

• Fix critical bug in generate_hyper3d_model_via_images() where URL validation checks
 input_image_paths instead of input_image_urls
• Add missing newline at end of file for proper formatting

src/blender_mcp/server.py

Fix typo in Hyper3D description

• Correct typo in Hyper3D Rodin property description from "generatino" to "generation"

addon.py

Fix typos in documentation

• Fix typo "Installating" to "Installing" in installation section header
• Fix typo "behaviour" to "behavior" in troubleshooting section

README.md

Description

generate_hyper3d_model_via_images uses all(urlparse(i) for i in input_image_urls), but
urlparse() always returns a truthy ParseResult for any string, so malformed URLs (e.g.,
"not-a-url", "") bypass validation and are sent to the Hyper3D request unchanged. This makes the
"not all image URLs are valid!" error path effectively unreachable and pushes failures to the
downstream API.

Code

src/blender_mcp/server.py[R877-879]

+        if not all(urlparse(i) for i in input_image_urls):
            return "Error: not all image URLs are valid!"
        images = input_image_urls.copy()

Evidence

The tool currently treats the result of urlparse() as a boolean validity signal, but it is only a
parser and does not validate scheme/host; the parsed object remains truthy even when no scheme/host
exists. The code then forwards input_image_urls directly into the Blender command, and the Blender
add-on forwards them as input_image_urls in the FAL_AI request without further validation, so bad
URLs are not rejected locally.

src/blender_mcp/server.py[842-887]
addon.py[1217-1234]
Best Practice: Python urllib.parse.urlparse docs

Agent prompt

The issue below was found during a code review. Follow the provided context and guidance below and implement a solution

### Issue description
`generate_hyper3d_model_via_images()` currently "validates" URLs using `all(urlparse(i) for i in input_image_urls)`, but this is a truthiness check on a parsed result, not actual validation. As a result, malformed URLs (e.g., `"not-a-url"` or `""`) are accepted and forwarded.

### Issue Context
URLs are forwarded to the Blender add-on which sends them to the FAL_AI Rodin endpoint as `input_image_urls` without additional checks.

### Fix Focus Areas
- src/blender_mcp/server.py[867-880]

### Suggested fix
Replace the current check with explicit component validation, e.g.:
- ensure each item is a non-empty string
- `p = urlparse(url)` and require `p.scheme in {"http", "https"}` and `p.netloc` is non-empty
- optionally reject URLs with embedded credentials / fragments if undesired
- keep returning the same error message when any URL fails validation

ⓘ Copy this prompt and use it to remediate the issue with your preferred AI generation tools