starred/pgtui

Fork 0

mirror of https://github.com/pgplex/pgtui.git synced 2026-04-26 18:45:57 +03:00

[PR #5] [MERGED] fix(data): display UUID values in standard format #4

New issue

Closed

opened 2026-03-02 15:50:22 +03:00 by kerem · 0 comments

kerem commented

2026-03-02 15:50:22 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/pgplex/pgtui/pull/5
Author: @rebelice
Created: 2/6/2026
Status: ✅ Merged
Merged: 2/6/2026
Merged by: @rebelice

Base: main ← Head: fix/uuid-display

📝 Commits (2)

8093dec fix(data): display UUID values in standard format
6178b11 fix(lint): handle errcheck for TogglePin in tests

📊 Changes

3 files changed (+9 additions, -3 deletions)

View changed files

📝 internal/db/metadata/data.go (+3 -0)
📝 internal/db/query/executor.go (+3 -0)
📝 internal/ui/components/table_view_pin_test.go (+3 -3)

📄 Description

Summary

Format PostgreSQL UUID columns as xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx instead of raw byte arrays
Handle [16]byte type returned by pgx driver in both query executor and metadata data layer

Closes #4

Test plan

Create a table with UUID columns and verify values display correctly
Verify existing data types (text, integer, JSONB, etc.) are unaffected

🤖 Generated with Claude Code

Summary by CodeRabbit

Bug Fixes
- Improved handling of PostgreSQL UUID values so they are consistently recognized and formatted as standard UUID strings in query results.
Tests
- Minor test adjustments to suppress returned errors in pin-related table view tests (no behavior change for end users).

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/pgplex/pgtui/pull/5 **Author:** [@rebelice](https://github.com/rebelice) **Created:** 2/6/2026 **Status:** ✅ Merged **Merged:** 2/6/2026 **Merged by:** [@rebelice](https://github.com/rebelice) **Base:** `main` ← **Head:** `fix/uuid-display` --- ### 📝 Commits (2) - [`8093dec`](https://github.com/pgplex/pgtui/commit/8093decf44882d8db2beab8c7e3cfde6772e578f) fix(data): display UUID values in standard format - [`6178b11`](https://github.com/pgplex/pgtui/commit/6178b119c029f6cf70101b7c1f2de3f27578dbc6) fix(lint): handle errcheck for TogglePin in tests ### 📊 Changes **3 files changed** (+9 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `internal/db/metadata/data.go` (+3 -0) 📝 `internal/db/query/executor.go` (+3 -0) 📝 `internal/ui/components/table_view_pin_test.go` (+3 -3) </details> ### 📄 Description <img width="1214" height="426" alt="CleanShot 2026-02-06 at 17 59 06@2x" src="https://github.com/user-attachments/assets/9ef6abfd-ab2c-469a-a288-0ee21bd55234" /> ## Summary - Format PostgreSQL UUID columns as `xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx` instead of raw byte arrays - Handle `[16]byte` type returned by pgx driver in both query executor and metadata data layer Closes #4 ## Test plan - [x] Create a table with UUID columns and verify values display correctly - [x] Verify existing data types (text, integer, JSONB, etc.) are unaffected 🤖 Generated with [Claude Code](https://claude.com/claude-code)  ## Summary by CodeRabbit * **Bug Fixes** * Improved handling of PostgreSQL UUID values so they are consistently recognized and formatted as standard UUID strings in query results. * **Tests** * Minor test adjustments to suppress returned errors in pin-related table view tests (no behavior change for end users).  --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

kerem

2026-03-02 15:50:22 +03:00

closed this issue
added the
pull-request
label

No labels

pull-request

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/pgtui#4

No description provided.

Rows
Columns