feat(types,medusa): add inventory quantity to products endpoint (#7541)

what:

- when inventory_quantity is requested through the API, we calculate the inventory based on sales channels + stock locations and return the total available inventory.

A variant can have multiple inventory items. As an example:

Table: (variant)
  - 4 (required_quantity via link) x legs (inventory item)
  - 2 x table top

Only if all individual inventory items of a variant are available, do we mark the variant as available as a single unit. 

RESOLVES CORE-2187
This commit is contained in:
Riqwan Thamir authored and GitHub committed 2024-05-30 13:20:06 +00:00
1 parent 4e20588522
commit 15e9787465
8 files changed
+409 -5

No files matched your search

@@ -1,5 +1,10 @@
import { ModuleRegistrationName } from "@medusajs/modules-sdk" import { ModuleRegistrationName } from "@medusajs/modules-sdk"
import { ApiKeyType, ProductStatus } from "@medusajs/utils" import {
ApiKeyType,
ContainerRegistrationKeys,
Modules,
ProductStatus,
} from "@medusajs/utils"
import { medusaIntegrationTestRunner } from "medusa-test-utils" import { medusaIntegrationTestRunner } from "medusa-test-utils"
import { createAdminUser } from "../../../../helpers/create-admin-user" import { createAdminUser } from "../../../../helpers/create-admin-user"
import { createDefaultRuleTypes } from "../../../helpers/create-default-rule-types" import { createDefaultRuleTypes } from "../../../helpers/create-default-rule-types"
@@ -89,6 +94,7 @@ medusaIntegrationTestRunner({
variants: [ variants: [
{ {
title: "test variant 1", title: "test variant 1",
manage_inventory: true,
prices: [{ amount: 3000, currency_code: "usd" }], prices: [{ amount: 3000, currency_code: "usd" }],
}, },
], ],
@@ -96,7 +102,9 @@ medusaIntegrationTestRunner({
;[product2, [variant2]] = await createProducts({ ;[product2, [variant2]] = await createProducts({
title: "test product 2 uniquely", title: "test product 2 uniquely",
status: ProductStatus.PUBLISHED, status: ProductStatus.PUBLISHED,
variants: [{ title: "test variant 2", prices: [] }], variants: [
{ title: "test variant 2", manage_inventory: false, prices: [] },
],
}) })
;[product3, [variant3]] = await createProducts({ ;[product3, [variant3]] = await createProducts({
title: "product not in price list", title: "product not in price list",
@@ -356,6 +364,215 @@ medusaIntegrationTestRunner({
]) ])
) )
}) })
describe("with inventory items", () => {
let location1
let location2
let inventoryItem1
let inventoryItem2
let salesChannel1
let publishableKey1
beforeEach(async () => {
location1 = (
await api.post(
`/admin/stock-locations`,
{ name: "test location" },
adminHeaders
)
).data.stock_location
location2 = (
await api.post(
`/admin/stock-locations`,
{ name: "test location 2" },
adminHeaders
)
).data.stock_location
salesChannel1 = await createSalesChannel(
{ name: "sales channel test" },
[product.id, product2.id]
)
const api1Res = await api.post(
`/admin/api-keys`,
{ title: "Test publishable KEY", type: ApiKeyType.PUBLISHABLE },
adminHeaders
)
publishableKey1 = api1Res.data.api_key
await api.post(
`/admin/api-keys/${publishableKey1.id}/sales-channels`,
{ add: [salesChannel1.id] },
adminHeaders
)
inventoryItem1 = (
await api.post(
`/admin/inventory-items`,
{ sku: "test-sku" },
adminHeaders
)
).data.inventory_item
inventoryItem2 = (
await api.post(
`/admin/inventory-items`,
{ sku: "test-sku-2" },
adminHeaders
)
).data.inventory_item
inventoryItem1 = (
await api.post(
`/admin/inventory-items/${inventoryItem1.id}/location-levels`,
{
location_id: location1.id,
stocked_quantity: 20,
},
adminHeaders
)
).data.inventory_item
inventoryItem2 = (
await api.post(
`/admin/inventory-items/${inventoryItem2.id}/location-levels`,
{
location_id: location2.id,
stocked_quantity: 30,
},
adminHeaders
)
).data.inventory_item
const remoteLink = appContainer.resolve(
ContainerRegistrationKeys.REMOTE_LINK
)
// TODO: Missing API endpoint. Remove this when its available
await remoteLink.create([
{
[Modules.SALES_CHANNEL]: { sales_channel_id: salesChannel1.id },
[Modules.STOCK_LOCATION]: { stock_location_id: location1.id },
},
{
[Modules.SALES_CHANNEL]: { sales_channel_id: salesChannel1.id },
[Modules.STOCK_LOCATION]: { stock_location_id: location2.id },
},
])
})
it("should list all inventory items for a variant", async () => {
const remoteLink = appContainer.resolve(
ContainerRegistrationKeys.REMOTE_LINK
)
// TODO: Missing API endpoint. Remove this when its available
await remoteLink.create([
{
[Modules.PRODUCT]: { variant_id: variant.id },
[Modules.INVENTORY]: { inventory_item_id: inventoryItem1.id },
data: { required_quantity: 20 },
},
{
[Modules.PRODUCT]: { variant_id: variant.id },
[Modules.INVENTORY]: { inventory_item_id: inventoryItem2.id },
data: { required_quantity: 20 },
},
])
let response = await api.get(
`/store/products?sales_channel_id[]=${salesChannel1.id}&fields=variants.inventory_items.inventory.location_levels.*`,
{ headers: { "x-publishable-api-key": publishableKey1.token } }
)
expect(response.status).toEqual(200)
expect(response.data.count).toEqual(2)
expect(response.data.products).toEqual(
expect.arrayContaining([
expect.objectContaining({
id: product.id,
variants: expect.arrayContaining([
expect.objectContaining({
inventory_items: expect.arrayContaining([
expect.objectContaining({
inventory_item_id: inventoryItem1.id,
}),
expect.objectContaining({
inventory_item_id: inventoryItem2.id,
}),
]),
}),
]),
}),
])
)
})
it("should return inventory quantity when variant's manage_inventory is true", async () => {
const remoteLink = appContainer.resolve(
ContainerRegistrationKeys.REMOTE_LINK
)
// TODO: Missing API endpoint. Remove this when its available
await remoteLink.create([
{
[Modules.PRODUCT]: { variant_id: variant.id },
[Modules.INVENTORY]: { inventory_item_id: inventoryItem1.id },
data: { required_quantity: 20 },
},
{
[Modules.PRODUCT]: { variant_id: variant.id },
[Modules.INVENTORY]: { inventory_item_id: inventoryItem2.id },
data: { required_quantity: 20 },
},
])
let response = await api.get(
`/store/products?sales_channel_id[]=${salesChannel1.id}&fields=%2bvariants.inventory_quantity`,
{
headers: { "x-publishable-api-key": publishableKey1.token },
}
)
const product1Res = response.data.products.find(
(p) => p.id === product.id
)
const product2Res = response.data.products.find(
(p) => p.id === product2.id
)
expect(response.status).toEqual(200)
expect(response.data.count).toEqual(2)
expect(product1Res).toEqual(
expect.objectContaining({
id: product.id,
variants: expect.arrayContaining([
expect.objectContaining({
inventory_quantity: 1,
manage_inventory: true,
}),
]),
})
)
expect(product2Res).toEqual(
expect.objectContaining({
id: product2.id,
variants: expect.arrayContaining([
expect.objectContaining({
manage_inventory: false,
}),
]),
})
)
expect(product2Res.variants[0].inventory_quantity).toEqual(
undefined
)
})
})
}) })
describe("GET /store/products/:id", () => { describe("GET /store/products/:id", () => {
@@ -88,6 +88,7 @@ export type InventoryItemDTO = {
created_at: string | Date created_at: string | Date
updated_at: string | Date updated_at: string | Date
deleted_at: string | Date | null deleted_at: string | Date | null
location_levels?: InventoryLevelDTO[]
} }
/** /**
@@ -195,6 +196,7 @@ export type InventoryLevelDTO = {
location_id: string location_id: string
stocked_quantity: number stocked_quantity: number
reserved_quantity: number reserved_quantity: number
available_quantity: number
incoming_quantity: number incoming_quantity: number
metadata: Record<string, unknown> | null metadata: Record<string, unknown> | null
created_at: string | Date created_at: string | Date
@@ -1,12 +1,22 @@
import { isPresent } from "@medusajs/utils" import { isPresent } from "@medusajs/utils"
import { MedusaRequest, MedusaResponse } from "../../../../types/routing" import { MedusaRequest, MedusaResponse } from "../../../../types/routing"
import { refetchProduct } from "../helpers" import { refetchProduct, wrapVariantsWithInventoryQuantity } from "../helpers"
import { StoreGetProductsParamsType } from "../validators" import { StoreGetProductsParamsType } from "../validators"
export const GET = async ( export const GET = async (
req: MedusaRequest<StoreGetProductsParamsType>, req: MedusaRequest<StoreGetProductsParamsType>,
res: MedusaResponse res: MedusaResponse
) => { ) => {
const withInventoryQuantity = req.remoteQueryConfig.fields.some((field) =>
field.includes("variants.inventory_quantity")
)
if (withInventoryQuantity) {
req.remoteQueryConfig.fields = req.remoteQueryConfig.fields.filter(
(field) => !field.includes("variants.inventory_quantity")
)
}
const filters: object = { const filters: object = {
id: req.params.id, id: req.params.id,
...req.filterableFields, ...req.filterableFields,
@@ -24,5 +34,9 @@ export const GET = async (
req.remoteQueryConfig.fields req.remoteQueryConfig.fields
) )
if (withInventoryQuantity) {
await wrapVariantsWithInventoryQuantity(req, product.variants || [])
}
res.json({ product }) res.json({ product })
} }
@@ -1,5 +1,11 @@
import { MedusaContainer } from "@medusajs/types" import { InventoryItemDTO, MedusaContainer } from "@medusajs/types"
import { refetchEntity } from "../../utils/refetch-entity" import {
ContainerRegistrationKeys,
MedusaError,
remoteQueryObjectFromString,
} from "@medusajs/utils"
import { MedusaRequest } from "../../../types/routing"
import { refetchEntities, refetchEntity } from "../../utils/refetch-entity"
export const refetchProduct = async ( export const refetchProduct = async (
idOrFilter: string | object, idOrFilter: string | object,
@@ -8,3 +14,117 @@ export const refetchProduct = async (
) => { ) => {
return await refetchEntity("product", idOrFilter, scope, fields) return await refetchEntity("product", idOrFilter, scope, fields)
} }
type VariantInventoryType = {
variant_id: string
variant: { manage_inventory: boolean }
required_quantity: number
inventory: InventoryItemDTO
}
export const wrapVariantsWithInventoryQuantity = async (
req: MedusaRequest,
variants: {
id: string
inventory_quantity?: number
manage_inventory?: boolean
}[]
) => {
const remoteQuery = req.scope.resolve(ContainerRegistrationKeys.REMOTE_QUERY)
const variantIds = variants.map((variant) => variant.id).flat(1)
if (!variantIds.length) {
return
}
if (!req.context?.stock_location_id?.length) {
throw new MedusaError(
MedusaError.Types.INVALID_DATA,
`Stock locations are required to compute inventory`
)
}
const linkQuery = remoteQueryObjectFromString({
entryPoint: "product_variant_inventory_item",
variables: {
filters: { variant_id: variantIds },
inventory: {
filters: {
location_levels: {
location_id: req.context?.stock_location_id || [],
},
},
},
},
fields: [
"variant_id",
"variant.manage_inventory",
"required_quantity",
"inventory.*",
"inventory.location_levels.*",
],
})
const links: VariantInventoryType[] = await remoteQuery(linkQuery)
const variantInventoriesMap = new Map<string, VariantInventoryType[]>()
links.forEach((link) => {
const array: VariantInventoryType[] =
variantInventoriesMap.get(link.variant_id) || []
array.push(link)
variantInventoriesMap.set(link.variant_id, array)
})
for (const variant of variants || []) {
if (!variant.manage_inventory) {
continue
}
const links = variantInventoriesMap.get(variant.id) || []
const inventoryQuantities: number[] = []
for (const link of links) {
const requiredQuantity = link.required_quantity
const availableQuantity = (link.inventory.location_levels || []).reduce(
(sum, level) => sum + level.available_quantity || 0,
0
)
// This will give us the maximum deliverable quantities for each inventory item
const maxInventoryQuantity = Math.floor(
availableQuantity / requiredQuantity
)
inventoryQuantities.push(maxInventoryQuantity)
}
// Since each of these inventory items need to be available to perform an order,
// we pick the smallest of the deliverable quantities as the total inventory quantity.
variant.inventory_quantity = inventoryQuantities.length
? Math.min(...inventoryQuantities)
: 0
}
}
export const maybeApplyStockLocationId = async (req: MedusaRequest, ctx) => {
const withInventoryQuantity = req.remoteQueryConfig.fields.some((field) =>
field.includes("variants.inventory_quantity")
)
if (!withInventoryQuantity) {
return
}
const salesChannelId = req.filterableFields.sales_channel_id || []
const entities = await refetchEntities(
"sales_channel_location",
{ sales_channel_id: salesChannelId },
req.scope,
["stock_location_id"]
)
return entities.map((entity) => entity.stock_location_id)
}
@@ -6,7 +6,9 @@ import {
filterByValidSalesChannels, filterByValidSalesChannels,
setPricingContext, setPricingContext,
} from "../../utils/middlewares" } from "../../utils/middlewares"
import { setContext } from "../../utils/middlewares/common/set-context"
import { validateAndTransformQuery } from "../../utils/validate-query" import { validateAndTransformQuery } from "../../utils/validate-query"
import { maybeApplyStockLocationId } from "./helpers"
import * as QueryConfig from "./query-config" import * as QueryConfig from "./query-config"
import { import {
StoreGetProductsParams, StoreGetProductsParams,
@@ -23,6 +25,9 @@ export const storeProductRoutesMiddlewares: MiddlewareRoute[] = [
QueryConfig.listProductQueryConfig QueryConfig.listProductQueryConfig
), ),
filterByValidSalesChannels(), filterByValidSalesChannels(),
setContext({
stock_location_id: maybeApplyStockLocationId,
}),
maybeApplyLinkFilter({ maybeApplyLinkFilter({
entryPoint: "product_sales_channel", entryPoint: "product_sales_channel",
resourceId: "product_id", resourceId: "product_id",
@@ -53,6 +58,9 @@ export const storeProductRoutesMiddlewares: MiddlewareRoute[] = [
QueryConfig.retrieveProductQueryConfig QueryConfig.retrieveProductQueryConfig
), ),
filterByValidSalesChannels(), filterByValidSalesChannels(),
setContext({
stock_location_id: maybeApplyStockLocationId,
}),
maybeApplyLinkFilter({ maybeApplyLinkFilter({
entryPoint: "product_sales_channel", entryPoint: "product_sales_channel",
resourceId: "product_id", resourceId: "product_id",
@@ -4,6 +4,7 @@ import {
remoteQueryObjectFromString, remoteQueryObjectFromString,
} from "@medusajs/utils" } from "@medusajs/utils"
import { MedusaRequest, MedusaResponse } from "../../../types/routing" import { MedusaRequest, MedusaResponse } from "../../../types/routing"
import { wrapVariantsWithInventoryQuantity } from "./helpers"
import { StoreGetProductsParamsType } from "./validators" import { StoreGetProductsParamsType } from "./validators"
export const GET = async ( export const GET = async (
@@ -12,6 +13,15 @@ export const GET = async (
) => { ) => {
const remoteQuery = req.scope.resolve(ContainerRegistrationKeys.REMOTE_QUERY) const remoteQuery = req.scope.resolve(ContainerRegistrationKeys.REMOTE_QUERY)
const context: object = {} const context: object = {}
const withInventoryQuantity = req.remoteQueryConfig.fields.some((field) =>
field.includes("variants.inventory_quantity")
)
if (withInventoryQuantity) {
req.remoteQueryConfig.fields = req.remoteQueryConfig.fields.filter(
(field) => !field.includes("variants.inventory_quantity")
)
}
if (isPresent(req.pricingContext)) { if (isPresent(req.pricingContext)) {
context["variants.calculated_price"] = { context["variants.calculated_price"] = {
@@ -31,6 +41,13 @@ export const GET = async (
const { rows: products, metadata } = await remoteQuery(queryObject) const { rows: products, metadata } = await remoteQuery(queryObject)
if (withInventoryQuantity) {
await wrapVariantsWithInventoryQuantity(
req,
products.map((product) => product.variants).flat(1)
)
}
res.json({ res.json({
products, products,
count: metadata.count, count: metadata.count,
@@ -0,0 +1,22 @@
import { NextFunction } from "express"
import { MedusaRequest } from "../../../../types/routing"
export function setContext(context: Record<string, any>) {
return async (req: MedusaRequest, _, next: NextFunction) => {
const ctx: Record<string, any> = { ...(req.context || {}) }
for (const [contextKey, contextValue] of Object.entries(context || {})) {
let valueToApply = contextValue
if (typeof contextValue === "function") {
valueToApply = await contextValue(req, ctx)
}
ctx[contextKey] = valueToApply
}
req.context = ctx
return next()
}
}
+4
View File
@@ -57,6 +57,10 @@ export interface MedusaRequest<Body = unknown>
* An object that carries the context that is used to calculate prices for variants * An object that carries the context that is used to calculate prices for variants
*/ */
pricingContext?: MedusaPricingContext pricingContext?: MedusaPricingContext
/**
* A generic context object that can be used across the request lifecycle
*/
context?: Record<string, any>
} }
export interface AuthContext { export interface AuthContext {