Skip to content

Commit 45b93f0

Browse files
committed
Update review process to compare against dev branch; refactor Tailwind CSS configuration to use ES module syntax; enhance user lock polling logic; improve PostHog initialization; remove unused API endpoints for checkpoint and consensus RPC.
1 parent 6f2868c commit 45b93f0

6 files changed

Lines changed: 14 additions & 69 deletions

File tree

.claude/commands/reviewchanges.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ name: code-review
33
description: Entropy-reducing code review. Diff-anchored but context-aware. Favors deletion, consolidation, and simplification over additive fixes.
44
---
55

6-
> **TEMPORARY RULE**: Always compare against `main-v2` branch (not `main` or `dev`). Use `git diff main-v2...HEAD` for all diffs.
6+
> **TEMPORARY RULE**: Always compare against `dev` branch. Use `git diff dev...HEAD` for all diffs. Do not do any changes, review only.
77
88
## How This Review Works
99

apps/app/hooks/htlc/useUserLockPolling.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ const useUserLockPolling = ({
2727
}: UseUserLockPollingParams) => {
2828
const type: 'erc20' | 'native' = sourceAsset?.contractAddress && sourceAsset.contractAddress !== '0x0000000000000000000000000000000000000000' ? 'erc20' : 'native'
2929

30-
const shouldPoll = !!(network && hashlock && contractAddress && enabled)
30+
const shouldPoll = !!(network && hashlock && contractAddress && enabled && client)
3131

3232
const key = shouldPoll
3333
? `/htlc/userLock/${network!.caip2Id}/${hashlock}/${contractAddress}/${type}`

apps/app/pages/_app.js

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -32,11 +32,13 @@ Router.events.on("routeChangeComplete", progress.finish);
3232
Router.events.on("routeChangeError", progress.finish);
3333

3434
const INTERCOM_APP_ID = 'h5zisg78'
35+
const posthogKey = process.env.NEXT_PUBLIC_POSTHOG_KEY;
36+
const posthogHost = process.env.NEXT_PUBLIC_POSTHOG_HOST || 'https://us.i.posthog.com';
3537

3638
// Check that PostHog is client-side (used to handle Next.js SSR)
37-
if (typeof window !== 'undefined') {
38-
posthog.init(process.env.NEXT_PUBLIC_POSTHOG_KEY, {
39-
api_host: process.env.NEXT_PUBLIC_POSTHOG_HOST || 'https://us.i.posthog.com',
39+
if (typeof window !== 'undefined' && posthogKey) {
40+
posthog.init(posthogKey, {
41+
api_host: posthogHost,
4042
person_profiles: 'identified_only',
4143
// Enable debug mode in development
4244
loaded: (posthog) => {

apps/app/pages/api/consensusRpc/[...slug].ts

Lines changed: 0 additions & 30 deletions
This file was deleted.

apps/app/pages/api/getCheckpoint.ts

Lines changed: 0 additions & 30 deletions
This file was deleted.

apps/app/tailwind.config.js

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,10 @@
11
//@ts-check
22
import plugin from 'tailwindcss/plugin'
3+
import forms from '@tailwindcss/forms'
4+
import typography from '@tailwindcss/typography'
5+
import animate from 'tailwindcss-animate'
36

4-
module.exports = {
7+
export default {
58
content: ["./pages/**/*.{js,ts,jsx,tsx}", "./components/**/*.{js,ts,jsx,tsx}"],
69
darkMode: 'media',
710
theme: {
@@ -270,9 +273,9 @@ module.exports = {
270273
},
271274
},
272275
plugins: [
273-
require("@tailwindcss/forms"),
274-
require("@tailwindcss/typography"),
275-
require("tailwindcss-animate"),
276+
forms,
277+
typography,
278+
animate,
276279
plugin(function ({ addVariant }) {
277280
// Add a `third` variant, ie. `third:pb-0`
278281
addVariant('scrollbar', '&::-webkit-scrollbar');

0 commit comments

Comments
 (0)