Skip to content

Conversation

@JeanMeijer
Copy link
Collaborator

Summary

  • support Temporal polyfill as peerDependency
  • handle ZonedDateTime values when exporting events to iCal
  • expose helpers to import/export VTIMEZONE components

Testing

  • bun run format
  • bun run lint

https://chatgpt.com/codex/tasks/task_e_68765fec40f4832b80bd06916feac8de

@coderabbitai
Copy link

coderabbitai bot commented Jul 15, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


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

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

@vercel
Copy link

vercel bot commented Jul 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
analog Ready Ready Preview Comment Sep 10, 2025 1:22pm

@JeanMeijer JeanMeijer changed the title Add @analog/ical timezone support feat(ical) add @analog/ical package support Jul 27, 2025
@JeanMeijer JeanMeijer changed the title feat(ical) add @analog/ical package support feat(ical): add @analog/ical package support Jul 27, 2025
@JeanMeijer JeanMeijer force-pushed the codex/add-@analog/ical-package-for-ical-handling branch from 66eeda3 to da3ede7 Compare September 8, 2025 12:00
@JeanMeijer JeanMeijer marked this pull request as ready for review September 10, 2025 13:20
@JeanMeijer JeanMeijer changed the title feat(ical): add @analog/ical package support feat(ical): add icalendar package Sep 10, 2025
@socket-security
Copy link

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
safer-buffer@2.1.2 has Obfuscated code.

Confidence: 0.94

Location: Package overview

From: ?npm/trpc-to-mcp@1.3.2npm/trpc-to-openapi@3.0.1npm/safer-buffer@2.1.2

ℹ Read more on: This package | This alert | What is obfuscated code?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Packages should not obfuscate their code. Consider not using packages with obfuscated code.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/safer-buffer@2.1.2. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

3 issues found across 11 files

React with 👍 or 👎 to teach cubic. Mention @cubic-dev-ai to give feedback, ask questions, or re-run the review.

// Re-export all functions and types
export * from "./import";
export * from "./export";
export type { iCalendarEvent } from "./interfaces";
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Sep 10, 2025

Choose a reason for hiding this comment

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

Comment claims all types are re-exported, but iCalendar interface isn’t exposed via the public entrypoint; re-export it for API completeness.

Prompt for AI agents
Address the following comment on packages/ical/src/index.ts at line 4:

<comment>Comment claims all types are re-exported, but iCalendar interface isn’t exposed via the public entrypoint; re-export it for API completeness.</comment>

<file context>
@@ -0,0 +1,4 @@
+// Re-export all functions and types
+export * from &quot;./import&quot;;
+export * from &quot;./export&quot;;
+export type { iCalendarEvent } from &quot;./interfaces&quot;;
</file context>
Fix with Cubic

"@repo/typescript-config": "workspace:*",
"@types/node": "^22.9.0",
"eslint": "^9.30.1",
"typescript": "^5.8.3",
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Sep 10, 2025

Choose a reason for hiding this comment

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

TypeScript version here diverges from the monorepo (^5.9.2 elsewhere), which can cause inconsistent type checking; align to the shared version.

Prompt for AI agents
Address the following comment on bun.lock at line 263:

<comment>TypeScript version here diverges from the monorepo (^5.9.2 elsewhere), which can cause inconsistent type checking; align to the shared version.</comment>

<file context>
@@ -248,6 +249,23 @@
+        &quot;@repo/typescript-config&quot;: &quot;workspace:*&quot;,
+        &quot;@types/node&quot;: &quot;^22.9.0&quot;,
+        &quot;eslint&quot;: &quot;^9.30.1&quot;,
+        &quot;typescript&quot;: &quot;^5.8.3&quot;,
+      },
+      &quot;peerDependencies&quot;: {
</file context>
Fix with Cubic

"@repo/eslint-config": "workspace:*",
"@repo/typescript-config": "workspace:*",
"@types/node": "^22.9.0",
"eslint": "^9.30.1",
Copy link
Contributor

@cubic-dev-ai cubic-dev-ai bot Sep 10, 2025

Choose a reason for hiding this comment

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

ESLint version here diverges from the monorepo (^9.34.0 elsewhere), leading to inconsistent linting; align to the shared version.

Prompt for AI agents
Address the following comment on bun.lock at line 262:

<comment>ESLint version here diverges from the monorepo (^9.34.0 elsewhere), leading to inconsistent linting; align to the shared version.</comment>

<file context>
@@ -248,6 +249,23 @@
+        &quot;@repo/eslint-config&quot;: &quot;workspace:*&quot;,
+        &quot;@repo/typescript-config&quot;: &quot;workspace:*&quot;,
+        &quot;@types/node&quot;: &quot;^22.9.0&quot;,
+        &quot;eslint&quot;: &quot;^9.30.1&quot;,
+        &quot;typescript&quot;: &quot;^5.8.3&quot;,
+      },
</file context>
Fix with Cubic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants