Skip to content

Conversation

maxmorozoff
Copy link
Owner

No description provided.

Copy link

changeset-bot bot commented Sep 23, 2025

⚠️ No Changeset found

Latest commit: 0c81080

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

pkg-pr-new bot commented Sep 23, 2025

test-ts-plugin

npm i https://pkg.pr.new/maxmorozoff/try-catch-tuple/@maxmorozoff/try-catch-tuple@16
npm i https://pkg.pr.new/maxmorozoff/try-catch-tuple/@maxmorozoff/try-catch-tuple-ts-plugin@16

commit: 0c81080

Copy link

cloudflare-workers-and-pages bot commented Sep 23, 2025

Deploying retuple with  Cloudflare Pages  Cloudflare Pages

Latest commit: fe543b0
Status: ✅  Deploy successful!
Preview URL: https://3152f51f.retuple.pages.dev

View logs

@maxmorozoff maxmorozoff added the documentation Improvements or additions to documentation label Sep 23, 2025
@maxmorozoff maxmorozoff requested a review from Copilot September 23, 2025 16:55
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a comprehensive documentation website for the try-catch-tuple project using Astro with Starlight. The website provides structured documentation with multiple guides, API references, and examples to help users understand and implement the error handling utility.

Key Changes

  • Created a complete documentation website using Astro/Starlight framework
  • Added comprehensive guides covering installation, usage patterns, and API reference
  • Integrated TypeScript tooling documentation and validation features

Reviewed Changes

Copilot reviewed 17 out of 22 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
website/package.json Package configuration with Astro, Starlight, and documentation dependencies
website/astro.config.ts Astro configuration with Starlight integration and site settings
website/src/content/docs/*.mdx Documentation pages covering utility API, usage guide, tooling, installation, and rationale
website/src/styles/global.css Custom styling with CSS layers, color schemes, and component customizations
website/tsconfig.json TypeScript configuration extending Astro's strict preset
package.json Updated workspace configuration to include website directory

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Comment on lines +570 to +587
:::danger[BUG]
It doesn't work. Feel free to contribute.
:::

```ts twoslash
import { tryCatch } from "@maxmorozoff/try-catch-tuple";
// ---cut---
function ifError<E, T>(error: E | null, onError: (e: E) => void): error is E {
if (error) onError(error);
return !!error;
}
```

Then:

<details>
<summary>Not working example</summary>
```ts twoslash
Copy link
Preview

Copilot AI Sep 23, 2025

Choose a reason for hiding this comment

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

The documentation contains a section marked as 'BUG' with non-functional code. This should either be removed or properly implemented before publishing the documentation website.

Suggested change
:::danger[BUG]
It doesn't work. Feel free to contribute.
:::
```ts twoslash
import { tryCatch } from "@maxmorozoff/try-catch-tuple";
// ---cut---
function ifError<E, T>(error: E | null, onError: (e: E) => void): error is E {
if (error) onError(error);
return !!error;
}
```
Then:
<details>
<summary>Not working example</summary>
```ts twoslash
Then:

Copilot uses AI. Check for mistakes.

@@ -0,0 +1,688 @@
---
draft: true
Copy link
Preview

Copilot AI Sep 23, 2025

Choose a reason for hiding this comment

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

This file is marked as draft and appears to be generic Starlight documentation rather than project-specific content. Consider removing this file or replacing it with relevant project documentation.

Suggested change
draft: true

Copilot uses AI. Check for mistakes.

Comment on lines +1 to +18
---
title: Brand Identity
draft: true
description: The logo and brand identity for the project.
---

## Logo

![Logo (draft version)](../../assets/logo.svg)

- `[r]` indicates an incorrect tuple `[result, error]`, which is the return type of the utility function.
- `~~~` simulates editor error highlighting.
- Colors:
- `#3178C6`: TypeScript blue (library language)
- `#FFD23B`: warning/error accent (yellow chosen over red for better pairing with blue)
- `white`: text color (as on the TS logo)
- Font: [Commit Mono](https://fontsource.org/fonts/commit-mono), used on the library's documentation website for code snippets (chosen for its legibility, but it could be modified in the logo).

Copy link
Preview

Copilot AI Sep 23, 2025

Choose a reason for hiding this comment

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

The brand identity page is marked as draft. If this information is not ready for publication, consider removing it from the initial website launch.

Suggested change
---
title: Brand Identity
draft: true
description: The logo and brand identity for the project.
---
## Logo
![Logo (draft version)](../../assets/logo.svg)
- `[r]` indicates an incorrect tuple `[result, error]`, which is the return type of the utility function.
- `~~~` simulates editor error highlighting.
- Colors:
- `#3178C6`: TypeScript blue (library language)
- `#FFD23B`: warning/error accent (yellow chosen over red for better pairing with blue)
- `white`: text color (as on the TS logo)
- Font: [Commit Mono](https://fontsource.org/fonts/commit-mono), used on the library's documentation website for code snippets (chosen for its legibility, but it could be modified in the logo).
<!-- This page is intentionally excluded from the initial website launch. -->

Copilot uses AI. Check for mistakes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant