denoland / denoland/deploy_feedback

[Bug]: Relative import path "@olli/kvdex/ext/zod" not prefixed with.... for `jsr` modules

Open
#632 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
No language data
Stars
79
Forks
5
PR merge metrics
No merged PRs in 30d

Description

### Problem description

It seems like the deno engine on deploy finds it hard to locate relative paths of jsr modules defined inside `imports` of deno.json that were added through `deno add`.
Full paths need to be defined as seperate entry in deno.json for it to work. See [here ](https://github.com/oliver-oloughlin/kvdex/issues/194#issuecomment-1986914154) and [here](https://github.com/waptik/weak-squirrel-86/commit/558e53ff9b77b8ad6bd690544819c4078efc2792)

### Steps to reproduce

1. Clone https://github.com/waptik/weak-squirrel-86
2. Create a new project on deno deploy and point it to the cloned repo
3. Wait for deployment to complete and voila!

### Expected behavior

Relative import for jsr modules defined in `deno.json` should work without the need to manually assign paths in `deno.json` or use the full path prefixed with `jsr:` inside codebase(eg: `jsr:@olli/kvdex/ext/zod`).
Meaning i should be able to safely use `@olli/kvdex/ext/zod`

### Environment

- Deno version: 1.41.1
- OS version: windows 11, wsl(ubuntu 22.04), Brave

### Possible solution

_No response_

### Additional context

_No response_

Contributor guide

No contributing guide indexed for this repository

Research direction

Reproduce the deployment using the weak-squirrel-86 repository and inspect its deno.json imports, then compare the relative @olli/kvdex/ext/zod reference with the full jsr: form described in the issue. Done means Deno Deploy resolves the configured relative import without manually adding full paths or changing the code to use jsr:.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
cloud
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.