Remove host label from git diff screen
This commit is contained in:
@@ -82,7 +82,6 @@ export default function GitDiffScreen() {
|
|||||||
<GitDiffContent
|
<GitDiffContent
|
||||||
serverId={session.serverId}
|
serverId={session.serverId}
|
||||||
agentId={agentId}
|
agentId={agentId}
|
||||||
serverLabel={serverLabel}
|
|
||||||
/>
|
/>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
@@ -90,11 +89,9 @@ export default function GitDiffScreen() {
|
|||||||
function GitDiffContent({
|
function GitDiffContent({
|
||||||
serverId,
|
serverId,
|
||||||
agentId,
|
agentId,
|
||||||
serverLabel,
|
|
||||||
}: {
|
}: {
|
||||||
serverId: string;
|
serverId: string;
|
||||||
agentId?: string;
|
agentId?: string;
|
||||||
serverLabel: string;
|
|
||||||
}) {
|
}) {
|
||||||
const [isLoading, setIsLoading] = useState(true);
|
const [isLoading, setIsLoading] = useState(true);
|
||||||
const hasRequestedRef = useRef<string | null>(null);
|
const hasRequestedRef = useRef<string | null>(null);
|
||||||
@@ -143,7 +140,6 @@ function GitDiffContent({
|
|||||||
return (
|
return (
|
||||||
<View style={styles.container}>
|
<View style={styles.container}>
|
||||||
<BackHeader title="Changes" />
|
<BackHeader title="Changes" />
|
||||||
<Text style={styles.metaText}>Host: {serverLabel}</Text>
|
|
||||||
<View style={styles.errorContainer}>
|
<View style={styles.errorContainer}>
|
||||||
<Text style={styles.errorText}>Agent not found</Text>
|
<Text style={styles.errorText}>Agent not found</Text>
|
||||||
</View>
|
</View>
|
||||||
@@ -158,7 +154,6 @@ function GitDiffContent({
|
|||||||
return (
|
return (
|
||||||
<View style={styles.container}>
|
<View style={styles.container}>
|
||||||
<BackHeader title="Changes" />
|
<BackHeader title="Changes" />
|
||||||
<Text style={styles.metaText}>Host: {serverLabel}</Text>
|
|
||||||
|
|
||||||
<ScrollView style={styles.scrollView} contentContainerStyle={styles.contentContainer}>
|
<ScrollView style={styles.scrollView} contentContainerStyle={styles.contentContainer}>
|
||||||
{isLoading ? (
|
{isLoading ? (
|
||||||
@@ -232,7 +227,6 @@ function SessionUnavailableState({
|
|||||||
return (
|
return (
|
||||||
<View style={styles.container}>
|
<View style={styles.container}>
|
||||||
<BackHeader title="Changes" />
|
<BackHeader title="Changes" />
|
||||||
<Text style={styles.metaText}>Host: {serverLabel}</Text>
|
|
||||||
<View style={styles.sessionStateContainer}>
|
<View style={styles.sessionStateContainer}>
|
||||||
{isConnecting ? (
|
{isConnecting ? (
|
||||||
<>
|
<>
|
||||||
@@ -269,12 +263,6 @@ const styles = StyleSheet.create((theme) => ({
|
|||||||
paddingHorizontal: theme.spacing[6],
|
paddingHorizontal: theme.spacing[6],
|
||||||
gap: theme.spacing[3],
|
gap: theme.spacing[3],
|
||||||
},
|
},
|
||||||
metaText: {
|
|
||||||
paddingHorizontal: theme.spacing[6],
|
|
||||||
marginBottom: theme.spacing[2],
|
|
||||||
color: theme.colors.mutedForeground,
|
|
||||||
fontSize: theme.fontSize.xs,
|
|
||||||
},
|
|
||||||
scrollView: {
|
scrollView: {
|
||||||
flex: 1,
|
flex: 1,
|
||||||
},
|
},
|
||||||
|
|||||||
@@ -88,10 +88,11 @@ Improvements to the new agent screen in the app.
|
|||||||
- Fix to handle both absolute and relative paths correctly.
|
- Fix to handle both absolute and relative paths correctly.
|
||||||
- **Done (2025-12-29 00:15)**: WHAT: `packages/app/src/components/agent-stream-view.tsx:118` now normalizes inline paths using the agent cwd; `packages/app/src/components/agent-stream-view.tsx:455` adds path normalization helpers to convert absolute paths within the cwd to relative form before file explorer navigation. RESULT: absolute paths pointing at the agent cwd resolve to the correct directory/file in the explorer. EVIDENCE: Not run (not requested).
|
- **Done (2025-12-29 00:15)**: WHAT: `packages/app/src/components/agent-stream-view.tsx:118` now normalizes inline paths using the agent cwd; `packages/app/src/components/agent-stream-view.tsx:455` adds path normalization helpers to convert absolute paths within the cwd to relative form before file explorer navigation. RESULT: absolute paths pointing at the agent cwd resolve to the correct directory/file in the explorer. EVIDENCE: Not run (not requested).
|
||||||
|
|
||||||
- [ ] **Iteration 9**: Remove host label from git diff screen.
|
- [x] **Iteration 9**: Remove host label from git diff screen.
|
||||||
|
|
||||||
- The host label is shown on the git diff screen but it's not relevant there.
|
- The host label is shown on the git diff screen but it's not relevant there.
|
||||||
- Remove it from the UI.
|
- Remove it from the UI.
|
||||||
|
- **Done (2025-12-29 00:16)**: WHAT: `packages/app/src/app/git-diff.tsx:81-210` removes the Host label from the git diff header and error state; `packages/app/src/app/git-diff.tsx:253-307` drops the unused `metaText` style. RESULT: git diff screen no longer shows the host label. EVIDENCE: Not run (not requested).
|
||||||
|
|
||||||
- [ ] **Checkpoint**: Run typecheck and review code quality.
|
- [ ] **Checkpoint**: Run typecheck and review code quality.
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user