highrise.bot 1.1.2 → 1.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "highrise.bot",
-  "version": "1.1.2",
+  "version": "1.1.4",
   "description": "Unofficial JavaScript SDK for the Highrise platform. Feature-complete WebSocket client with TypeScript support, built for production environments.",
   "keywords": [
     "highrise.bot",

package/src/classes/Actions/Channel.js

@@ -1,327 +1,56 @@
-const crypto = require('crypto');
 const ErrorConstants = require('highrise.bot/src/constants/ErrorConstants');
 
 class ChannelClass {
     constructor(bot) {
         this.bot = bot;
         this.logger = bot.logger;
-
-        this._messageHistory = [];
-        this._maxHistory = 500;
-        this._listeners = new Map();
-        this._tagIndex = new Map();
         
         this.HighriseError = bot.HighriseError;
         this.HighriseResponse = bot.HighriseResponse;
+        
     }
-
+    
     async send(message, tags = []) {
-        let storedMessage
-
         try {
             if (typeof message !== 'string' || message.trim().length === 0) {
                 return this.HighriseError.invalidMessage(message).toResult();
             }
-
-            const normalizedTags = this._normalizeTags(tags);
-
+            
+            if (!Array.isArray(tags) || !tags.every(tag => typeof tag === 'string')) {
+                return new this.HighriseError(
+                    ErrorConstants.CHANNEL.INVALID_TAGS,
+                    `tags must be a valid strings array`,
+                    { tags }
+                ).toResult();
+            }
+            
+            const normalizedTags = tags;
+            
             const payload = {
                 _type: 'ChannelRequest',
                 message: message,
-                tags: normalizedTags,
-                rid: crypto.randomUUID()
+                tags: normalizedTags
             };
-
-            const response = await this.bot._fireAndForget.send(payload);
-
+            
+            const response = await this.bot.fireAndForget.send(payload);
+            
             if (response.success) {
-                storedMessage = {
-                    id: crypto.randomUUID(),
+                const ChannelMessage = {
                     message,
                     tags: normalizedTags,
                     timestamp: Date.now(),
-                    senderId: this.bot.info.user?.id || 'unknown',
                     sent: true
                 };
-
-                this._storeMessage(storedMessage);
-
-                return this.HighriseResponse.success('message', storedMessage);
-
+                return this.HighriseResponse.success('message', ChannelMessage);
             }
-
         } catch (error) {
             return new this.HighriseError(
-                ErrorConstants.CHANNEL.LISTENER_ERROR,
+                ErrorConstants.HIGHRISE_API.MESSAGE_SEND_FAILED,
                 `Failed to send channel message: ${error.message}`,
-                { storedMessage }
+                { message, tags }
             ).toResult();
         }
     }
-
-    on(tags, callback, options = {}) {
-        let listener
-
-        try {
-            if (!callback || typeof callback !== 'function') {
-                return new this.HighriseError(
-                    ErrorConstants.VALIDATION.INVALID_CHANNEL_CALLBACK,
-                    `callback must be a function`
-                ).toResult();
-            }
-
-            const normalizedTags = this._normalizeTags(Array.isArray(tags) ? tags : [tags]);
-
-            const listenerId = crypto.randomUUID();
-            listener = {
-                id: listenerId,
-                callback,
-                tags: normalizedTags,
-                options: {
-                    once: options.once || false,
-                }
-            };
-
-            this._listeners.set(listenerId, listener);
-
-            normalizedTags.forEach(tag => {
-                if (!this._tagIndex.has(tag)) {
-                    this._tagIndex.set(tag, new Set());
-                }
-                this._tagIndex.get(tag).add(listenerId);
-            });
-
-            return this.HighriseResponse.success('listener', {
-                id: listenerId,
-                tags: normalizedTags}
-            );
-
-        } catch (error) {            
-            return new this.HighriseError(
-                ErrorConstants.CHANNEL.LISTENER_ERROR,
-                `Failed to register listener: ${error.message}`,
-                { listener }
-            ).toResult();
-        }
-    }
-
-    once(tags, callback) {
-        return this.on(tags, callback, { once: true });
-    }
-
-    off(listenerId) {
-
-        try {
-            const listener = this._listeners.get(listenerId);
-            if (!listener) {
-                return new this.HighriseError(
-                    ErrorConstants.NOT_FOUND.LISTENER_NOT_FOUND,
-                    `Listener '${listenerId ? `${listenerId.substring(0, 12)}...` : 'EMPTY'}' Not found to be removed.`
-                ).toResult();
-            }
-
-            listener.tags.forEach(tag => {
-                const set = this._tagIndex.get(tag);
-                if (set) {
-                    set.delete(listenerId);
-                    if (set.size === 0) {
-                        this._tagIndex.delete(tag);
-                    }
-                }
-            });
-
-            this._listeners.delete(listenerId);
-
-            return this.HighriseResponse.success('listenerId', listenerId);
-
-        } catch (error) {            
-            return new this.HighriseError(
-                ErrorConstants.CHANNEL.LISTENER_ERROR,
-                `Failed to remove listener: ${error.message}`,
-                { listenerId }
-            ).toResult();
-        }
-    }
-
-    offAll(tags) {
-        const normalizedTags = this._normalizeTags(Array.isArray(tags) ? tags : [tags]);
-
-        const listenersToRemove = new Set();
-
-        normalizedTags.forEach(tag => {
-            const listeners = this._tagIndex.get(tag);
-            if (listeners) {
-                listeners.forEach(listenerId => listenersToRemove.add(listenerId));
-            }
-        });
-
-        let removed = 0;
-        let failed = {
-            count: 0,
-            errors: []
-        }
-        listenersToRemove.forEach(listenerId => {
-            const result = this.off(listenerId);
-            if (result.success) {
-                removed++;
-            } else {
-                failed.errors.push(result.error)
-                failed.count++
-            }
-        });
-
-        return this.HighriseResponse.success('removed', {
-            count: removed,
-            failed,
-            tags: normalizedTags
-        });
-    }
-
-    query(filter = {}) {
-        const {
-            tags = [],
-            since = 0,
-            until = Date.now(),
-            limit = 50
-        } = filter;
-
-        const normalizedTags = this._normalizeTags(Array.isArray(tags) ? tags : [tags]);
-
-        const results = [];
-
-        for (const msg of this._messageHistory) {
-            if (this._matchesFilter(msg, { tags: normalizedTags, since, until })) {
-                results.push(msg);
-                if (results.length >= limit) break;
-            }
-        }
-
-        return this.HighriseResponse.success('query', {
-            count: results.length,
-            messages: results,
-            filter
-        });
-    }
-
-    stats() {
-        return this.HighriseResponse.success('stats', {
-            totalMessages: this._messageHistory.length,
-            activeListeners: this._listeners.size,
-            indexedTags: this._tagIndex.size
-        });
-    }
-
-    storeIncomingMessage(senderId, message, tags) {
-        try {
-            const normalizedTags = this._normalizeTags(tags);
-
-            const storedMsg = {
-                id: crypto.randomUUID(),
-                message,
-                tags: normalizedTags,
-                timestamp: Date.now(),
-                senderId,
-                sent: false
-            };
-
-            this._storeMessage(storedMsg);
-            this._triggerListeners(storedMsg);
-
-            return this.HighriseResponse.success('stored', {
-                id: storedMsg.id,
-                senderId: storedMsg.senderId,
-                tags: storedMsg.tags,
-                timestamp: storedMsg.timestamp
-            });
-
-        } catch (error) {
-            this.logger.error('ChannelManager', 'Failed to store incoming message', { error: error.message });
-            
-            return new this.HighriseError(
-                ErrorConstants.CHANNEL.LISTENER_ERROR,
-                'Failed to store incoming message',
-                { error: error.message }
-            ).toResult();
-        }
-    }
-
-    _triggerListeners(message) {
-        const matchedListeners = new Map();
-
-        message.tags.forEach(tag => {
-            const listeners = this._tagIndex.get(tag);
-            if (listeners) {
-                listeners.forEach(listenerId => {
-                    const listener = this._listeners.get(listenerId);
-                    if (listener && !matchedListeners.has(listenerId)) {
-                        const hasAllTags = listener.tags.every(t => message.tags.includes(t));
-                        if (hasAllTags) {
-                            matchedListeners.set(listenerId, listener);
-                        }
-                    }
-                });
-            }
-        });
-
-        matchedListeners.forEach((listener, listenerId) => {
-            try {
-                listener.callback(message);
-
-                if (listener.options.once) {
-                    this.off(listenerId);
-                }
-
-            } catch (error) {
-                this.logger.error('ChannelManager', 'Listener callback error', { listenerId, error: error.message });
-            }
-        });
-    }
-
-    _normalizeTags(tags) {
-        const normalized = tags
-            .map(tag => {
-                if (typeof tag !== 'string') return '';
-                return tag.toLowerCase()
-                    .trim()
-                    .replace(/[^a-z0-9_:.-]/g, '-')
-                    .replace(/-+/g, '-')
-                    .substring(0, 50);
-            })
-            .filter(tag => tag.length > 0)
-            .filter((tag, index, self) => self.indexOf(tag) === index)
-            .sort();
-
-        return normalized.length > 0 ? normalized : ['global'];
-    }
-
-    _storeMessage(message) {
-        this._messageHistory.unshift(message);
-
-        if (this._messageHistory.length > this._maxHistory) {
-            this._messageHistory.splice(this._maxHistory);
-        }
-    }
-
-    _matchesFilter(message, filter) {
-        const { tags = [], since = 0, until = Date.now() } = filter;
-
-        if (message.timestamp < since || message.timestamp > until) {
-            return false;
-        }
-
-        if (tags.length > 0) {
-            const hasAllTags = tags.every(tag => message.tags.includes(tag));
-            if (!hasAllTags) return false;
-        }
-
-        return true;
-    }
-
-    clear() {
-        this._messageHistory = [];
-        this._listeners.clear();
-        this._tagIndex.clear();
-    }
 }
 
 module.exports = { ChannelClass };

package/src/classes/Actions/Direct.js

@@ -29,7 +29,7 @@ class DirectClass {
                     type: 'text'
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
                 return response;
             }
 
@@ -99,7 +99,7 @@ class DirectClass {
                 type: 'text'
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('broadcast', {
@@ -179,7 +179,7 @@ class DirectClass {
                 payload.world_id = options.world_id;
             }
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('invites', {
@@ -251,7 +251,7 @@ class DirectClass {
                 payload.world_id = options.world_id;
             }
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('invite', {
@@ -296,7 +296,7 @@ class DirectClass {
                     not_joined,
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('conversations', {
@@ -327,7 +327,7 @@ class DirectClass {
                     conversation_id: conversation_id,
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('left', {
@@ -367,7 +367,7 @@ class DirectClass {
                     last_message_id,
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
                 if (response.success) {
                     return this.HighriseResponse.success('messages', {
                         list: response.data.messages || [],

package/src/classes/Actions/Inventory.js

@@ -15,7 +15,7 @@ class InventoryClass {
                 _type: 'GetInventoryRequest'
             };
 
-            const response = await this.bot._requestResponse.send(payload);
+            const response = await this.bot.requestResponse.send(payload);
             if (response.success) {
                 return this.HighriseResponse.success('items', {
                     list: response.data.items || [],
@@ -37,7 +37,7 @@ class InventoryClass {
                     _type: 'GetWalletRequest'
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
                 
                 if (response.success) {
                     const walletData = (response.data.content || []).reduce((acc, item) => {
@@ -85,7 +85,7 @@ class InventoryClass {
                     amount: amount
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
                 if (response.success && response.data.result === 'insufficient_funds') {
                     return new this.HighriseError(
                         ErrorConstants.ECONOMY.INSUFFICIENT_FUNDS,
@@ -127,7 +127,7 @@ class InventoryClass {
                     payment_method: payment_method
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
 
                 if (response.success && response.data.result === 'insufficient_funds') {
                     return new this.HighriseError(
@@ -169,7 +169,7 @@ class InventoryClass {
                     item_id: item_id
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
                 console.log(response.data)
                 
                 if (response.success && response.result === 'success') {

package/src/classes/Actions/Outfit.js

@@ -24,7 +24,7 @@ class OutfitClass {
                 outfit: outfit
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('outfit', {

package/src/classes/Actions/Player.js

@@ -33,7 +33,7 @@ class PlayerClass {
                 destination: destination
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('walked', {
@@ -74,7 +74,7 @@ class PlayerClass {
                 anchor: anchor
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('sat', {
@@ -119,7 +119,7 @@ class PlayerClass {
                 destination: { x, y, z, facing }
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('teleported', {
@@ -164,7 +164,7 @@ class PlayerClass {
                 room_id: room_id
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('transported', {
@@ -201,7 +201,7 @@ class PlayerClass {
                 target_user_id: user_id
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('emoted', {
@@ -246,7 +246,7 @@ class PlayerClass {
                 target_user_id: user_id
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('reacted', {
@@ -312,7 +312,7 @@ class PlayerClass {
                 gold_bar: PayladGoldBars[gold_bar]
             };
 
-            const response = await this.bot._fireAndForget.send(payload);
+            const response = await this.bot.fireAndForget.send(payload);
 
             if (response.success) {
                 return this.HighriseResponse.success('tipped', {
@@ -383,7 +383,7 @@ class PlayerClass {
                     user_id: user_id
                 };
 
-                const response = await this.bot._requestResponse.send(payload);
+                const response = await this.bot.requestResponse.send(payload);
                 if (response.success) {
                     return this.HighriseResponse.success('outfit', {
                         user_id,
@@ -423,7 +423,7 @@ class PlayerClass {
                     action_length: null
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('kicked', {
@@ -471,7 +471,7 @@ class PlayerClass {
                     action_length: duration
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('banned', {
@@ -519,7 +519,7 @@ class PlayerClass {
                     action_length: duration
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('muted', {
@@ -550,7 +550,7 @@ class PlayerClass {
                     action_length: 1
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('unmuted', {
@@ -581,7 +581,7 @@ class PlayerClass {
                     action_length: null
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
 
                 if (response.success) {
                     return this.HighriseResponse.success('unbanned', {

package/src/classes/Actions/Public.js

@@ -28,7 +28,7 @@ class MessageClass {
                     message: msg
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
                 return response
             }
 
@@ -100,7 +100,7 @@ class WhisperClass {
                     whisper_target_id: user_id
                 };
 
-                const response = await this.bot._fireAndForget.send(payload);
+                const response = await this.bot.fireAndForget.send(payload);
                 return response;
             }