improve: add try-catch error handling

[Divine-P-77777]

Addressed Issues

Closes: #132

---

What Changed

Two small, focused changes in src/social-share-button.js:

1. createButton() — guard against invalid CSS selectors

document.querySelector() throws a DOMException (SyntaxError) when passed a malformed selector string. Previously, this would crash init() entirely, silently breaking modal creation and event attachment downstream.

• Wrapped querySelector in a try-catch
• Errors are surfaced using _debugWarn() when debug: true is enabled

---

2. fallbackCopy() — stop silently discarding errors

The existing catch block used _err (indicating an unused variable) and did not log anything.

• Renamed _err → error
• Added _debugWarn() logging
• Matches the existing error-handling pattern used in _emit()

---

Why

Both cases could lead to confusing runtime failures:

• Component crash due to invalid selector
• Silent failure in copy functionality

This update ensures:

• Errors are properly surfaced
• Debugging becomes easier
• Behavior aligns with existing error-handling patterns in the codebase

---

Screenshots / Recordings

---

Additional Notes

• No behavior change for valid selectors
• try-catch only triggers for malformed selectors (edge case)
• _debugWarn() logs only when debug: true is enabled
• No impact on production behavior

---

✅ Checklist

• My code follows the project's code style and conventions
• I have made corresponding changes to the documentation
• My changes generate no new warnings or errors
• I have joined the Discord server and shared this PR with maintainers
• I have read the Contributing Guidelines

---

⚠️ AI Notice - Important!

We encourage responsible use of AI tools when creating Pull Requests.

Ensure that:

• Your changes meet the task requirements
• The code builds successfully
• Relevant tests are included (if applicable)
• All linters pass

Low-quality or careless submissions may be closed without warning.

Do not submit AI-generated code without understanding it.
Spamming repositories with low-quality contributions may lead to bans.

Summary by CodeRabbit

• Bug Fixes
  • Enhanced error handling for the social share button to gracefully manage failures related to container selection and copy operations, improving overall stability and preventing crashes.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 6bd81193-7b11-488d-86c2-7944c7f7d044

📥 Commits

Reviewing files that changed from the base of the PR and between d0686c4 and 8ae8e69.

📒 Files selected for processing (1)

• src/social-share-button.js

---

Walkthrough

Error handling improvements added to src/social-share-button.js. Two methods now include try-catch blocks: createButton wraps container resolution to catch invalid CSS selectors, and fallbackCopy enhances its existing catch block with error variable binding and debug logging via _debugWarn.

Changes

Cohort / File(s)	Summary
Error handling enhancements src/social-share-button.js	Added try-catch block in createButton() around document.querySelector() to handle invalid container selectors; enhanced fallbackCopy() catch block by renaming parameter from _err to error and adding _debugWarn() logging before setting failure feedback.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

• fix: avoid empty catch blocks in analytics #99: Introduced the _debugWarn() helper function that is now utilized by the error handling additions in this PR.

Suggested labels

Typescript Lang

Suggested reviewers

• kpj2006

Poem

🐰 A bunny hops through error gates,
Catching issues before it's too late,
With debug logs to light the way,
No silent fails shall ruin the day! 🌟

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'improve: add try-catch error handling' clearly summarizes the main change: adding try-catch blocks for error handling in the social-share-button.js file.
Linked Issues check	✅ Passed	The PR fully implements all acceptance criteria from issue #132: try-catch added to createButton() with _debugWarn logging and fallbackCopy() error variable binding and logging.
Out of Scope Changes check	✅ Passed	All changes in src/social-share-button.js are directly related to the error handling improvements specified in issue #132; no out-of-scope modifications detected.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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.}

[github-actions]

Hello 👋 This PR has had no activity for more than 2 weeks. If you are still working on it, please push an update or leave a comment. Ping a maintainer if you believe it is ready for review or merge! This PR will be automatically closed in 7 days if there is no further activity.