Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add output to current locations and block_hash to inscriptions #365

Merged
merged 1 commit into from
Jun 17, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
22 changes: 22 additions & 0 deletions migrations/1718498685557_inscriptions-block-hash.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,22 @@
/* eslint-disable @typescript-eslint/naming-convention */
import { MigrationBuilder, ColumnDefinitions } from 'node-pg-migrate';

export const shorthands: ColumnDefinitions | undefined = undefined;

export function up(pgm: MigrationBuilder): void {
pgm.addColumn('inscriptions', {
block_hash: {
type: 'text',
},
});
pgm.sql(`
UPDATE inscriptions SET block_hash = (
SELECT block_hash FROM locations AS l WHERE l.ordinal_number = ordinal_number LIMIT 1
)
`);
pgm.alterColumn('inscriptions', 'block_hash', { notNull: true });
}

export function down(pgm: MigrationBuilder): void {
pgm.dropColumn('inscriptions', 'block_hash');
}
26 changes: 26 additions & 0 deletions migrations/1718498703916_current-locations-output.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,26 @@
/* eslint-disable @typescript-eslint/naming-convention */
import { MigrationBuilder, ColumnDefinitions } from 'node-pg-migrate';

export const shorthands: ColumnDefinitions | undefined = undefined;

export function up(pgm: MigrationBuilder): void {
pgm.addColumn('current_locations', {
output: {
type: 'text',
},
});
pgm.sql(`
UPDATE current_locations SET output = (
SELECT output FROM locations AS l
WHERE l.ordinal_number = ordinal_number
AND l.block_height = block_height
AND l.tx_index = tx_index
LIMIT 1
)
`);
pgm.alterColumn('current_locations', 'output', { notNull: true });
}

export function down(pgm: MigrationBuilder): void {
pgm.dropColumn('current_locations', 'output');
}
9 changes: 7 additions & 2 deletions src/pg/block-cache.ts
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,14 @@ export class BlockCache {
const recursive_refs = getInscriptionRecursion(reveal.content_bytes);
const content_type = removeNullBytes(reveal.content_type);
const mime_type = content_type.split(';')[0];
const output = `${satpoint.tx_id}:${satpoint.vout}`;
this.inscriptions.push({
genesis_id: reveal.inscription_id,
mime_type,
content_type,
content_length: reveal.content_length,
block_height: this.blockHeight,
block_hash: this.blockHash,
tx_index: reveal.tx_index,
address: reveal.inscriber_address ?? '',
number: reveal.inscription_number.jubilee,
Expand Down Expand Up @@ -87,7 +89,7 @@ export class BlockCache {
tx_index: reveal.tx_index,
ordinal_number,
address: reveal.inscriber_address ?? '',
output: `${satpoint.tx_id}:${satpoint.vout}`,
output,
offset: satpoint.offset ?? null,
prev_output: null,
prev_offset: null,
Expand All @@ -99,6 +101,7 @@ export class BlockCache {
ordinal_number,
block_height: this.blockHeight,
tx_index: reveal.tx_index,
output,
address: reveal.inscriber_address ?? '',
});
if (recursive_refs.length > 0) this.recursiveRefs.set(reveal.inscription_id, recursive_refs);
Expand All @@ -109,14 +112,15 @@ export class BlockCache {
const prevSatpoint = parseSatPoint(transfer.satpoint_pre_transfer);
const ordinal_number = transfer.ordinal_number.toString();
const address = transfer.destination.value ?? '';
const output = `${satpoint.tx_id}:${satpoint.vout}`;
this.locations.push({
block_hash: this.blockHash,
block_height: this.blockHeight,
tx_id,
tx_index: transfer.tx_index,
ordinal_number,
address,
output: `${satpoint.tx_id}:${satpoint.vout}`,
output,
offset: satpoint.offset ?? null,
prev_output: `${prevSatpoint.tx_id}:${prevSatpoint.vout}`,
prev_offset: prevSatpoint.offset ?? null,
Expand All @@ -132,6 +136,7 @@ export class BlockCache {
ordinal_number,
block_height: this.blockHeight,
tx_index: transfer.tx_index,
output,
address,
});
}
Expand Down
12 changes: 6 additions & 6 deletions src/pg/pg-store.ts
Original file line number Diff line number Diff line change
Expand Up @@ -330,9 +330,9 @@ export class PgStore extends BasePgStore {
if (cache.currentLocations.size) {
for (const ordinal_number of moved_sats) {
await sql`
INSERT INTO current_locations (ordinal_number, block_height, tx_index, address)
INSERT INTO current_locations (ordinal_number, block_height, tx_index, output, address)
(
SELECT ordinal_number, block_height, tx_index, address
SELECT ordinal_number, block_height, tx_index, output, address
FROM locations
WHERE ordinal_number = ${ordinal_number}
ORDER BY block_height DESC, tx_index DESC
Expand Down Expand Up @@ -545,9 +545,9 @@ export class PgStore extends BasePgStore {
: sql``
}
${
/*filters?.genesis_block_hash
? sql`AND gen_l.block_hash = ${filters.genesis_block_hash}`
:*/ sql``
filters?.genesis_block_hash
? sql`AND i.block_hash = ${filters.genesis_block_hash}`
: sql``
}
${
filters?.from_genesis_block_height
Expand Down Expand Up @@ -600,7 +600,7 @@ export class PgStore extends BasePgStore {
${
filters?.mime_type?.length ? sql`AND i.mime_type IN ${sql(filters.mime_type)}` : sql``
}
${/*filters?.output ? sql`AND cur_l.output = ${filters.output}` : */ sql``}
${filters?.output ? sql`AND cur.output = ${filters.output}` : sql``}
${filters?.sat_rarity?.length ? sql`AND s.rarity IN ${sql(filters.sat_rarity)}` : sql``}
${filters?.sat_ordinal ? sql`AND i.ordinal_number = ${filters.sat_ordinal}` : sql``}
${
Expand Down
2 changes: 2 additions & 0 deletions src/pg/types.ts
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ export type DbInscriptionInsert = {
number: number;
classic_number: number;
block_height: number;
block_hash: string;
tx_index: number;
address: string;
mime_type: string;
Expand Down Expand Up @@ -48,6 +49,7 @@ export type DbCurrentLocationInsert = {
ordinal_number: PgNumeric;
block_height: number;
tx_index: number;
output: string;
address: string;
};

Expand Down
4 changes: 2 additions & 2 deletions tests/api/inscriptions.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2318,7 +2318,7 @@ describe('/inscriptions', () => {
expect(responseJson4.results[1].genesis_block_height).toBe(778575);
});

test.skip('index filtered by block hash', async () => {
test('index filtered by block hash', async () => {
await db.updateInscriptions(
new TestChainhookPayloadBuilder()
.apply()
Expand Down Expand Up @@ -2794,7 +2794,7 @@ describe('/inscriptions', () => {
expect(responseJson3.results[0].number).toBe(0);
});

test.skip('index filtered by output', async () => {
test('index filtered by output', async () => {
await db.updateInscriptions(
new TestChainhookPayloadBuilder()
.apply()
Expand Down
Loading