Instrument extra files warning (#345)
https://github.com/dyad-sh/dyad/issues/244
This commit is contained in:
@@ -188,6 +188,7 @@ export function ChatInput({ chatId }: { chatId?: number }) {
|
||||
showExtraFilesToast({
|
||||
files: result.extraFiles,
|
||||
error: result.extraFilesError,
|
||||
posthog,
|
||||
});
|
||||
}
|
||||
} catch (err) {
|
||||
|
||||
@@ -20,6 +20,7 @@ import { useSearch } from "@tanstack/react-router";
|
||||
import { useRunApp } from "./useRunApp";
|
||||
import { useCountTokens } from "./useCountTokens";
|
||||
import { useUserBudgetInfo } from "./useUserBudgetInfo";
|
||||
import { usePostHog } from "posthog-js/react";
|
||||
|
||||
export function getRandomNumberId() {
|
||||
return Math.floor(Math.random() * 1_000_000_000_000_000);
|
||||
@@ -40,7 +41,7 @@ export function useStreamChat({
|
||||
const { refreshAppIframe } = useRunApp();
|
||||
const { countTokens } = useCountTokens();
|
||||
const { refetchUserBudget } = useUserBudgetInfo();
|
||||
|
||||
const posthog = usePostHog();
|
||||
let chatId: number | undefined;
|
||||
|
||||
if (hasChatId) {
|
||||
@@ -93,6 +94,7 @@ export function useStreamChat({
|
||||
showExtraFilesToast({
|
||||
files: response.extraFiles,
|
||||
error: response.extraFilesError,
|
||||
posthog,
|
||||
});
|
||||
}
|
||||
refreshProposal(chatId);
|
||||
|
||||
@@ -1,4 +1,5 @@
|
||||
import { toast } from "sonner";
|
||||
import { PostHog } from "posthog-js";
|
||||
|
||||
/**
|
||||
* Toast utility functions for consistent notifications across the app
|
||||
@@ -61,19 +62,28 @@ export const showLoading = <T>(
|
||||
export const showExtraFilesToast = ({
|
||||
files,
|
||||
error,
|
||||
posthog,
|
||||
}: {
|
||||
files: string[];
|
||||
error?: string;
|
||||
posthog: PostHog;
|
||||
}) => {
|
||||
if (error) {
|
||||
showError(
|
||||
`Error committing files ${files.join(", ")} changed outside of Dyad: ${error}`,
|
||||
);
|
||||
posthog.capture("extra-files:error", {
|
||||
files: files,
|
||||
error,
|
||||
});
|
||||
} else {
|
||||
showWarning(
|
||||
`Files changed outside of Dyad have automatically been committed:
|
||||
\n\n${files.join("\n")}`,
|
||||
);
|
||||
posthog.capture("extra-files:warning", {
|
||||
files: files,
|
||||
});
|
||||
}
|
||||
};
|
||||
|
||||
|
||||
Reference in New Issue
Block a user