feat: add iCalSequence to new booking data

[Devanshusharma2005]

What does this PR do?

• Fixes Rescheduled bookings: ICS sequence numbers don't increment after 1 #22779
• Fixes CAL-6164

Visual Demo (For contributors especially)

A visual demonstration is strongly recommended, for both the original and new change (video / image - any one).

Video Demo (if applicable):

• Show screen recordings of the issue or feature.
• Demonstrate how to reproduce the issue, the behavior before and after the change.

Image Demo (if applicable):

• Add side-by-side screenshots of the original and updated change.
• Highlight any significant change(s).

Mandatory Tasks (DO NOT REMOVE)

• I have self-reviewed the code (A decent size PR without self-review might be rejected).
• I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. If N/A, write N/A here and check the checkbox.
• I confirm automated tests are in place that prove my fix is effective or that my feature works.

How should this be tested?

• Are there environment variables that should be set?
• What are the minimal test data to have?
• What is expected (happy path) to have (input and output)?
• Any other important info that could help to test that PR

Checklist

• I haven't read the contributing guide
• My code doesn't follow the style guidelines of this project
• I haven't commented my code, particularly in hard-to-understand areas
• I haven't checked if my changes generate no new warnings

[vercel]

@Devanshusharma2005 is attempting to deploy a commit to the cal Team on Vercel.

A member of the Team first needs to authorize it.

[coderabbitai]

Caution

Review failed

The head commit changed during the review from 2d869ac to f8e9537.

📝 Walkthrough

Walkthrough

The buildNewBookingData function in packages/features/bookings/lib/handleNewBooking/createBooking.ts was modified to update the logic for setting the iCalSequence property. Instead of assigning iCalSequence to evt.iCalSequence or defaulting to 0, it now conditionally increments the sequence by 1 if originalRescheduledBooking exists, setting iCalSequence to (evt.iCalSequence || 0) + 1; otherwise, it is set to 0. No other logic, control flow, or exported/public entities were changed.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Assessment against linked issues

Objective	Addressed	Explanation
Ensure iCalSequence is included and correctly incremented for rescheduled bookings (#22779, CAL-6164)	✅
Ensure default value for iCalSequence is set when not provided (#22779, CAL-6164)	✅

Assessment against linked issues: Out-of-scope changes

No out-of-scope changes detected.

Possibly related PRs

• fix: increment iCalSequence when changing booking location #22847: Modifies handling of iCalSequence in booking creation, updates, calendar event building, and related tests, directly relating to the addition of iCalSequence in booking data.

✨ Finishing Touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment

---

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

🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

• Review comments: Directly reply to a review comment made by CodeRabbit. Example:
  • I pushed a fix in commit <commit_id>, please review it.
  • Open a follow-up GitHub issue for this discussion.
• Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query.
• PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
  • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
  • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

CodeRabbit Commands (Invoked using PR/Issue comments)

Type @coderabbitai help to get the list of available commands.

Other keywords and placeholders

• Add @coderabbitai ignore or @coderabbit ignore anywhere in the PR description to prevent this PR from being reviewed.
• Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
• Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

• You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
• Please see the configuration documentation for more information.
• If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Status, Documentation and Community

• Visit our Status Page to check the current availability of CodeRabbit.
• Visit our Documentation for detailed information on how to use CodeRabbit.
• Join our Discord Community to get help, request features, and share feedback.
• Follow us on X/Twitter for updates and announcements.

[graphite-app]

Graphite Automations

"Add consumer team as reviewer" took an action on this PR • (08/05/25)

1 reviewer was added to this PR based on Keith Williams's automation.

"Add community label" took an action on this PR • (08/05/25)

1 label was added to this PR based on Keith Williams's automation.

[anikdhabal]

The issue is for reschedule booking not for new booking. Also need to increment the sequence on each reschedule. So iCalSequence: (evt.iCalSequence || 0) + 1,

[anikdhabal]

We have a function called getICalSequence inside the handleNewBooking file, where we handle the iCalSequence. Can you please check why it's not working and keep this change in a single place?

[Devanshusharma2005]

Sure. Will look at it.

[github-actions]

This PR is being marked as stale due to inactivity.

[anikdhabal]

Looks good

[github-actions]

E2E results are ready!

• Workflow #74920.1 latest results