openhands
99d9c563d6
fix: Use Optimizer (base) in deploy scripts — Push3 lacks initialize/getLiquidityParams
...
OptimizerV3Push3 is an equivalence-proof contract with only isBullMarket().
It cannot serve as an ERC1967Proxy implementation because it has no initialize()
or getLiquidityParams(). The CI bootstrap was failing because the proxy
deployment reverted when calling initialize() on the Push3 implementation.
Switch deploy scripts to Optimizer.sol (the base UUPS contract) which has the
full interface required by ERC1967Proxy and LiquidityManager.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-02-26 19:37:12 +00:00
openhands
e925538309
fix: Remove dead Optimizer V2/V3 — Push3 is the active optimizer ( #312 )
...
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
2026-02-26 19:37:12 +00:00
openhands
d64b63aff4
feat: deployment scripts, E2E tests, and documentation
...
- DeployBase: shared deployment logic with OptimizerV3 UUPS proxy
- DeployBaseMainnet: Base mainnet configuration (feeDest, WETH, factory)
- DeployLocal: local Anvil deployment with OptimizerV3
- UpgradeOptimizer: UUPS upgrade script for existing proxy
- DEPLOYMENT_RUNBOOK: step-by-step mainnet deployment guide
- E2E tests: recenter position verification, optimizer integration
- Landing page: updated docs for OptimizerV3 and protocol changes
- Remove dead DeployScript2.sol
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
2026-02-13 18:21:49 +00:00
johba
d7c2184ccf
Add Solidity linting with solhint, Foundry formatter, and pre-commit hooks ( #51 )
...
## Changes
### Configuration
- Added .solhint.json with recommended rules + custom config
- 160 char line length (warn)
- Double quotes enforcement (error)
- Explicit visibility required (error)
- Console statements allowed (scripts/tests need them)
- Gas optimization warnings enabled
- Ignores test/helpers/, lib/, out/, cache/, broadcast/
- Added foundry.toml [fmt] section
- 160 char line length
- 4-space tabs
- Double quotes
- Thousands separators for numbers
- Sort imports enabled
- Added .lintstagedrc.json for pre-commit auto-fix
- Runs solhint --fix on .sol files
- Runs forge fmt on .sol files
- Added husky pre-commit hook via lint-staged
### NPM Scripts
- lint:sol - run solhint
- lint:sol:fix - auto-fix solhint issues
- format:sol - format with forge fmt
- format:sol:check - check formatting
- lint / lint:fix - combined commands
### Code Changes
- Added explicit visibility modifiers (internal) to constants in scripts and tests
- Fixed quote style in DeployLocal.sol
- All Solidity files formatted with forge fmt
## Verification
- ✅ forge fmt --check passes
- ✅ No solhint errors (warnings only)
- ✅ forge build succeeds
- ✅ forge test passes (107/107)
resolves #44
Co-authored-by: johba <johba@harb.eth>
Reviewed-on: https://codeberg.org/johba/harb/pulls/51
2025-10-04 15:17:09 +02:00
johba
769fa105b8
added web-app and landing
2025-09-23 14:18:04 +02:00
