Trim another narrow batch of dialog cleanup noise (#317)
This follow-up pass continues the phased unused-code cleanup from issue #314 with four dialog components that shared the same low-risk pattern: avoidable unused imports, dormant Props aliases, and untyped callback plumbing that only existed as compiler noise. The changes keep behavior intact while reducing the next layer of cleanup friction. Constraint: Keep the second pass narrowly scoped and homogeneous so it stays easy to review beside PR #316 Rejected: Fold in EffortCallout and wider component cleanup at the same time | larger surface area and less uniform risk profile Confidence: high Scope-risk: narrow Reversibility: clean Directive: Continue batching future passes by shared cleanup pattern; leave broader refactors and compatibility placeholders for separate PRs Tested: bun run build; bun run smoke; targeted noUnused grep for touched files via bun x tsc --noEmit --noUnusedLocals --noUnusedParameters --pretty false Not-tested: full repo typecheck (broader baseline noise remains outside this pass) Co-authored-by: anandh8x <test@example.com>
This commit is contained in:
@@ -1,5 +1,5 @@
|
||||
import { c as _c } from "react-compiler-runtime";
|
||||
import React, { useCallback } from 'react';
|
||||
import React from 'react';
|
||||
import { logEvent } from 'src/services/analytics/index.js';
|
||||
import { Box, Link, Text } from '../ink.js';
|
||||
import type { ExternalClaudeMdInclude } from '../utils/claudemd.js';
|
||||
@@ -11,14 +11,14 @@ type Props = {
|
||||
isStandaloneDialog?: boolean;
|
||||
externalIncludes?: ExternalClaudeMdInclude[];
|
||||
};
|
||||
export function ClaudeMdExternalIncludesDialog(t0) {
|
||||
export function ClaudeMdExternalIncludesDialog(t0: Props) {
|
||||
const $ = _c(18);
|
||||
const {
|
||||
onDone,
|
||||
isStandaloneDialog,
|
||||
externalIncludes
|
||||
} = t0;
|
||||
let t1;
|
||||
let t1: [];
|
||||
if ($[0] === Symbol.for("react.memo_cache_sentinel")) {
|
||||
t1 = [];
|
||||
$[0] = t1;
|
||||
@@ -28,7 +28,7 @@ export function ClaudeMdExternalIncludesDialog(t0) {
|
||||
React.useEffect(_temp, t1);
|
||||
let t2;
|
||||
if ($[1] !== onDone) {
|
||||
t2 = value => {
|
||||
t2 = (value: 'yes' | 'no') => {
|
||||
if (value === "no") {
|
||||
logEvent("tengu_claude_md_external_includes_dialog_declined", {});
|
||||
saveCurrentProjectConfig(_temp2);
|
||||
@@ -94,7 +94,7 @@ export function ClaudeMdExternalIncludesDialog(t0) {
|
||||
}
|
||||
let t10;
|
||||
if ($[10] !== handleSelection) {
|
||||
t10 = <Select options={t9} onChange={value_0 => handleSelection(value_0 as 'yes' | 'no')} />;
|
||||
t10 = <Select options={t9} onChange={(value_0: string) => handleSelection(value_0 as 'yes' | 'no')} />;
|
||||
$[10] = handleSelection;
|
||||
$[11] = t10;
|
||||
} else {
|
||||
@@ -114,17 +114,17 @@ export function ClaudeMdExternalIncludesDialog(t0) {
|
||||
}
|
||||
return t11;
|
||||
}
|
||||
function _temp4(include, i) {
|
||||
function _temp4(include: ExternalClaudeMdInclude, i: number) {
|
||||
return <Text key={i} dimColor={true}>{" "}{include.path}</Text>;
|
||||
}
|
||||
function _temp3(current_0) {
|
||||
function _temp3(current_0: any) {
|
||||
return {
|
||||
...current_0,
|
||||
hasClaudeMdExternalIncludesApproved: true,
|
||||
hasClaudeMdExternalIncludesWarningShown: true
|
||||
};
|
||||
}
|
||||
function _temp2(current) {
|
||||
function _temp2(current: any) {
|
||||
return {
|
||||
...current,
|
||||
hasClaudeMdExternalIncludesApproved: false,
|
||||
|
||||
Reference in New Issue
Block a user