Skip to content

feature: ipfs support#9

Merged
nipsysdev merged 4 commits intomainfrom
feature/ipfs_support
Jul 8, 2025
Merged

feature: ipfs support#9
nipsysdev merged 4 commits intomainfrom
feature/ipfs_support

Conversation

@nipsysdev
Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings July 8, 2025 18:41

This comment was marked as outdated.

@nipsysdev nipsysdev requested a review from Copilot July 8, 2025 19:08

This comment was marked as outdated.

@nipsysdev nipsysdev requested a review from Copilot July 8, 2025 19:56
Copy link
Copy Markdown
Contributor

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 support for displaying and truncating IPFS/IPNS hosts in the terminal prompt, updates related tests, and adjusts configuration and documentation for IPFS deployment.

  • Introduces getDisplayHost in TerminalPrompt to truncate long IPFS/IPNS hosts.
  • Expands TerminalPrompt tests to cover various host scenarios (regular, IPFS/IPNS, ports, and edge cases).
  • Updates next.config.ts, README.md, and development docs for IPFS deployment and testing requirements.

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
src/components/terminal/TerminalPrompt.tsx Add getDisplayHost logic and constants for host formatting
src/components/terminal/tests/TerminalPrompt.test.tsx Replace rigid matcher and add tests for IPFS/IPNS host display
next.config.ts Enable trailingSlash and disable image optimization for IPFS
README.md Change site title and mark IPFS deployment status
.ai/DEVELOPMENT.md Add a "Testing Requirements" section
Comments suppressed due to low confidence (1)

src/components/terminal/tests/TerminalPrompt.test.tsx:935

  • Add a test for a host string without any . characters to ensure the fallback branch (return fullHost) is exercised (edge case when parts.length <= 1).
  });

Comment thread src/components/terminal/TerminalPrompt.tsx
Comment thread src/components/terminal/TerminalPrompt.tsx
@nipsysdev nipsysdev merged commit 2290d56 into main Jul 8, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants