Initial repository setup with listing information and reservation request accept functionality by Copilot · Pull Request #258 · simnova/sharethrift

Copilot · 2025-11-05T12:09:34Z

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Summary by Sourcery

Add authenticated reservation request acceptance and overlapping request auto-rejection, expose it via GraphQL and wire it into the My Listings UI, while standardizing request states and improving data hydration and testing across layers.

New Features:

Introduce an update application service for reservation requests that supports accepting and closing flags, including overlap-aware auto-rejection logic.
Expose an acceptReservationRequest GraphQL mutation with authorization and listing ownership checks, and integrate it into the My Listings requests table UI with feedback and refetching.
Add My Listings dashboard wiring for authenticated sharers, including per-sharer requests count and sharer-scoped requests tab with dynamic action buttons.
Extend request listing GraphQL mapping to return real listing imagery and normalized request statuses, and support richer listing/request Storybook stories.

Bug Fixes:

Fix Mongo read repository population so populate operations run before lean and add missing reserver lookup in listing-requests-by-sharer aggregation to avoid hydration issues.
Correct mock data relationships for item listings and reservation requests to use valid sharer and reserver IDs, and adjust tests for resolver helper behaviors around invalid or non-ObjectId identifiers.
Ensure GraphQL server initialization tests assert depth-limit configuration more robustly without relying on specific function instances.

Enhancements:

Refine reservation request domain model to emit a ReservationRequestAcceptedEvent when a request is accepted, wiring through a new domain event type export.
Improve reservation request read and domain repositories with overlap query support and simplified hydration for listing and reserver relations.
Standardize reservation request status terminology (e.g., Requested vs Pending, Expired support) across UI helpers, filters, seed data, and Storybook stories, and add unit tests for status/action helpers.
Polish My Listings UI components with sharer-scoped data loading, safer error logging, improved button text and Ant Design App context usage, plus small code style and type tweaks across UI and GraphQL tests.

CI:

Relax exact depthLimit validation in GraphQL handler tests to assert on structure rather than specific function instances, keeping CI resilient to implementation changes.

Documentation:

Document reservation request update and acceptance behavior through new BDD-style .feature files for both application service and GraphQL resolver layers.

Tests:

Add cucumber-style tests for the reservation request update application service, including acceptance, close flags, overlap auto-rejection, and error cases.
Extend reservation request GraphQL resolver tests to cover acceptReservationRequest mutation behaviors and add feature definitions for these scenarios.
Add unit tests for request status helper functions and additional scenarios in resolver-helper and tracing tests to verify fallback and tracing behaviors.
Update Storybook stories and Apollo mocks for profile, requests card, and requests table to use the new schemas and status variants, and verify My Listings data flows.

…in listing-information component

dani-vaibhav · 2025-11-05T14:31:49Z

@copilot
@copilot /plan
@copilot /suggest
Implement the logic for handleAction function based on the following actions

Update UI (My Listings → Requests) to show dynamic actions based on status:
- Pending → Accept / Reject
- Accepted → Close / Message
- Rejected → Delete / Archive
- Expired → Archive

…tatus - Update action buttons to match status requirements: - Pending → Accept / Reject - Accepted → Close / Message - Rejected → Delete / Archive - Expired → Archive - Closed → Message - Cancelled → Delete - Implement handleAction function with switch statement for each action - Add Expired status to filter options and status tag styling - Update Storybook stories to demonstrate all status variants - Reorganize getActionButtons helper for clarity with inline comments Co-authored-by: dani-vaibhav <182140623+dani-vaibhav@users.noreply.github.com>

Copilot · 2025-11-05T14:42:10Z

@copilot
@copilot /plan
@copilot /suggest
...

Implemented dynamic action buttons based on reservation request status in commit 815cd1f.

Changes made:

Status → Actions mapping:

Pending → Accept / Reject
Accepted → Close / Message
Rejected → Delete / Archive
Expired → Archive
Closed → Message
Cancelled → Delete

Implementation details:

Updated getActionButtons helper in requests-status-helpers.tsx with clear status-based logic and inline comments
Implemented handleAction function in requests-table.container.tsx with switch statement for each action (accept, reject, close, message, delete, archive)
Added "Expired" status to filter options and status tag styling
Created Storybook stories demonstrating all status variants (Pending, Accepted, Rejected, Closed, Expired)

The UI now dynamically displays the appropriate action buttons based on the current status of each reservation request. The handleAction function is ready for future integration with GraphQL mutations.

dani-vaibhav · 2025-11-05T16:48:24Z

@copilot
@workspace /fix Implement acceptReservationRequest and implement the logic for the case 'accept'

- Create accept.ts application service for accepting reservation requests - Add acceptReservationRequest GraphQL mutation with input type - Implement mutation resolver with authentication check - Add GraphQL mutation to requests-table.container.graphql - Integrate useMutation hook in RequestsTableContainer - Implement handleAction 'accept' case with mutation call and refetch - Add loading state for accept action - Export accept service from reservation-request index - Pass biome linting checks Co-authored-by: dani-vaibhav <182140623+dani-vaibhav@users.noreply.github.com>

Copilot · 2025-11-05T16:59:24Z

@copilot
@workspace /fix Implement acceptReservationRequest and implement the logic for the case 'accept'

Implemented acceptReservationRequest mutation and integrated it with the UI in commit 8eba1a5.

Implementation details:

1. Application Service Layer (accept.ts):

Created ReservationRequestAcceptCommand interface with id and sharerEmail
Implemented accept() service that:
- Validates reservation request exists
- Verifies sharer owns the listing
- Calls domain state = 'Accepted' to trigger acceptance logic
- Persists changes via Unit of Work transaction

2. GraphQL Layer:

Added acceptReservationRequest mutation to schema with AcceptReservationRequestInput
Implemented resolver with authentication check using verifiedJwt.email
Returns updated ReservationRequest with new state

3. UI Layer (requests-table.container.tsx):

Added useMutation hook for acceptReservationRequest
Implemented handleAction 'accept' case:
- Calls mutation with request ID
- Auto-refetches list to show updated status
- Includes error handling with console logging
- Shows loading state during mutation execution

The accept button now fully functions - when clicked on a Pending request, it accepts the reservation and updates the status to Accepted in real-time.

…p and improved data handling

…gs and error handling

…nd improve image handling in reservation requests

dani-vaibhav · 2025-11-07T19:21:45Z

@copilot
Deploy this branch

Copilot · 2025-11-07T19:24:10Z

@copilot
Deploy this branch

I don't have the ability to deploy branches or trigger deployment pipelines. Deployment typically requires access to CI/CD systems, cloud infrastructure, and production credentials that I don't have.

To deploy this branch, you'll need to:

Merge this PR to your main/deployment branch, or
Manually trigger your CI/CD pipeline for this branch, or
Use your deployment tooling (e.g., Azure DevOps, GitHub Actions, etc.)

If you need help with code changes, configuration files, or deployment scripts that I can assist with, please let me know the specific changes needed.

Fixes TypeScript build error TS6133: 'useNavigate' is declared but its value is never read. The import was added but not used in the component, causing the strict TypeScript build to fail in CI/CD.

jasonmorais

Small comments for the most part

.../sthrift/application-services/src/contexts/reservation-request/reservation-request/update.ts

apps/ui-sharethrift/src/main.tsx

apps/ui-sharethrift/src/components/layouts/home/my-listings/components/requests-table.tsx

...-sharethrift/src/components/layouts/home/my-listings/components/requests-table.container.tsx

apps/ui-sharethrift/src/components/layouts/home/my-listings/components/requests-card.tsx

apps/ui-sharethrift/src/components/layouts/home/my-listings/components/all-listings-table.tsx

…Table component

…onsistency

…d action handling

…or consistency

…onfig

jasonmorais · 2026-01-06T16:56:28Z

...ui-sharethrift/src/components/layouts/home/my-listings/components/all-listings-table-row.tsx

Dont want a single component that exports multiple versions of the component, goes against our practices. Also the file names here arent following our naming conventions with dashes.

jasonmorais · 2026-01-06T16:56:50Z

...ui-sharethrift/src/components/layouts/home/my-listings/components/AllListingsTableHeader.tsx

Same idea here as the above comment

jasonmorais · 2026-01-06T16:57:10Z

...harethrift/src/components/layouts/home/my-listings/components/all-listings-table-actions.tsx

Same thing here with file name

jasonmorais · 2026-01-06T18:16:20Z

...-sharethrift/src/components/layouts/home/my-listings/components/requests-table.container.tsx

+	let displayRequests = transformedAllRequests;
+
+	// Apply sorting (keep existing behavior)
+	if (sorter.field && sorter.order) {


typically sorting and filtering and pagination we do on the backend

jasonmorais · 2026-01-06T18:17:21Z

...-sharethrift/src/components/layouts/home/my-listings/components/requests-table.container.tsx

+	const allRequests = data?.myListingsRequests ?? [];
+
+	// Transform domain fields to UI format (no client-side search/status filtering in this container)
+	const transformedAllRequests = allRequests.map((request) => {


Some of the handling here seems strange, like checking for type personaluser. I would determine if this is all needed or is an issue with just the interface mapping, as in we should adjust that rather than remap fully.

jasonmorais · 2026-01-06T18:19:11Z

apps/ui-sharethrift/src/components/layouts/home/my-listings/components/requests-table.tsx

 	onPageChange,
-	onAction,
+	onAccept,
+	onReject: _onReject,


Why underscores for the actual function names? It isnt the field itself so from what i understand it isnt making it optional. if it serves no actual purpose i would name them normally

…onfig

dani-vaibhav and others added 2 commits November 5, 2025 16:25

refactor: standardize import formatting and improve code readability …

7675212

…in listing-information component

Initial plan

0404a49

Copilot AI assigned Copilot and dani-vaibhav Nov 5, 2025

Copilot AI mentioned this pull request Nov 5, 2025

refactor: standardize import formatting and improve code readability in listing-information component #257

Closed

Copilot started work on behalf of dani-vaibhav November 5, 2025 12:10 View session

Copilot AI changed the title ~~[WIP] Refactor import formatting in listing information component~~ Initial repository setup with listing information component Nov 5, 2025

Copilot finished work on behalf of dani-vaibhav November 5, 2025 12:14

Copilot AI requested a review from dani-vaibhav November 5, 2025 12:14

Copilot started work on behalf of dani-vaibhav November 5, 2025 14:32 View session

Copilot AI changed the title ~~Initial repository setup with listing information component~~ Initial repository setup with listing information and dynamic reservation request actions Nov 5, 2025

Copilot finished work on behalf of dani-vaibhav November 5, 2025 14:43

Copilot started work on behalf of dani-vaibhav November 5, 2025 16:50 View session

Copilot AI changed the title ~~Initial repository setup with listing information and dynamic reservation request actions~~ Initial repository setup with listing information and reservation request accept functionality Nov 5, 2025

Copilot finished work on behalf of dani-vaibhav November 5, 2025 17:00

dani-vaibhav added 3 commits November 6, 2025 23:06

feat: enhance MyListingsDashboard and RequestsTable with sharerId pro…

1f8a9c4

…p and improved data handling

feat: enhance reservation request handling with improved state mappin…

c3b3d3c

…gs and error handling

feat: update MyListingsDashboard to use sharerId for requests count a…

fde5275

…nd improve image handling in reservation requests

Copilot started work on behalf of dani-vaibhav November 7, 2025 19:22 View session

Copilot finished work on behalf of dani-vaibhav November 7, 2025 19:26

dani-vaibhav marked this pull request as ready for review November 7, 2025 19:39

dani-vaibhav requested a review from arif-u-ahmed November 7, 2025 19:39

jasonmorais marked this pull request as draft December 18, 2025 20:21

dani-vaibhav added 4 commits December 19, 2025 11:54

fix : Data population

5354d77

Merge branch 'main' into copilot/sub-pr-257

97fe325

Merge branch 'main' into copilot/sub-pr-257

e762bfc

fix: remove unused useNavigate import

68170d9

Fixes TypeScript build error TS6133: 'useNavigate' is declared but its value is never read. The import was added but not used in the component, causing the strict TypeScript build to fail in CI/CD.

dani-vaibhav marked this pull request as ready for review December 22, 2025 16:30

dani-vaibhav requested a review from jasonmorais December 22, 2025 16:30

jasonmorais reviewed Dec 23, 2025

View reviewed changes

dani-vaibhav added 13 commits December 24, 2025 21:48

Merge branch 'main' into copilot/sub-pr-257

9a702a8

PR comments fix

5391e9a

refactor: modularize action buttons and cell rendering in AllListings…

35f019f

…Table component

fix: update reservation request states and properties for clarity

efb5d3d

fix: add pnpm overrides for qs package to resolve version conflicts

361292f

refactor: reorganize imports and improve code formatting for readability

0be20b3

fix: update environment variable access to use bracket notation for c…

003b0d7

…onsistency

refactor: streamline action rendering logic in RequestsTable component

b19d2e1

feat: implement AllListingsTable component with enhanced filtering an…

238bf27

…d action handling

fix: change STATUS_OPTIONS export to const for consistency

cc81763

fix: update environment variable access for admin and user profiles f…

59bbf18

…or consistency

Merge branch 'main' into copilot/sub-pr-257

4d77a7e

fix: correct pnpm configuration by consolidating overrides and auditC…

1f4a66f

…onfig

jasonmorais reviewed Jan 6, 2026

View reviewed changes

dani-vaibhav added 8 commits January 7, 2026 23:32

Merge branch 'main' into copilot/sub-pr-257

558a52e

feat: add listing title and status filter components

175e460

feat: refactor listings table components and remove unused files

7996028

refactor: clean up requests table components for improved readability

2fe1243

refactor: clean up requests table components for improved readability

5683e61

fix: correct pnpm configuration by consolidating overrides and auditC…

e792066

…onfig

Merge branch 'main' into copilot/sub-pr-257

7435bfd

ProfilePage updated

6c51ae6

Conversation

Copilot AI commented Nov 5, 2025 • edited by sourcery-ai bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary by Sourcery

Uh oh!

dani-vaibhav commented Nov 5, 2025

Uh oh!

Copilot AI commented Nov 5, 2025

Uh oh!

dani-vaibhav commented Nov 5, 2025

Uh oh!

Copilot AI commented Nov 5, 2025

Uh oh!

dani-vaibhav commented Nov 7, 2025

Uh oh!

Copilot AI commented Nov 7, 2025

Uh oh!

jasonmorais left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

jasonmorais Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

jasonmorais Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

jasonmorais Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

jasonmorais Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

jasonmorais Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

jasonmorais Jan 6, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Copilot AI commented Nov 5, 2025 •

edited by sourcery-ai bot

Loading