bdougie / bdougie/codebunny

Fix type safety: commentId should be required, not optional

Open
#17 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
TypeScript
Stars
7
Forks
0
PR merge metrics
No merged PRs in 30d

Description

## Problem
The `commentId` field is marked as optional in the interface but is always provided in practice. This is a type safety issue that suggests incomplete refactoring.

## Solution
- If the field is always required in practice, update the interface to reflect this
- Remove optional marker and update all call sites accordingly
- Ensure type definitions match actual usage patterns

## Related
PR #2 review feedback

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.