@marcos_feitoza/personal-finance-backen-trades-assets 1.1.1 → 1.2.0

package/CHANGELOG.md

@@ -1,3 +1,11 @@
+# [1.2.0](https://github.com/MarcosOps/personal-finance-backend-trades-assets/compare/v1.1.1...v1.2.0) (2025-11-28)
+
+
+### Features
+
+* add confirmation dialog for deletion operations ([1a568c8](https://github.com/MarcosOps/personal-finance-backend-trades-assets/commit/1a568c880a79b438dd421e9314b5785c88db6c36))
+* update readme.me ([82c5d27](https://github.com/MarcosOps/personal-finance-backend-trades-assets/commit/82c5d279aeea11ca73103178c84727c1a0d33e7a))
+
 ## [1.1.1](https://github.com/MarcosOps/personal-finance-backend-trades-assets/compare/v1.1.0...v1.1.1) (2025-11-26)
 
 

package/README.md

@@ -1,22 +1,34 @@
-# Biblioteca de Trades e Ativos - Personal Finance
+# Biblioteca de Trades e Ativos - Personal Finance Backend
 
-Este projeto contém a lógica de negócios específica para o gerenciamento de `Trades` (transações de compra/venda) e `Assets` (ativos como ações, ETFs, etc.).
+Este projeto contém a lógica de negócios e os modelos de dados específicos para o gerenciamento de `Trades` (transações de compra/venda de ativos) e `Assets` (ativos como ações, criptomoedas, etc.).
 
 ## Propósito
 
-Esta biblioteca encapsula as regras para criar e validar trades. Sua principal responsabilidade é, ao receber um novo trade para um símbolo de ação que ainda não existe no banco de dados, buscar os detalhes desse ativo (nome, setor, etc.) usando uma API externa e salvá-lo corretamente.
+O objetivo desta biblioteca é encapsular toda a lógica de criação, leitura, atualização e exclusão (soft-delete) de trades e ativos. Ela interage com serviços externos (como o `market-data-service` para buscar detalhes de ativos) e com o banco de dados via `personal-finance-backend-shared`.
+
+## Conteúdo Principal
+
+- **`routers/trades.py`**: Define endpoints para a criação, listagem e exclusão (soft-delete) de trades.
+- **`routers/assets.py`**: Define endpoints para a listagem e gerenciamento de ativos.
 
 ## Tecnologias
 
-- **Busca de Dados:** Usa a biblioteca `yfinance` para buscar os detalhes de novos ativos.
-- **Banco de Dados:** Interage com o banco de dados através da `personal-finance-backend-shared`.
+- **Framework**: FastAPI (seus routers são usados por outros serviços)
+- **Busca de Dados Externos**: Utiliza `httpx` para comunicação com o `market-data-service`.
+- **Banco de Dados**: Interage com o PostgreSQL através da `personal-finance-backend-shared`.
+- **Dependências Locais**:
+  - `personal-finance-backend-shared`: Para acesso aos modelos, `crud` e utilitários.
 
-## Como Usar
+---
 
-Este projeto não é um serviço executável. Ele é projetado para ser instalado como uma dependência local pelo serviço principal (`core-transactions-service`).
+## Como Usar (Instalação como Dependência)
 
-No `requirements.txt` do serviço principal, adicione a seguinte linha para incluí-lo:
+Este projeto não é um serviço executável por si só. Ele é projetado para ser instalado como uma dependência local por outros serviços do backend (ex: `personal-finance-backend-core`).
+
+Para incluir esta biblioteca como uma dependência editável em outro serviço (por exemplo, no `requirements.txt` do `personal-finance-backend-core`), adicione a seguinte linha no `requirements.txt` do serviço consumidor:
 
 ```
--e ./trades-assets-library
+-e ./personal-finance-backend-trades-assets
 ```
+
+**Importante:** Certifique-se de que o contexto de build do Docker (`Dockerfile`) ou o script de build (`dockerbuild.sh`) do serviço consumidor está copiando corretamente o diretório `personal-finance-backend-trades-assets` para o local esperado para que o `pip install -e` funcione.

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@marcos_feitoza/personal-finance-backen-trades-assets",
-    "version": "1.1.1",
+    "version": "1.2.0",
     "publishConfig": {
         "access": "public"
     },

package/personal_finance_trades_assets/routers/trades.py

@@ -79,9 +79,9 @@ async def create_trade(
     db_trade = models.Trade(**trade_data, asset_id=db_asset.id, user_id=current_user.id)
     
     if db_trade.trade_type == 'buy':
-        logger.info("STOCK PURCHASE recorded", extra={'user_id': current_user.id, 'shares': db_trade.shares, 'symbol': trade_symbol, 'price': db_trade.price})
+        logger.info("STOCK PURCHASE recorded", extra={'user_id': current_user.id, 'shares': float(db_trade.shares), 'symbol': trade_symbol, 'price': float(db_trade.price)})
     elif db_trade.trade_type == 'sell':
-        logger.info("STOCK SALE recorded", extra={'user_id': current_user.id, 'shares': db_trade.shares, 'symbol': trade_symbol, 'price': db_trade.price})
+        logger.info("STOCK SALE recorded", extra={'user_id': current_user.id, 'shares': float(db_trade.shares), 'symbol': trade_symbol, 'price': float(db_trade.price)})
 
     db.add(db_trade)
     db.commit()
@@ -104,11 +104,8 @@ def read_trades(
 ):
     logger.info("Reading trades", extra={'user_id': current_user.id, 'account': investment_account, 'skip': skip, 'limit': limit})
     query = db.query(models.Trade).filter(
-        
         models.Trade.user_id == current_user.id,
-        models.Trade.deleted_at.is_(None) # Filter out soft-deleted trades
-    ,
-        models.Trade.deleted_at.is_(None) # Filter out soft-deleted trades
+        models.Trade.deleted_at.is_(None)
     )
     if investment_account:
         query = query.filter(models.Trade.investment_account == investment_account)
@@ -123,20 +120,20 @@ def delete_trade(
     db: Session = Depends(get_db),
     current_user: models.User = Depends(get_current_user)
 ):
-    logger.info(f"Attempting soft-delete for trade ID: {trade_id} by user: {current_user.id}")
+    logger.info("Attempting soft-delete for trade", extra={'user_id': current_user.id, 'trade_id': trade_id})
     db_trade = db.query(models.Trade).filter(
         models.Trade.id == trade_id,
         models.Trade.user_id == current_user.id,
-        models.Trade.deleted_at.is_(None) # Ensure it's not already deleted
+        models.Trade.deleted_at.is_(None)
     ).first()
 
     if not db_trade:
-        logger.warning(f"Trade ID: {trade_id} not found or already deleted for user: {current_user.id}")
+        logger.warning("Trade not found or already deleted for soft-delete", extra={'user_id': current_user.id, 'trade_id': trade_id})
         raise HTTPException(status_code=404, detail="Trade not found or already deleted")
 
-    db_trade.deleted_at = datetime.utcnow() # Set deletion timestamp
+    db_trade.deleted_at = datetime.utcnow()
     db.add(db_trade)
     db.commit()
     db.refresh(db_trade)
-    logger.info(f"Successfully soft-deleted trade ID: {trade_id} for user: {current_user.id}")
-    return db_trade
+    logger.info("Successfully soft-deleted trade", extra={'user_id': current_user.id, 'trade_id': trade_id})
+    return db_trade