Feature/voc 14 post to linkedin

[FSS3096]

Summary

Type of Change

• 🐛 Bug fix

• - [ ] ✨ New feature

• - [ ] 🔨 Refactor / tech debt

• - [ ] 📝 Documentation update

• - [ ] 🚀 Performance improvement

• - [ ] 🔧 Config / tooling change

• ## What Changed

• -

• -

• ## Testing

• - [ ] Manual testing done

• - [ ] Existing tests pass

• - [ ] New tests added (if applicable)

• ## Screenshots / Demo

• ## Checklist

• - [ ] Self-reviewed the code

• - [ ] No console errors or warnings

• - [ ] PR title follows type: short description convention

• - [ ] Linked to relevant issue (closes #)

• - [ ] Ready for review (not a draft)

[AradhyaTiwari10]

A few issues to address before merging.

Issues to address

1. components/DraftCard.tsx

useEffect(() => {
if (content !== localContent) {
setLocalContent(content);
}
}, [content]);

localContent is read inside the effect but is not included in the dependency array. Update the effect so it follows the React Hooks dependency rules.

2. components/GenerateButton.tsx

const data: any = await res.json().catch(() => ({}))
catch (err: any)

Replace any with unknown in both places and narrow the types before use.

3. RepoSelector.tsx

The test repo entries are added unconditionally after every successful fetch. Gate this block so it only runs when process.env.NODE_ENV !== 'production'.

4. pages/api/generate.ts

The trigger-auth-expired override runs before the session check. Move it to after the session guard.

5. components/DraftCard.tsx

const handleLinkedInShare = async () => {
try {
await navigator.clipboard.writeText(localContent);
} catch (err) { ... }
window.open(url, '_blank', 'noopener,noreferrer');
};

window.open is called after an await, which means the call no longer happens directly within the original user interaction. The popup may be blocked in browsers such as Safari. Consider opening the window before the asynchronous clipboard operation, or restructuring the handler so window.open runs within the synchronous part of the user event.