Skip to content

itdove/devaiflow#358: Session names truncated in 'daf list' making sessions impossible to reopen on small terminals#359

Merged
itdove merged 2 commits intomainfrom
358
Apr 3, 2026
Merged

itdove/devaiflow#358: Session names truncated in 'daf list' making sessions impossible to reopen on small terminals#359
itdove merged 2 commits intomainfrom
358

Conversation

@itdove
Copy link
Copy Markdown
Owner

@itdove itdove commented Apr 2, 2026

This PR does not need a corresponding Jira item.

Description

This PR fixes an issue where session names were being truncated in the daf list command output on small terminals, making it impossible to identify and reopen sessions. The fix modifies the list command to display full session names regardless of terminal width, ensuring users can always see complete session information needed to reopen them.

The changes update the output formatting logic in the list command to prevent truncation while maintaining readability. Integration and unit tests have been added to verify the fix works across different terminal sizes.

Related GitHub Issue: #358

Assisted-by: Claude

Testing

Steps to test

  1. Pull down the PR
  2. Create multiple DevAIFlow sessions with long descriptive names
  3. Run daf list in a terminal with reduced width (e.g., resize terminal to ~80 columns or less)
  4. Verify that full session names are displayed without truncation
  5. Verify that session names are selectable/copyable for reopening sessions
  6. Run the integration test suite: ./integration-tests/test_investigation.sh
  7. Run the unit tests: pytest tests/test_list_command.py

Scenarios tested

  • List command output with various terminal widths (80, 100, 120+ columns)
  • Sessions with names of varying lengths (short, medium, long)
  • Multiple sessions displayed simultaneously in list view
  • Session reopening workflow using full names from list output

Deployment considerations

  • This code change is ready for deployment on its own
  • This code change requires the following considerations before being deployed:

- Add no_wrap=True to Name column in session list table
- Update integration tests to verify full session names instead of prefixes
- Update unit test assertions to check complete session names
- Remove truncation handling logic from test comments and assertions

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add prompt_session_selection() with Rich table UI
- Implement 25-item pagination with page navigation
- Display session metadata (workspace, issue, status, last activity)
- Highlight most recent session with green indicator

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@itdove itdove marked this pull request as ready for review April 3, 2026 00:57
@itdove itdove merged commit 4346410 into main Apr 3, 2026
7 checks passed
@itdove itdove deleted the 358 branch April 3, 2026 00:57
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.

1 participant