fix getCookies import

[guitavano]

Summary by CodeRabbit

• Chores
  • Updated internal module dependencies and reorganized import paths for improved module resolution.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[github-actions]

Tagging Options

Should a new tag be published when this PR is merged?

• 👍 for Patch 0.133.17 update
• 🎉 for Minor 0.134.0 update
• 🚀 for Major 1.0.0 update

[coderabbitai]

Walkthrough

The pull request removes an import alias for @std/http from deno.json and updates a dependent file to use the explicit file path std/http/cookie.ts instead. This refactors module resolution without altering functional behavior.

Changes

Cohort / File(s)	Summary
Import path refactoring deno.json, vtex/loaders/orders/orderplaced.ts	Removed @std/http convenience alias from import mappings and updated getCookies import to use explicit file path std/http/cookie.ts instead of bare module specifier

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5 minutes

• Verify that std/http/cookie.ts is the correct and available path for the getCookies export
• Confirm no other files in the codebase reference the removed @std/http alias

Poem

🐰 Hops of joy, imports take flight,
From @std/http to paths so right,
No aliases hide, just truth so clear,
Direct as a carrot, precise and dear! 🥕

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The pull request has no description. The repository template requires sections like 'What is this Contribution About?', 'Issue Link', 'Loom Video', and 'Demonstration Link', none of which are present.	Add a pull request description following the template, including: a brief explanation of why the import path was changed, a link to the relevant issue, and optionally a Loom video or demonstration link.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'fix getCookies import' is specific and directly matches the main change: updating the getCookies import path from '@std/http' to 'std/http/cookie.ts'.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch tavano/fix-getcookies-import

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a94f6ad and 59b8991.

📒 Files selected for processing (2)

• deno.json (0 hunks)
• vtex/loaders/orders/orderplaced.ts (1 hunks)

💤 Files with no reviewable changes (1)

• deno.json

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Bundle & Check Apps (ubuntu-latest)

🔇 Additional comments (1)

vtex/loaders/orders/orderplaced.ts (1)

1-1: Import path change is correct and properly wired

The import map in deno.json correctly defines "std/" → https://deno.land/std@0.204.0/, so "std/http/cookie.ts" resolves correctly. No remaining @std/http usages exist in the codebase. The change is safe and will not cause runtime resolution errors.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}